From a89e6622eda51af6065f48aa332f88da30da546e Mon Sep 17 00:00:00 2001
From: Yohan Boniface <yohanboniface@free.fr>
Date: Mon, 21 Aug 2023 17:16:34 +0200
Subject: [PATCH] User profile page: only display providers if there are some

Someone uses uMap with classic auth username/password, and they
don't want those empty HTML elements
---
 umap/templates/auth/user_form.html | 48 ++++++++++++++++--------------
 1 file changed, 25 insertions(+), 23 deletions(-)

diff --git a/umap/templates/auth/user_form.html b/umap/templates/auth/user_form.html
index c5401399..06d04079 100644
--- a/umap/templates/auth/user_form.html
+++ b/umap/templates/auth/user_form.html
@@ -19,30 +19,32 @@
         <input type="submit" value="{% trans 'Save' %}" />
       </form>
     </div>
-    <div class="row">
-      <h3>{% trans "Your current providers" %}</h3>
-      <ul>
-        {% for name in providers %}<li>{{ name|title }}</li>{% endfor %}
-      </ul>
-    </div>
-    <div class="row">
-      <h3>{% trans "Connect to another provider" %}</h3>
-      <p>
-        {% blocktrans %}It's a good habit to connect your account to more than one provider, in case one provider becomes unavailable, temporarily or even permanently.{% endblocktrans %}
-      </p>
-      <div>
-        <ul class="login-grid block-grid">
-          {% for name in backends.backends %}
-            {% if name not in providers %}
-              <li>
-                <a href="{% url "social:begin" name %}"
-                   class="umap-login-popup login-{{ name }}"
-                   title="{{ name|title }}"></a>
-              </li>
-            {% endif %}
-          {% endfor %}
+    {% if backends.backends|length %}
+      <div class="row">
+        <h3>{% trans "Your current providers" %}</h3>
+        <ul>
+          {% for name in providers %}<li>{{ name|title }}</li>{% endfor %}
         </ul>
       </div>
-    </div>
+      <div class="row">
+        <h3>{% trans "Connect to another provider" %}</h3>
+        <p>
+          {% blocktrans %}It's a good habit to connect your account to more than one provider, in case one provider becomes unavailable, temporarily or even permanently.{% endblocktrans %}
+        </p>
+        <div>
+          <ul class="login-grid block-grid">
+            {% for name in backends.backends %}
+              {% if name not in providers %}
+                <li>
+                  <a href="{% url "social:begin" name %}"
+                     class="umap-login-popup login-{{ name }}"
+                     title="{{ name|title }}"></a>
+                </li>
+              {% endif %}
+            {% endfor %}
+          </ul>
+        </div>
+      </div>
+    {% endif %}
   </div>
 {% endblock maincontent %}