summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMagnus Hagander2018-09-16 21:47:52 +0000
committerMagnus Hagander2018-09-16 21:47:52 +0000
commitdf80486b73b1b8543f1a69cfd06a5397f1311590 (patch)
tree9a993d4ae7d96725aac0357b67e2d84372275847
parent37da576ffa4be185e669e04c1e968ec64013c198 (diff)
Don't fail on incorrect statuses in URLtest
The comment said we should ignore the status if int() on it fails, but to do that we have to fail it before we actually add the WHERE clause. Thus, reorder the code to do so.
-rw-r--r--pgcommitfest/commitfest/views.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/pgcommitfest/commitfest/views.py b/pgcommitfest/commitfest/views.py
index 3a04115..4083839 100644
--- a/pgcommitfest/commitfest/views.py
+++ b/pgcommitfest/commitfest/views.py
@@ -108,8 +108,8 @@ def commitfest(request, cfid):
whereparams = {}
if request.GET.has_key('status') and request.GET['status'] != "-1":
try:
- whereclauses.append("poc.status=%(status)s")
whereparams['status'] = int(request.GET['status'])
+ whereclauses.append("poc.status=%(status)s")
except ValueError:
# int() failed -- so just ignore this filter
pass