[Fixes GeoNode#12167] providing multiple openid_connect django.allaut…

…h socialproviders fails when loading login page
zalf-rdm · Apr 24, 2024 · 44bd725 · 44bd725
1 parent f424449
commit 44bd725
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/geonode/people/templatetags/socialaccount_extra.py b/geonode/people/templatetags/socialaccount_extra.py
@@ -15,10 +15,10 @@ def get_user_social_providers(user):
 @register.simple_tag
 def get_other_social_providers(user):
     user_providers = get_user_social_providers(user)
-    user_provider_names = [p.name.lower() for p in user_providers]
+    user_provider_names = [p.id.lower() for p in user_providers]
     other_providers = []
-    for provider in providers.registry.get_list():
-        if provider.name.lower() not in user_provider_names:
+    for provider in providers.registry.get_class_list():
+        if provider.id.lower() not in user_provider_names:
             other_providers.append(provider)
     return other_providers
 

diff --git a/geonode/templates/socialaccount/snippets/provider_list.html b/geonode/templates/socialaccount/snippets/provider_list.html
@@ -5,7 +5,7 @@
 {% get_providers as socialaccount_providers %}
 
 {% for provider in socialaccount_providers %}
-        {% if provider.id == "openid" %}
+        {% if provider.id == "openid_connect" %}
             {% for brand in provider.get_brands %}
                 <a class="list-group-item" title="{{brand.name}}" href="{% provider_login_url provider.id openid=brand.openid_url process=process %}">
                     {{brand.name}}

diff --git a/geonode/templates/socialaccount/snippets/remaining_providers_list.html b/geonode/templates/socialaccount/snippets/remaining_providers_list.html
@@ -5,7 +5,7 @@
 {% get_other_social_providers user as other_providers %}
 
 {% for provider in other_providers %}
-    {% if provider.id == "openid" %}
+    {% if provider.id == "openid_connect" %}
         {% for brand in provider.get_brands %}
             <a class="list-group-item" title="{{brand.name}}" href="{% provider_login_url provider.id openid=brand.openid_url process=process %}">
                 {{brand.name}}