Skip to content

Commit 017b0d2

Browse files
committed
did naming changes
1 parent 4aa8c44 commit 017b0d2

File tree

5 files changed

+70
-70
lines changed

5 files changed

+70
-70
lines changed

core-api/src/main/java/com/optimizely/ab/Optimizely.java

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -669,22 +669,22 @@ String getFeatureVariableValueForType(@Nonnull String featureKey,
669669

670670
Object convertedValue = convertStringToType(variableValue, variableType);
671671
Map<String, Object> decisionInfo = new HashMap<>();
672-
decisionInfo.put(DecisionInfoEnums.GetFeatureVariableDecisionInfo.FEATURE_KEY.toString(), featureKey);
673-
decisionInfo.put(DecisionInfoEnums.GetFeatureVariableDecisionInfo.FEATURE_ENABLED.toString(), featureEnabled);
674-
decisionInfo.put(DecisionInfoEnums.GetFeatureVariableDecisionInfo.VARIABLE_KEY.toString(), variableKey);
675-
decisionInfo.put(DecisionInfoEnums.GetFeatureVariableDecisionInfo.VARIABLE_TYPE.toString(), variableType);
676-
decisionInfo.put(DecisionInfoEnums.GetFeatureVariableDecisionInfo.VARIABLE_VALUE.toString(), convertedValue);
672+
decisionInfo.put(DecisionInfoEnums.FeatureVariableDecisionInfo.FEATURE_KEY.toString(), featureKey);
673+
decisionInfo.put(DecisionInfoEnums.FeatureVariableDecisionInfo.FEATURE_ENABLED.toString(), featureEnabled);
674+
decisionInfo.put(DecisionInfoEnums.FeatureVariableDecisionInfo.VARIABLE_KEY.toString(), variableKey);
675+
decisionInfo.put(DecisionInfoEnums.FeatureVariableDecisionInfo.VARIABLE_TYPE.toString(), variableType);
676+
decisionInfo.put(DecisionInfoEnums.FeatureVariableDecisionInfo.VARIABLE_VALUE.toString(), convertedValue);
677677
if (featureDecision.decisionSource != null && featureDecision.decisionSource.equals(FeatureDecision.DecisionSource.EXPERIMENT)) {
678-
decisionInfo.put(DecisionInfoEnums.GetFeatureVariableDecisionInfo.SOURCE_EXPERIMENT_KEY.toString(), featureDecision.experiment.getKey());
679-
decisionInfo.put(DecisionInfoEnums.GetFeatureVariableDecisionInfo.SOURCE_VARIATION_KEY.toString(), featureDecision.variation.getKey());
680-
decisionInfo.put(DecisionInfoEnums.GetFeatureVariableDecisionInfo.SOURCE.toString(), featureDecision.decisionSource);
678+
decisionInfo.put(DecisionInfoEnums.FeatureVariableDecisionInfo.SOURCE_EXPERIMENT_KEY.toString(), featureDecision.experiment.getKey());
679+
decisionInfo.put(DecisionInfoEnums.FeatureVariableDecisionInfo.SOURCE_VARIATION_KEY.toString(), featureDecision.variation.getKey());
680+
decisionInfo.put(DecisionInfoEnums.FeatureVariableDecisionInfo.SOURCE.toString(), featureDecision.decisionSource);
681681
} else {
682-
decisionInfo.put(DecisionInfoEnums.GetFeatureVariableDecisionInfo.SOURCE_EXPERIMENT_KEY.toString(), null);
683-
decisionInfo.put(DecisionInfoEnums.GetFeatureVariableDecisionInfo.SOURCE_VARIATION_KEY.toString(), null);
684-
decisionInfo.put(DecisionInfoEnums.GetFeatureVariableDecisionInfo.SOURCE.toString(), FeatureDecision.DecisionSource.ROLLOUT);
682+
decisionInfo.put(DecisionInfoEnums.FeatureVariableDecisionInfo.SOURCE_EXPERIMENT_KEY.toString(), null);
683+
decisionInfo.put(DecisionInfoEnums.FeatureVariableDecisionInfo.SOURCE_VARIATION_KEY.toString(), null);
684+
decisionInfo.put(DecisionInfoEnums.FeatureVariableDecisionInfo.SOURCE.toString(), FeatureDecision.DecisionSource.ROLLOUT);
685685
}
686686
notificationCenter.sendNotifications(NotificationCenter.NotificationType.Decision,
687-
NotificationCenter.OnDecisionNotificationType.GET_FEATURE_VARIABLE.toString(),
687+
NotificationCenter.DecisionNotificationType.FEATURE_VARIABLE.toString(),
688688
userId,
689689
copiedAttributes,
690690
decisionInfo);

core-api/src/main/java/com/optimizely/ab/notification/DecisionInfoEnums.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
package com.optimizely.ab.notification;
1818

1919
public class DecisionInfoEnums {
20-
public enum GetFeatureVariableDecisionInfo {
20+
public enum FeatureVariableDecisionInfo {
2121
SOURCE_EXPERIMENT_KEY("source_experiment_key"),
2222
SOURCE_VARIATION_KEY("source_variation_key"),
2323
FEATURE_KEY("feature_key"),
@@ -29,7 +29,7 @@ public enum GetFeatureVariableDecisionInfo {
2929

3030
private final String key;
3131

32-
GetFeatureVariableDecisionInfo(String key) {
32+
FeatureVariableDecisionInfo(String key) {
3333
this.key = key;
3434
}
3535

core-api/src/main/java/com/optimizely/ab/notification/DecisionNotificationListenerInterface.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ public interface DecisionNotificationListenerInterface {
2727
* @param type - The notification type.
2828
* @param userId - The userId passed into activate.
2929
* @param attributes - The filtered attribute list passed into activate
30-
* @param decisionInfo - The decision Information containing all parameters passed in api.
30+
* @param decisionInfo - The decision information containing all parameters passed in API.
3131
*/
3232
void onDecision(@Nonnull String type,
3333
@Nonnull String userId,

core-api/src/main/java/com/optimizely/ab/notification/NotificationCenter.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,14 +33,14 @@
3333
* more flexible.
3434
*/
3535
public class NotificationCenter {
36-
public enum OnDecisionNotificationType {
37-
IS_FEATURE_ENABLED("feature"),
38-
GET_FEATURE_VARIABLE("feature_variable"),
36+
public enum DecisionNotificationType {
37+
FEATURE("feature"),
38+
FEATURE_VARIABLE("feature_variable"),
3939
EXPERIMENT("experiment");
4040

4141
private final String key;
4242

43-
OnDecisionNotificationType(String key) {
43+
DecisionNotificationType(String key) {
4444
this.key = key;
4545
}
4646

0 commit comments

Comments
 (0)