optimizely · msohailhussain · Dec 22, 2021 · Dec 13, 2021 · Dec 13, 2021 · Dec 14, 2021
diff --git a/optimizely/event/user_event_factory.py b/optimizely/event/user_event_factory.py
@@ -49,11 +49,12 @@ def create_impression_event(
         if activated_experiment:
             experiment_id = activated_experiment.id
 
-        if variation_id and experiment_id:
-            variation = project_config.get_variation_from_id_by_experiment_id(experiment_id, variation_id)
-        # need this condition when we send events involving forced decisions
-        elif variation_id and flag_key:
+        if variation_id and flag_key:
             variation = project_config.get_flag_variation(flag_key, 'id', variation_id)
+        if not variation and variation_id and experiment_id:
+            # need this condition when we send events involving forced decisions
+            variation = project_config.get_variation_from_id_by_experiment_id(experiment_id, variation_id)
+
         event_context = user_event.EventContext(
             project_config.account_id, project_config.project_id, project_config.revision, project_config.anonymize_ip,
         )

diff --git a/optimizely/project_config.py b/optimizely/project_config.py
@@ -612,8 +612,8 @@ def get_variation_from_id_by_experiment_id(self, experiment_id, variation_id):
                 variation_id in self.variation_id_map_by_experiment_id[experiment_id]):
             return self.variation_id_map_by_experiment_id[experiment_id][variation_id]
 
-        self.logger.error('Variation with id "%s" not defined in the datafile for experiment "%s".',
-                          variation_id, experiment_id)
+        self.logger.error('Variation with id "%s" not defined in the datafile for experiment "%s".' %
+                          (variation_id, experiment_id))
 
         return {}
 
@@ -628,8 +628,8 @@ def get_variation_from_key_by_experiment_id(self, experiment_id, variation_key):
                 variation_key in self.variation_key_map_by_experiment_id[experiment_id]):
             return self.variation_key_map_by_experiment_id[experiment_id][variation_key]
 
-        self.logger.error('Variation with key "%s" not defined in the datafile for experiment "%s".',
-                              variation_key, experiment_id)
+        self.logger.error('Variation with key "%s" not defined in the datafile for experiment "%s".' %
+                          (variation_key, experiment_id))
 
         return {}
 
@@ -661,8 +661,9 @@ def get_flag_variation(self, flag_key, variation_attribute, target_value):
             return None
 
         variations = self.flag_variations_map.get(flag_key)
-        for variation in variations:
-            if getattr(variation, variation_attribute) == target_value:
-                return variation
+        if variations:
+            for variation in variations:
+                if getattr(variation, variation_attribute) == target_value:
+                    return variation
 
         return None