change in logic to get experiment_id by key or rollout_id

optimizely · FarhanAnjum-opti · May 16, 2025 · Mar 24, 2025 · Apr 4, 2025 · Apr 4, 2025
commit 3c1903b2054ae5d76a84b82f02c9b1623a02ccb1
diff --git a/optimizely/project_config.py b/optimizely/project_config.py
@@ -728,13 +728,7 @@ def get_experiment_id_by_key_or_rollout_id(self, key: str, rollout_id: Optional[
         Returns:
             Optional[str]: The experiment ID if found, otherwise None.
         """
-        # Try getting the experiment from experiment_key_map first
-        if key:
-            experiment = self.get_experiment_from_key(key)
-            if experiment:
-                return experiment.id
-
-        # If key is not found in experiment_key_map, check a specific rollout (if provided)
+        # Check a specific rollout (if provided)
         if rollout_id:
             rollout = self.get_rollout_from_id(rollout_id)
             if rollout:
@@ -743,4 +737,10 @@ def get_experiment_id_by_key_or_rollout_id(self, key: str, rollout_id: Optional[
                     if experiment.key == key:
                         return experiment.id
 
+        # Try getting the experiment from experiment_key_map
+        if key:
+            experiment = self.get_experiment_from_key(key)
+            if experiment:
+                return experiment.id
+
         return None