openwisp-users 1.2.1__tar.gz → 1.2.2__tar.gz

{openwisp_users-1.2.1 → openwisp_users-1.2.2}/CHANGES.rst

@@ -1,6 +1,15 @@
 Changelog
 =========
 
+Version 1.2.2 [2026-04-25]
+--------------------------
+
+Bugfixes
+~~~~~~~~
+
+- Added SocialAccount inline to UserAdmin when needed `#501
+  <https://github.com/openwisp/openwisp-users/issues/501>`_
+
 Version 1.2.1 [2026-04-22]
 --------------------------
 

{openwisp_users-1.2.1/openwisp_users.egg-info → openwisp_users-1.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openwisp-users
-Version: 1.2.1
+Version: 1.2.2
 Summary: Provides basic multi-tenancy features for OpenWISP (using the Django web-framework)
 Home-page: http://openwisp.org
 Download-URL: https://github.com/openwisp/openwisp-users/releases

{openwisp_users-1.2.1 → openwisp_users-1.2.2}/openwisp_users/__init__.py

@@ -1,4 +1,4 @@
-VERSION = (1, 2, 1, "final")
+VERSION = (1, 2, 2, "final")
 __version__ = VERSION  # alias
 
 

{openwisp_users-1.2.1 → openwisp_users-1.2.2}/openwisp_users/accounts/urls.py

@@ -77,12 +77,11 @@ urlpatterns = [
 
 if app_settings.SOCIALACCOUNT_ENABLED:
     urlpatterns += [path("social/", include("allauth.socialaccount.urls"))]
-
-for provider in providers.registry.get_class_list():
-    try:
-        prov_mod = import_module(provider.get_package() + ".urls")
-    except ImportError:
-        continue
-    prov_urlpatterns = getattr(prov_mod, "urlpatterns", None)
-    if prov_urlpatterns:
-        urlpatterns += prov_urlpatterns
+    for provider in providers.registry.get_class_list():
+        try:
+            prov_mod = import_module(provider.get_package() + ".urls")
+        except ImportError:
+            continue
+        prov_urlpatterns = getattr(prov_mod, "urlpatterns", None)
+        if prov_urlpatterns:
+            urlpatterns += prov_urlpatterns

{openwisp_users-1.2.1 → openwisp_users-1.2.2}/openwisp_users/admin.py

@@ -652,14 +652,32 @@ if admin.site.is_registered(EmailAddress):
     admin.site.unregister(EmailAddress)
 
 if allauth_settings.SOCIALACCOUNT_ENABLED:
-    socialaccount_models = [
+    _unregister_socialaccount_models = [
         ("socialaccount", "SocialToken"),
         ("socialaccount", "SocialAccount"),
     ]
-    # Allow managing secrets if OAuth/SAML is enabled
-    if not app_settings.SOCIALACCOUNT_ADMIN_NEEDED:
-        socialaccount_models.append(("socialaccount", "SocialApp"))
-    for model in socialaccount_models:
+    # allauth OAuth/SAML not enabled
+    if not app_settings.SOCIALACCOUNT_ADMIN_NEEDED:  # pragma: no cover
+        _unregister_socialaccount_models.append(("socialaccount", "SocialApp"))
+    # allauth OAuth/SAML enabled
+    else:
+        from allauth.socialaccount.models import SocialAccount
+
+        class SocialAccountInline(admin.StackedInline):
+            model = SocialAccount
+            extra = 0
+            readonly_fields = ("provider", "uid", "extra_data")
+
+            def has_add_permission(self, request, obj=None):
+                return False
+
+            def has_delete_permission(self, request, obj=None):
+                return False
+
+        UserAdmin.inlines.append(SocialAccountInline)
+
+    # Un-register cluttering socialaccount models
+    for model in _unregister_socialaccount_models:
         model_class = apps.get_model(*model)
         if admin.site.is_registered(model_class):
             admin.site.unregister(model_class)

{openwisp_users-1.2.1 → openwisp_users-1.2.2}/openwisp_users/tests/test_admin.py

@@ -73,7 +73,7 @@ class TestUsersAdmin(
 
     @property
     def add_user_inline_params(self):
-        return {
+        params = {
             "emailaddress_set-TOTAL_FORMS": 0,
             "emailaddress_set-INITIAL_FORMS": 0,
             "emailaddress_set-MIN_NUM_FORMS": 0,
@@ -83,6 +83,19 @@ class TestUsersAdmin(
             f"{self.app_label}_organizationuser-MIN_NUM_FORMS": 0,
             f"{self.app_label}_organizationuser-MAX_NUM_FORMS": 0,
         }
+        self._add_socialaccount_inline_params(params)
+        return params
+
+    def _add_socialaccount_inline_params(self, params):
+        if app_settings.SOCIALACCOUNT_ADMIN_NEEDED:
+            params.update(
+                {
+                    "socialaccount_set-TOTAL_FORMS": 0,
+                    "socialaccount_set-INITIAL_FORMS": 0,
+                    "socialaccount_set-MIN_NUM_FORMS": 0,
+                    "socialaccount_set-MAX_NUM_FORMS": 0,
+                }
+            )
 
     def test_admin_add_user_auto_email(self):
         admin = self._create_admin()
@@ -1702,6 +1715,7 @@ class TestBasicUsersIntegration(
 
     app_label = "openwisp_users"
     is_integration_test = True
+    _add_socialaccount_inline_params = TestUsersAdmin._add_socialaccount_inline_params
 
     def _get_user_edit_form_inline_params(self, user, organization):
         params = {
@@ -1740,6 +1754,7 @@ class TestBasicUsersIntegration(
                     f"{self.app_label}_organizationuser-0-user": str(user.pk),
                 }
             )
+        self._add_socialaccount_inline_params(params)
         return params
 
     def test_change_user(self):

{openwisp_users-1.2.1 → openwisp_users-1.2.2/openwisp_users.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openwisp-users
-Version: 1.2.1
+Version: 1.2.2
 Summary: Provides basic multi-tenancy features for OpenWISP (using the Django web-framework)
 Home-page: http://openwisp.org
 Download-URL: https://github.com/openwisp/openwisp-users/releases