Skip to content

Commit

Permalink
optimizely_client validity fixed
Browse files Browse the repository at this point in the history
  • Loading branch information
ozayr-zaviar committed Nov 16, 2021
1 parent 1977527 commit d2f1975
Showing 1 changed file with 12 additions and 4 deletions.
16 changes: 12 additions & 4 deletions lib/optimizely/optimizely_user_context.rb
Original file line number Diff line number Diff line change
Expand Up @@ -102,8 +102,10 @@ def decide_all(options = nil)
# @return - true if the forced decision has been set successfully.

def set_forced_decision(context, decision)
unless @optimizely_client&.is_valid
return false
end
flag_key = context[:flag_key]
return false if @optimizely_client&.get_optimizely_config.nil?
return false if flag_key.nil?

@forced_decision_mutex.synchronize { @forced_decisions[context] = decision }
Expand All @@ -126,7 +128,9 @@ def find_forced_decision(context)
# @return - A variation key or nil if forced decisions are not set for the parameters.

def get_forced_decision(context)
return nil if @optimizely_client&.get_optimizely_config.nil?
unless @optimizely_client&.is_valid
return nil
end

find_forced_decision(context)
end
Expand All @@ -138,7 +142,9 @@ def get_forced_decision(context)
# @return - true if the forced decision has been removed successfully.

def remove_forced_decision(context)
return false if @optimizely_client&.get_optimizely_config.nil?
unless @optimizely_client&.is_valid
return false
end

deleted = false
@forced_decision_mutex.synchronize do
Expand All @@ -155,7 +161,9 @@ def remove_forced_decision(context)
# @return - true if forced decisions have been removed successfully.

def remove_all_forced_decision
return false if @optimizely_client&.get_optimizely_config.nil?
unless @optimizely_client&.is_valid
return false
end

@forced_decision_mutex.synchronize { @forced_decisions.clear }
true
Expand Down

0 comments on commit d2f1975

Please sign in to comment.