Based on #157 and #158, revert back to existence checks and length rather than default empty array initialization for ease of reading

This commit is contained in:
Benjamin Southall 2019-08-10 12:01:07 +10:00
parent 878b9aa420
commit a5650df053
8 changed files with 10 additions and 10 deletions

View File

@ -1,4 +1,4 @@
{% if messages|default([])|length == 0 %}
{% if not messages %}
<p style="text-align:center" class="unimportant">({% trans 'No private messages for you.' %})</p>
{% else %}
<table class="modlog">

View File

@ -1,4 +1,4 @@
{% if themes|default([])|length == 0 %}
{% if not themes %}
<p style="text-align:center" class="unimportant">({% trans 'There are no themes available.' %})</p>
{% else %}
<table class="modlog">

View File

@ -87,7 +87,7 @@
</ul>
</form>
{% if logs|default([])|length > 0 %}
{% if logs and logslength > 0 %}
<table class="modlog" style="width:600px">
<tr>
<th>{% trans 'IP address' %}</th>

View File

@ -18,7 +18,7 @@
<legend>{{ notes|count }} {% trans notes_on_record %}</legend>
</legend>
{% if notes|default([])|length > 0 %}
{% if notes and noteslength > 0 %}
<table class="modlog">
<tr>
<th>{% trans 'Staff' %}</th>
@ -81,7 +81,7 @@
</fieldset>
{% endif %}
{% if bans|default([])|length > 0 and mod|hasPermission(config.mod.view_ban) %}
{% if bans and bans|length > 0 and mod|hasPermission(config.mod.view_ban) %}
<fieldset id="bans">
{% set bans_on_record = 'ban' ~ (bans|count != 1 ? 's' : '') ~ ' on record' %}
<legend>{{ bans|count }} {% trans bans_on_record %}</legend>
@ -174,7 +174,7 @@
</fieldset>
{% endif %}
{% if logs|default([])|length > 0 %}
{% if logs and logs|length > 0 %}
<fieldset id="history">
<legend>History</legend>
<table class="modlog" style="width:100%">

View File

@ -24,7 +24,7 @@
<img src="static/lain_is_cute_datass_small_teal.png" alt="mascot"/>
</center>
<div class="ban">
{% if news|default([])|length == 0 %}
{% if not news %}
<p style="text-align:center" class="unimportant">(No news to show.)</p>
{% else %}
{% for entry in news %}

View File

@ -14,7 +14,7 @@
</header>
<div class="ban">
{% if news|default([])|length == 0 %}
{% if not news %}
<p style="text-align:center" class="unimportant">{% trans %}(No news to show.){% endtrans %}</p>
{% else %}
{% for entry in news %}

View File

@ -13,7 +13,7 @@
</header>
<div class="ban">
{% if news|default([])|length == 0 %}
{% if not news %}
<p style="text-align:center" class="unimportant">(No news to show.)</p>
{% else %}
{% for entry in news %}

View File

@ -36,7 +36,7 @@
<br>
<div class="ban" id="global3">
<h2>Latest News</h2>
{% if recent_news|default([])|length == 0 %}
{% if not recent_news %}
<p style="text-align:center" class="unimportant">(No news to show.)</p>
{% else %}
{% for entry in recent_news %}