Skip to content

Commit 0d4a202

Browse files
Merge pull request #18 from optimizely/mnoman/decisionListenerFix
Fix: Removed camelCase to snake case function
2 parents 65ba456 + 7688a97 commit 0d4a202

File tree

2 files changed

+1
-21
lines changed

2 files changed

+1
-21
lines changed

android/src/main/java/com/optimizely/optimizely_flutter_sdk/OptimizelyFlutterClient.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@
4545
import com.optimizely.optimizely_flutter_sdk.helper_classes.ArgumentsParser;
4646

4747
import static com.optimizely.optimizely_flutter_sdk.helper_classes.Constants.*;
48-
import static com.optimizely.optimizely_flutter_sdk.helper_classes.Utils.convertKeysCamelCaseToSnakeCase;
4948

5049
import java.util.Collections;
5150
import java.util.LinkedHashMap;
@@ -430,7 +429,7 @@ protected void addNotificationListener(ArgumentsParser argumentsParser, @NonNull
430429
notificationMap.put(DecisionListenerKeys.TYPE, decisionNotification.getType());
431430
notificationMap.put(DecisionListenerKeys.USER_ID, decisionNotification.getUserId());
432431
notificationMap.put(DecisionListenerKeys.ATTRIBUTES, decisionNotification.getAttributes());
433-
notificationMap.put(DecisionListenerKeys.DECISION_INFO, convertKeysCamelCaseToSnakeCase(decisionNotification.getDecisionInfo()));
432+
notificationMap.put(DecisionListenerKeys.DECISION_INFO, decisionNotification.getDecisionInfo());
434433
invokeNotification(id, NotificationType.DECISION, notificationMap);
435434
});
436435
notificationIdsTracker.put(id, notificationId);

android/src/main/java/com/optimizely/optimizely_flutter_sdk/helper_classes/Utils.java

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -25,25 +25,6 @@
2525
import com.optimizely.ab.optimizelydecision.OptimizelyDecideOption;
2626

2727
public class Utils {
28-
public static Map<String, ?> convertKeysCamelCaseToSnakeCase(Map<String, ?> decisionInfo) {
29-
Map<String, Object> decisionInfoCopy = new HashMap<>(decisionInfo);
30-
31-
if (decisionInfo.containsKey(SOURCE_INFO) && decisionInfo.get(SOURCE_INFO) instanceof Map) {
32-
Map<String, String> sourceInfo = (Map<String, String>) decisionInfoCopy.get(SOURCE_INFO);
33-
Map<String, String> sourceInfoCopy = new HashMap<>(sourceInfo);
34-
35-
for (String key : sourceInfo.keySet()) {
36-
sourceInfoCopy.put(CaseFormat.UPPER_CAMEL.to(CaseFormat.LOWER_UNDERSCORE, key), sourceInfoCopy.remove(key));
37-
}
38-
decisionInfoCopy.remove(SOURCE_INFO);
39-
decisionInfoCopy.put(SOURCE_INFO, sourceInfoCopy);
40-
}
41-
42-
for (String key : decisionInfo.keySet()) {
43-
decisionInfoCopy.put(CaseFormat.UPPER_CAMEL.to(CaseFormat.LOWER_UNDERSCORE, key), decisionInfoCopy.remove(key));
44-
}
45-
return decisionInfoCopy;
46-
}
4728

4829
public static List<OptimizelyDecideOption> getDecideOptions(List<String> options) {
4930
if(options == null || options.isEmpty()) {

0 commit comments

Comments
 (0)