Skip to content

Commit 7ce768d

Browse files
committed
pass options to DecisonReasons
1 parent 43f7b01 commit 7ce768d

File tree

2 files changed

+7
-6
lines changed

2 files changed

+7
-6
lines changed

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

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,8 +102,7 @@ public OptimizelyDecision decide(@Nonnull String key,
102102
Boolean sentEvent = false;
103103
Boolean flagEnabled = false;
104104
List<OptimizelyDecideOption> allOptions = getAllOptions(options);
105-
Boolean includeReasons = allOptions.contains(OptimizelyDecideOption.INCLUDE_REASONS);
106-
DecisionReasons decisionReasons = new DecisionReasons(includeReasons);
105+
DecisionReasons decisionReasons = new DecisionReasons(allOptions);
107106

108107
Map<String, ?> copiedAttributes = new HashMap<>(attributes);
109108
FeatureDecision flagDecision = optimizely.decisionService.getVariationForFeature(

core-api/src/main/java/com/optimizely/ab/optimizelydecision/DecisionReasons.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,21 +16,23 @@
1616
*/
1717
package com.optimizely.ab.optimizelydecision;
1818

19+
import javax.annotation.Nonnull;
1920
import java.util.ArrayList;
21+
import java.util.Collections;
2022
import java.util.List;
2123

2224
public class DecisionReasons {
2325

24-
private boolean includeReasons;
2526
private final List<String> errors = new ArrayList<>();
2627
private final List<String> logs = new ArrayList<>();
28+
private boolean includeReasons;
2729

28-
public DecisionReasons(boolean includeReasons) {
29-
this.includeReasons = includeReasons;
30+
public DecisionReasons(@Nonnull List<OptimizelyDecideOption> options) {
31+
this.includeReasons = options.contains(OptimizelyDecideOption.INCLUDE_REASONS);
3032
}
3133

3234
public DecisionReasons() {
33-
this(false);
35+
this(Collections.emptyList());
3436
}
3537

3638
public void addError(String message) {

0 commit comments

Comments
 (0)