update: Fix key mapping in ProjectConfig to use 'id' instead of empty string

FarhanAnjum-opti · FarhanAnjum-opti · commit a2a10d991a13 · 2025-05-27T15:43:15.000+06:00
diff --git a/optimizely/project_config.py b/optimizely/project_config.py
@@ -98,7 +98,7 @@ def __init__(self, datafile: str | bytes, logger: Logger, error_handler: Any):
         for attribute in self.attributes:
             self.attribute_id_to_key_map[attribute['id']] = attribute['key']
         self.attribute_id_map: dict[str, entities.Attribute] = self._generate_key_map(
-            self.attributes, '', entities.Attribute
+            self.attributes, 'id', entities.Attribute
         )
         self.audience_id_map: dict[str, entities.Audience] = self._generate_key_map(
             self.audiences, 'id', entities.Audience

Original file line number	Diff line number	Diff line change
`@@ -98,7 +98,7 @@ def __init__(self, datafile: str \| bytes, logger: Logger, error_handler: Any):`
`98`	`98`	`for attribute in self.attributes:`
`99`	`99`	`self.attribute_id_to_key_map[attribute['id']] = attribute['key']`
`100`	`100`	`self.attribute_id_map: dict[str, entities.Attribute] = self._generate_key_map(`
`101`		`- self.attributes, '', entities.Attribute`
	`101`	`+ self.attributes, 'id', entities.Attribute`
`102`	`102`	`)`
`103`	`103`	`self.audience_id_map: dict[str, entities.Audience] = self._generate_key_map(`
`104`	`104`	`self.audiences, 'id', entities.Audience`