Merge pull request #792 from rdmorganiser/791-bug-in-template-for-soc…

…ialaccount-snippet-provider-list-when-using-multiple-openid_connect-providers fix #791 update provider_list for openid_connect providers and refactor keycloak snippet
rdmorganiser · Nov 16, 2023 · a82cd4d · a82cd4d
2 parents d28d15d + 4dae597
commit a82cd4d
Showing 1 changed file with 17 additions and 7 deletions.
diff --git a/rdmo/accounts/templates/socialaccount/snippets/provider_list.html b/rdmo/accounts/templates/socialaccount/snippets/provider_list.html
@@ -24,13 +24,23 @@
         <img src="{% static 'accounts/img/orcid-signin.png' %}" alt="ORCID sign in" />
     </a>
 </li>
-{% elif provider.id == 'keycloak' %}
-<li class="socialaccount_provider_break">
-    <a title="{{ provider.name }}" class="socialaccount_provider {{ provider.id }}"
-        href="{% provider_login_url provider.id process=process scope=scope auth_params=auth_params %}">
-        <img src="{% static 'accounts/img/keycloak_logo_200px.svg' %}" alt="Keycloak sign in" />
-    </a>
-</li>
+{% elif provider.id == 'openid_connect' %}
+    {% if provider.app.provider_id == 'keycloak' %}
+    <li class="socialaccount_provider_break">
+        <a title="{{ provider.name }}" class="socialaccount_provider {{ provider.id }}"
+            href="{% provider_login_url provider process=process scope=scope auth_params=auth_params %}">
+            <img src="{% static 'accounts/img/keycloak_logo_200px.svg' %}" alt="Keycloak sign in" />
+        </a>
+    </li>
+    {% else %}
+    <li>
+        <a title="{{provider.name}}" class="socialaccount_provider {{provider.id}}"
+            href="{% provider_login_url provider process=process scope=scope auth_params=auth_params %}">
+            {{ provider.name }}
+        </a>
+    </li>
+{% endif %}
+
 {% else %}
 <li>
     <a title="{{provider.name}}" class="socialaccount_provider {{provider.id}}"