Remove contest editorial column if empty

Revise editorial column fix Fix format for checks Add editorials heading
DMOJ · Jun 1, 2020 · 21a288f · 21a288f
1 parent bfa6aaa
commit 21a288f
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 6 deletions.
diff --git a/judge/views/contests.py b/judge/views/contests.py
@@ -217,6 +217,9 @@ def get_context_data(self, **kwargs):
             .annotate(has_public_editorial=Sum(Case(When(solution__is_public=True, then=1),
                                                     default=0, output_field=IntegerField()))) \
             .add_i18n_name(self.request.LANGUAGE_CODE)
+        context['contest_has_public_editorials'] = any(
+            problem.is_public and problem.has_public_editorial for problem in context['contest_problems']
+        )
         return context
 
 

diff --git a/templates/contest/contest.html b/templates/contest/contest.html
@@ -84,7 +84,9 @@ <h2 style="margin-bottom: 0.2em"><i class="fa fa-fw fa-question-circle"></i>{{ _
                     <th>{{ _('Points') }}</th>
                     <th>{{ _('AC Rate') }}</th>
                     <th>{{ _('Users') }}</th>
-                    <th></th>
+                    {% if contest_has_public_editorials %}
+                        <th>{{ _('Editorials') }}</th>
+                    {% endif %}
                 </tr>
                 </thead>
                 <tbody>
@@ -106,11 +108,13 @@ <h2 style="margin-bottom: 0.2em"><i class="fa fa-fw fa-question-circle"></i>{{ _
                                 {{ problem.user_count }}
                             {% endif %}
                         </td>
-                        <td>
-                            {% if problem.is_public and problem.has_public_editorial %}
-                                <a href="{{ url('problem_editorial', problem.code) }}">{{ _('Editorial') }}</a>
-                            {% endif %}
-                        </td>
+                        {% if contest_has_public_editorials %}
+                            <td>
+                                {% if problem.is_public and problem.has_public_editorial %}
+                                    <a href="{{ url('problem_editorial', problem.code) }}">{{ _('Editorial') }}</a>
+                                {% endif %}
+                            </td>
+                        {% endif %}
                     </tr>
                 {% endfor %}
                 </tbody>