From 9fdc0eeb8ec0654a0f79c05ec339470850ef9b90 Mon Sep 17 00:00:00 2001 From: Anders Kaseorg Date: Thu, 27 Jun 2024 11:16:37 -0700 Subject: [PATCH] scim: Fix PLR1704 Redefining argument with the local name `path`. Signed-off-by: Anders Kaseorg --- zerver/lib/scim.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/zerver/lib/scim.py b/zerver/lib/scim.py index affad22b2d..7da62f8331 100644 --- a/zerver/lib/scim.py +++ b/zerver/lib/scim.py @@ -229,19 +229,19 @@ class ZulipSCIMUser(SCIMUser): value = {path: value} assert isinstance(value, dict) - for path, val in (value or {}).items(): - if path.first_path == ("userName", None, None): + for attr_path, val in (value or {}).items(): + if attr_path.first_path == ("userName", None, None): assert isinstance(val, str) self.change_delivery_email(val) - elif path.first_path == ("name", "formatted", None): + elif attr_path.first_path == ("name", "formatted", None): # TODO: Add support name_formatted_included=False config like we do # for updates via PUT. assert isinstance(val, str) self.change_full_name(val) - elif path.first_path == ("active", None, None): + elif attr_path.first_path == ("active", None, None): assert isinstance(val, bool) self.change_is_active(val) - elif path.first_path == ("role", None, None): + elif attr_path.first_path == ("role", None, None): assert isinstance(val, str) self.change_role(val) else: