cleanup classicui controlpanel

Author: mamico

news/73.feature

@@ -0,0 +1 @@
+Make controlpanel compatible with collective.regenv. @mamico

src/pas/plugins/oidc/controlpanel/classic.py

@@ -12,147 +12,29 @@
 @adapter(IPloneSiteRoot)
 @implementer(IOIDCSettings)
 class OIDCControlPanelAdapter:
+    propertymap = None
+
     def __init__(self, context):
         self.context = context
         self.portal = api.portal.get()
         self.encoding = "utf-8"
         self.settings = self.portal.acl_users[PLUGIN_ID]
+        self.propertymap = {prop["id"]: prop for prop in self.settings.propertyMap()}
 
-    @property
-    def issuer(self):
-        return self.settings.getProperty("issuer")
-
-    @issuer.setter
-    def issuer(self, value):
-        self.settings.issuer = value
-
-    @property
-    def client_id(self):
-        return self.settings.getProperty("client_id")
-
-    @client_id.setter
-    def client_id(self, value):
-        self.settings.client_id = value
-
-    @property
-    def client_secret(self):
-        return self.settings.getProperty("client_secret")
-
-    @client_secret.setter
-    def client_secret(self, value):
-        self.settings.client_secret = value
-
-    @property
-    def redirect_uris(self):
-        return self.settings.getProperty("redirect_uris")
-
-    @redirect_uris.setter
-    def redirect_uris(self, value):
-        self.settings.redirect_uris = value
-
-    @property
-    def use_session_data_manager(self):
-        return self.settings.getProperty("use_session_data_manager")
-
-    @use_session_data_manager.setter
-    def use_session_data_manager(self, value):
-        self.settings.use_session_data_manager = value
-
-    @property
-    def create_user(self):
-        return self.settings.getProperty("create_user")
-
-    @create_user.setter
-    def create_user(self, value):
-        self.settings.create_user = value
-
-    @property
-    def create_groups(self):
-        return self.settings.getProperty("create_groups")
-
-    @create_groups.setter
-    def create_groups(self, value):
-        self.settings.create_groups = value
-
-    @property
-    def user_property_as_groupid(self):
-        return self.settings.getProperty("user_property_as_groupid")
-
-    @user_property_as_groupid.setter
-    def user_property_as_groupid(self, value):
-        self.settings.user_property_as_groupid = value
-
-    @property
-    def allowed_groups(self):
-        return self.settings.getPropertty("allowed_groups")
-
-    @allowed_groups.setter
-    def allowed_groups(self, value):
-        self.settings.allowed_groups = value
-
-    @property
-    def create_ticket(self):
-        return self.settings.getProperty("create_ticket")
-
-    @create_ticket.setter
-    def create_ticket(self, value):
-        self.settings.create_ticket = value
-
-    @property
-    def create_restapi_ticket(self):
-        return self.settings.getProperty("create_restapi_ticket")
-
-    @create_restapi_ticket.setter
-    def create_restapi_ticket(self, value):
-        self.settings.create_restapi_ticket = value
-
-    @property
-    def scope(self):
-        return self.settings.getProperty("scope")
-
-    @scope.setter
-    def scope(self, value):
-        self.settings.scope = value
-
-    @property
-    def use_pkce(self):
-        return self.settings.getProperty("use_pkce")
-
-    @use_pkce.setter
-    def use_pkce(self, value):
-        self.settings.use_pkce = value
-
-    @property
-    def use_deprecated_redirect_uri_for_logout(self):
-        return self.settings.getProperty("use_deprecated_redirect_uri_for_logout")
-
-    @use_deprecated_redirect_uri_for_logout.setter
-    def use_deprecated_redirect_uri_for_logout(self, value):
-        self.settings.use_deprecated_redirect_uri_for_logout = value
-
-    @property
-    def use_modified_openid_schema(self):
-        return self.settings.getProperty("use_modified_openid_schema")
-
-    @use_modified_openid_schema.setter
-    def use_modified_openid_schema(self, value):
-        self.settings.use_modified_openid_schema = value
-
-    @property
-    def user_property_as_userid(self):
-        return self.settings.getProperty("user_property_as_userid")
-
-    @user_property_as_userid.setter
-    def user_property_as_userid(self, value):
-        self.settings.user_property_as_userid = value
-
-    @property
-    def identity_domain_name(self):
-        return self.settings.getProperty("identity_domain_name")
+    def __getattr__(self, name):
+        if self.propertymap and name in self.propertymap:
+            return self.settings.getProperty(name)
+        else:
+            raise AttributeError(f"{name} not in oidcsettings")
 
-    @identity_domain_name.setter
-    def identity_domain_name(self, value):
-        self.settings.identity_domain_name = value
+    def __setattr__(self, name, value):
+        if self.propertymap and name in self.propertymap:
+            if "w" in self.propertymap[name].get("mode", ""):
+                return setattr(self.settings, name, value)
+            else:
+                raise TypeError(f"{name} readonly in oidcsettings")
+        else:
+            super().__setattr__(name, value)
 
 
 class OIDCSettingsForm(controlpanel.RegistryEditForm):
@@ -167,9 +49,9 @@ def getContent(self):
 
     def updateWidgets(self):
         super().updateWidgets()
-        modes = {p["id"]: p["mode"] for p in self.getContent().settings.propertyMap()}
-        for id, widget in self.widgets.items():
-            if id in modes and "w" not in modes.get(id):
+        pmap = self.getContent().settings.propertymap
+        for name, widget in self.widgets.items():
+            if name in pmap and "w" not in pmap[id].get("mode", ""):
                 widget.mode = DISPLAY_MODE
 
     def applyChanges(self, data):