Skip to content

Commit 5063b23

Browse files
[FSSDK-11547] fsc fix attempt 3
1 parent e818894 commit 5063b23

File tree

1 file changed

+3
-10
lines changed

1 file changed

+3
-10
lines changed

OptimizelySDK/Bucketing/DecisionService.cs

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -915,10 +915,7 @@ ProjectConfig config
915915
var infoMessage = $"Holdout \"{holdout.Key}\" is not running.";
916916
Logger.Log(LogLevel.INFO, infoMessage);
917917
reasons.AddInfo(infoMessage);
918-
return Result<FeatureDecision>.NewResult(
919-
new FeatureDecision(null, null, FeatureDecision.DECISION_SOURCE_HOLDOUT),
920-
reasons
921-
);
918+
return Result<FeatureDecision>.NullResult(reasons);
922919
}
923920

924921
var audienceResult = ExperimentUtils.DoesUserMeetAudienceConditions(
@@ -934,10 +931,7 @@ ProjectConfig config
934931
if (!audienceResult.ResultObject)
935932
{
936933
reasons.AddInfo($"User \"{userId}\" does not meet conditions for holdout ({holdout.Key}).");
937-
return Result<FeatureDecision>.NewResult(
938-
new FeatureDecision(null, null, FeatureDecision.DECISION_SOURCE_HOLDOUT),
939-
reasons
940-
);
934+
return Result<FeatureDecision>.NullResult(reasons);
941935
}
942936

943937
var attributes = user.GetAttributes();
@@ -955,8 +949,7 @@ ProjectConfig config
955949
}
956950

957951
reasons.AddInfo($"User \"{userId}\" is not bucketed into holdout variation \"{holdout.Key}\".");
958-
959-
return Result<FeatureDecision>.NewResult(null, reasons);
952+
return Result<FeatureDecision>.NullResult(reasons);
960953
}
961954
/// <summary>
962955
/// Finds a validated forced decision.

0 commit comments

Comments
 (0)