From a72ab61902961562048f487c2e102249b4a33964 Mon Sep 17 00:00:00 2001
From: Kevin Morris <kevr@0cost.org>
Date: Wed, 25 Aug 2021 16:34:37 -0700
Subject: [PATCH] [FastAPI] fix dashboard template

Some columns should only be shown when a user is authenticated.

Signed-off-by: Kevin Morris <kevr@0cost.org>
---
 templates/partials/packages/results.html | 52 ++++++++++++------------
 1 file changed, 27 insertions(+), 25 deletions(-)

diff --git a/templates/partials/packages/results.html b/templates/partials/packages/results.html
index 005bd5a9..80029d10 100644
--- a/templates/partials/packages/results.html
+++ b/templates/partials/packages/results.html
@@ -6,8 +6,10 @@
             <th>{{ "Version" | tr }}</th>
             <th>{{ "Votes" | tr }}</th>
             <th>{{ "Popularity" | tr }}</th>
-            <th>{{ "Voted" | tr }}</th>
-            <th>{{ "Notify" | tr }}</th>
+            {% if request.user.is_authenticated() %}
+                <th>{{ "Voted" | tr }}</th>
+                <th>{{ "Notify" | tr }}</th>
+            {% endif %}
             <th>{{ "Description" | tr }}</th>
             <th>{{ "Maintainer" | tr }}</th>
         </tr>
@@ -21,33 +23,33 @@
                         {{ pkg.Name }}
                     </a>
                 </td>
-                {% if flagged %}
-                    <td class="flagged">{{ pkg.Version }}</td>
-                {% else %}
-                    <td>{{ pkg.Version }}</td>
-                {% endif %}
+                <td{% if flagged %} class="flagged"{% endif %}>{{ pkg.Version }}</td>
                 <td>{{ pkg.PackageBase.NumVotes }}</td>
-                <td>
-                    {{ pkg.PackageBase.Popularity | number_format(2) }}
-                </td>
-                <td>
-                    <!-- If I voted, display "Yes". -->
-                    {% if request.user.voted_for(pkg) %}
-                        {{ "Yes" | tr }}
-                    {% endif %}
-                </td>
-                <td>
-                    <!-- If I'm being notified, display "Yes". -->
-                    {% if request.user.notified(pkg) %}
-                        {{ "Yes" | tr }}
-                    {% endif %}
-                </td>
+                <td>{{ pkg.PackageBase.Popularity | number_format(2) }}</td>
+                {% if request.user.is_authenticated() %}
+                    <td>
+                        <!-- If I voted, display "Yes". -->
+                        {% if request.user.voted_for(pkg) %}
+                            {{ "Yes" | tr }}
+                        {% endif %}
+                    </td>
+                    <td>
+                        <!-- If I'm being notified, display "Yes". -->
+                        {% if request.user.notified(pkg) %}
+                            {{ "Yes" | tr }}
+                        {% endif %}
+                    </td>
+                {% endif %}
                 <td class="wrap">{{ pkg.Description or '' }}</td>
                 <td>
                     {% set maintainer = pkg.PackageBase.Maintainer %}
-                    <a href="/account/{{ maintainer.Username }}">
-                        {{ maintainer.Username }}
-                    </a>
+                    {% if maintainer %}
+                        <a href="/account/{{ maintainer.Username }}">
+                            {{ maintainer.Username }}
+                        </a>
+                    {% else %}
+                        <span class="error">{{ "orphan" | tr }}</span>
+                    {% endif %}
                 </td>
             </tr>
         {% endfor %}