Defensive coding

This commit is contained in:
Ian Renton
2026-08-15 07:11:36 +01:00
parent ef2c23c8b6
commit 0dee8d827a
9 changed files with 17 additions and 17 deletions
+2 -2
View File
@@ -44,7 +44,7 @@ class CleanupTimer:
for i in list(DATA_STORE.spots.keys()): for i in list(DATA_STORE.spots.keys()):
try: try:
spot = DATA_STORE.spots.get(i) spot = DATA_STORE.spots.get(i)
if spot.expired(): if spot and spot.expired():
DATA_STORE.spots.delete(i) DATA_STORE.spots.delete(i)
except KeyError: except KeyError:
# Must have already been deleted, OK with that # Must have already been deleted, OK with that
@@ -52,7 +52,7 @@ class CleanupTimer:
for i in list(DATA_STORE.alerts.keys()): for i in list(DATA_STORE.alerts.keys()):
try: try:
alert = DATA_STORE.alerts.get(i) alert = DATA_STORE.alerts.get(i)
if alert.expired(): if alert and alert.expired():
DATA_STORE.alerts.delete(i) DATA_STORE.alerts.delete(i)
except KeyError: except KeyError:
# Must have already been deleted, OK with that # Must have already been deleted, OK with that
+1 -1
View File
@@ -76,7 +76,7 @@
</div> </div>
<script src="/static/js/add-spot.js?v=1786747642"></script> <script src="/static/js/add-spot.js?v=1786774296"></script>
<script>$(document).ready(function () { <script>$(document).ready(function () {
$("#nav-link-add-spot").addClass("active"); $("#nav-link-add-spot").addClass("active");
}); <!-- highlight active page in nav --></script> }); <!-- highlight active page in nav --></script>
+1 -1
View File
@@ -82,7 +82,7 @@
</div> </div>
<script src="/static/js/alerts.js?v=1786747642"></script> <script src="/static/js/alerts.js?v=1786774296"></script>
<script>$(document).ready(function () { <script>$(document).ready(function () {
$("#nav-link-alerts").addClass("active"); $("#nav-link-alerts").addClass("active");
}); <!-- highlight active page in nav --></script> }); <!-- highlight active page in nav --></script>
+2 -2
View File
@@ -79,8 +79,8 @@
</div> </div>
<script src="/static/js/spotsbandsandmap.js?v=1786747642"></script> <script src="/static/js/spotsbandsandmap.js?v=1786774296"></script>
<script src="/static/js/bands.js?v=1786747642"></script> <script src="/static/js/bands.js?v=1786774296"></script>
<script>$(document).ready(function () { <script>$(document).ready(function () {
$("#nav-link-bands").addClass("active"); $("#nav-link-bands").addClass("active");
}); <!-- highlight active page in nav --></script> }); <!-- highlight active page in nav --></script>
+5 -5
View File
@@ -1,6 +1,6 @@
{% extends "skeleton.html" %} {% extends "skeleton.html" %}
{% block head_extra %} {% block head_extra %}
<link rel="stylesheet" href="/static/css/style.css?v=1786747642" type="text/css"> <link rel="stylesheet" href="/static/css/style.css?v=1786774296" type="text/css">
<link href="/static/vendor/css/bootstrap-5.3.8.min.css" rel="stylesheet"> <link href="/static/vendor/css/bootstrap-5.3.8.min.css" rel="stylesheet">
<link href="/static/vendor/css/fontawesome-6.7.2.min.css" rel="stylesheet"> <link href="/static/vendor/css/fontawesome-6.7.2.min.css" rel="stylesheet">
<link href="/static/vendor/css/solid-6.7.2.min.css" rel="stylesheet"> <link href="/static/vendor/css/solid-6.7.2.min.css" rel="stylesheet">
@@ -15,10 +15,10 @@
window.fetchEventSource = fetchEventSource; window.fetchEventSource = fetchEventSource;
</script> </script>
<script src="/static/js/utils.js?v=1786747642"></script> <script src="/static/js/utils.js?v=1786774296"></script>
<script src="/static/js/ui-ham.js?v=1786747642"></script> <script src="/static/js/ui-ham.js?v=1786774296"></script>
<script src="/static/js/geo.js?v=1786747642"></script> <script src="/static/js/geo.js?v=1786774296"></script>
<script src="/static/js/common.js?v=1786747642"></script> <script src="/static/js/common.js?v=1786774296"></script>
{% end %} {% end %}
{% block body %} {% block body %}
<div class="container"> <div class="container">
+1 -1
View File
@@ -284,7 +284,7 @@
</div> </div>
<script src="/static/vendor/js/chart-4.4.9.umd.min.js"></script> <script src="/static/vendor/js/chart-4.4.9.umd.min.js"></script>
<script src="/static/js/conditions.js?v=1786747642"></script> <script src="/static/js/conditions.js?v=1786774296"></script>
<script>$(document).ready(function () { <script>$(document).ready(function () {
$("#nav-link-conditions").addClass("active"); $("#nav-link-conditions").addClass("active");
}); <!-- highlight active page in nav --></script> }); <!-- highlight active page in nav --></script>
+2 -2
View File
@@ -112,8 +112,8 @@
<script src="/static/vendor/js/leaflet-cqzones.js"></script> <script src="/static/vendor/js/leaflet-cqzones.js"></script>
<script src="/static/vendor/js/leaflet-workedallbritainireland.js" type="module"></script> <script src="/static/vendor/js/leaflet-workedallbritainireland.js" type="module"></script>
<script src="/static/js/spotsbandsandmap.js?v=1786747642"></script> <script src="/static/js/spotsbandsandmap.js?v=1786774296"></script>
<script src="/static/js/map.js?v=1786747642"></script> <script src="/static/js/map.js?v=1786774296"></script>
<script>$(document).ready(function () { <script>$(document).ready(function () {
$("#nav-link-map").addClass("active"); $("#nav-link-map").addClass("active");
}); <!-- highlight active page in nav --></script> }); <!-- highlight active page in nav --></script>
+2 -2
View File
@@ -118,8 +118,8 @@
</div> </div>
<script src="/static/js/spotsbandsandmap.js?v=1786747642"></script> <script src="/static/js/spotsbandsandmap.js?v=1786774296"></script>
<script src="/static/js/spots.js?v=1786747642"></script> <script src="/static/js/spots.js?v=1786774296"></script>
<script>$(document).ready(function () { <script>$(document).ready(function () {
$("#nav-link-spots").addClass("active"); $("#nav-link-spots").addClass("active");
}); <!-- highlight active page in nav --></script> }); <!-- highlight active page in nav --></script>
+1 -1
View File
@@ -86,7 +86,7 @@
</div> </div>
</div> </div>
<script src="/static/js/status.js?v=1786747642"></script> <script src="/static/js/status.js?v=1786774296"></script>
<script> <script>
$(document).ready(function () { $(document).ready(function () {
$("#nav-link-status").addClass("active"); $("#nav-link-status").addClass("active");