router/router_list/user.html: Indicate missing coordinates

This also indicates missing coordinates without a previous reset.

The router detail page shows different messages for both cases
(missing coords and reset). The lists show the "Reset!"
warning in both cases (previously only for real reset).

Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
This commit is contained in:
Adrian Schmutzler 2018-11-23 20:06:10 +01:00
parent 24e934b66c
commit dc9b71ae01
4 changed files with 6 additions and 4 deletions

View File

@ -52,7 +52,7 @@ def router_list():
mysql = FreifunkMySQL() mysql = FreifunkMySQL()
routers = mysql.fetchall(""" routers = mysql.fetchall("""
SELECT router.id, hostname, status, hoods.id AS hoodid, hoods.name AS hood, contact, nickname, hardware, router.created, sys_uptime, last_contact, clients, reset, blocked, v2, local SELECT router.id, hostname, status, hoods.id AS hoodid, hoods.name AS hood, contact, nickname, hardware, router.created, sys_uptime, last_contact, clients, router.lat, router.lng, reset, blocked, v2, local
FROM router FROM router
INNER JOIN hoods ON router.hood = hoods.id INNER JOIN hoods ON router.hood = hoods.id
LEFT JOIN users ON router.contact = users.email LEFT JOIN users ON router.contact = users.email
@ -490,7 +490,7 @@ def user_info(nickname):
else: else:
flash("<b>You are not authorized to perform this action!</b>", "danger") flash("<b>You are not authorized to perform this action!</b>", "danger")
routers = mysql.fetchall(""" routers = mysql.fetchall("""
SELECT router.id, hostname, status, hoods.id AS hoodid, hoods.name AS hood, firmware, hardware, created, sys_uptime, clients, reset, blocked, v2, local SELECT router.id, hostname, status, hoods.id AS hoodid, hoods.name AS hood, firmware, hardware, created, sys_uptime, clients, router.lat, router.lng, reset, blocked, v2, local
FROM router FROM router
INNER JOIN hoods ON router.hood = hoods.id INNER JOIN hoods ON router.hood = hoods.id
LEFT JOIN ( LEFT JOIN (

View File

@ -153,6 +153,8 @@
{%- endif -%} {%- endif -%}
{%- if router.reset %} {%- if router.reset %}
<span style="color:#d90000">- Router has lost its position!</span> <span style="color:#d90000">- Router has lost its position!</span>
{%- elif not router.lat and not router.lng %}
<span style="color:#d90000">- Router has no position!</span>
{%- endif -%} {%- endif -%}
</td></tr> </td></tr>
{%- endif %} {%- endif %}

View File

@ -44,7 +44,7 @@
{%- for router in routers %} {%- for router in routers %}
<tr> <tr>
<td class="text-responsive"><a href="{{ url_for("router_info", dbid=router.id) }}">{{ router.hostname }}</a> <td class="text-responsive"><a href="{{ url_for("router_info", dbid=router.id) }}">{{ router.hostname }}</a>
{%- if router.reset %} - <span style="color:#d90000">Reset!</span>{%- endif %}{%- if router.blocked and not router.v2 %} - <span style="color:#d90000">Blocked!</span>{%- endif %} {%- if not router.lat and not router.lng %} - <span style="color:#d90000">Reset!</span>{%- endif %}{%- if router.blocked and not router.v2 %} - <span style="color:#d90000">Blocked!</span>{%- endif %}
</td> </td>
<td class="text-center" data-order="{{ router.status }}"><span class="{{ router.status|status2css }}">{{ router.status }}</span></td> <td class="text-center" data-order="{{ router.status }}"><span class="{{ router.status|status2css }}">{{ router.status }}</span></td>
<td{%- if router.local %} class="hoodlocal"{%- elif router.v2 %} class="hoodv2"{%- endif %}>{{ router.hood }}</td> <td{%- if router.local %} class="hoodlocal"{%- elif router.v2 %} class="hoodv2"{%- endif %}>{{ router.hood }}</td>

View File

@ -104,7 +104,7 @@
{%- for router in routers %} {%- for router in routers %}
<tr> <tr>
<td class="text-nowrap-responsive"><a href="{{ url_for("router_info", dbid=router.id) }}">{{ router.hostname }}</a> <td class="text-nowrap-responsive"><a href="{{ url_for("router_info", dbid=router.id) }}">{{ router.hostname }}</a>
{%- if router.reset %} - <span style="color:#d90000">Reset!</span>{%- endif %}{%- if router.blocked and not router.v2 %} - <span style="color:#d90000">Blocked!</span>{%- endif %} {%- if not router.lat and not router.lng %} - <span style="color:#d90000">Reset!</span>{%- endif %}{%- if router.blocked and not router.v2 %} - <span style="color:#d90000">Blocked!</span>{%- endif %}
</td> </td>
<td class="text-center" data-order="{{ router.status }}"><span class="{{ router.status|status2css }}">{{ router.status }}</span></td> <td class="text-center" data-order="{{ router.status }}"><span class="{{ router.status|status2css }}">{{ router.status }}</span></td>
<td{%- if router.local %} class="hoodlocal"{%- elif router.v2 %} class="hoodv2"{%- endif %}>{{ router.hood }}</td> <td{%- if router.local %} class="hoodlocal"{%- elif router.v2 %} class="hoodv2"{%- endif %}>{{ router.hood }}</td>