Skip to content

Commit 48ccd22

Browse files
committed
Clarify logic
1 parent 1098767 commit 48ccd22

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

libs/labelbox/src/labelbox/schema/api_key.py

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -45,16 +45,22 @@ def created_by(self) -> Optional["User"]:
4545
"""Gets the User who created this API key.
4646
4747
Returns:
48-
Optional[User]: The User who created this API key, or None if not available
48+
Optional[User]: The User who created this API key, or None if not available.
4949
"""
5050
if not hasattr(self, "_created_by"):
51-
self._created_by = (
52-
self.client._get_single(User, self.created_by_user_id)
51+
# Use created_by_user_id if present, otherwise fall back to user_id
52+
# (typically needed for older API keys where created_by_user_id is NULL)
53+
user_id_to_fetch = (
54+
self.created_by_user_id
5355
if self.created_by_user_id is not None
54-
else self.client._get_single(User, self.user_id)
55-
if self.user_id is not None
56+
else self.user_id
57+
)
58+
self._created_by = (
59+
self.client._get_single(User, user_id_to_fetch)
60+
if user_id_to_fetch
5661
else None
5762
)
63+
5864
return self._created_by
5965

6066
@property

0 commit comments

Comments
 (0)