From f5eba7aaa4d4b4f93c01e812ab2460b4738f9e06 Mon Sep 17 00:00:00 2001
From: Benjami <benjami94@gmail.com>
Date: Tue, 1 Aug 2023 07:53:52 +0200
Subject: [PATCH] Merge oauth onchange

---
 energy_communities/models/auth_oauth_provider.py | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/energy_communities/models/auth_oauth_provider.py b/energy_communities/models/auth_oauth_provider.py
index d3092b5a4..c92eb3a72 100644
--- a/energy_communities/models/auth_oauth_provider.py
+++ b/energy_communities/models/auth_oauth_provider.py
@@ -34,7 +34,8 @@ class OAuthProvider(models.Model):
         if not self.superuser_pwd:
             raise UserError("Admin provider doesn't have a valid superuser password")
 
-    def _update_urls(self):
+    @api.onchange('root_endpoint', 'realm_name')
+    def _onchange_update_endpoints(self):
         if self.is_keycloak_provider and self.root_endpoint and self.realm_name:
             self.admin_user_endpoint = URL_ADMIN_USERS.format(**{
                 'root_endpoint': self.root_endpoint, 'realm_name': self.realm_name
@@ -59,14 +60,6 @@ class OAuthProvider(models.Model):
                 cliend_id=self.client_id,
             )
 
-    @api.onchange('root_endpoint')
-    def _onchange_root_endpoint(self):
-        self._update_urls()
-
-    @api.onchange('realm_name')
-    def _onchange_realm_name(self):
-        self._update_urls()
-
     def get_auth_link(self):
         self.ensure_one()
         provider_dict = [
-- 
GitLab