chore: remove usage of getattr()

JohnVillalovos · JohnVillalovos · commit a8f4c236c6a3 · 2021-03-14T16:21:45.000-07:00
Remove usage of getattr(self, "_update_uses_post", False)

Instead add it to class and set default value to False.
diff --git a/gitlab/mixins.py b/gitlab/mixins.py
@@ -331,6 +331,7 @@ class UpdateMixin(_RestManagerBase):
     _parent = Optional[base.RESTObject]
     _parent_attrs = Dict[str, Any]
     _path: Optional[str]
+    _update_uses_post: bool = False
     gitlab: gitlab.Gitlab
 
     def _check_missing_update_attrs(self, data: Dict[str, Any]) -> None:
@@ -357,7 +358,7 @@ def _get_update_method(
         Returns:
             object: http_put (default) or http_post
         """
-        if getattr(self, "_update_uses_post", False):
+        if self._update_uses_post:
             http_method = self.gitlab.http_post
         else:
             http_method = self.gitlab.http_put
