do not fall over if the user visits /alert/subscribe with no parameters or no email

1 files changed, 6 insertions, 1 deletions

diff --git a/perllib/FixMyStreet/App/Controller/Alert.pm b/perllib/FixMyStreet/App/Controller/Alert.pm
index 45696e4e7..b8b7cce40 100644
--- a/perllib/FixMyStreet/App/Controller/Alert.pm
+++ b/perllib/FixMyStreet/App/Controller/Alert.pm
@@ -58,9 +58,14 @@ sub subscribe : Path('subscribe') : Args(0) {
     if ( $c->req->param('rss') ) {
         $c->detach('rss');
     }
-    elsif ( $c->req->param('rznvy') ) {
+    # if it exists then it's been submitted so we should
+    # go to subscribe email and let it work out the next step
+    elsif ( exists $c->req->params->{'rznvy'} ) {
         $c->detach('subscribe_email');
     }
+
+    # shouldn't get to here but if we have then do something sensible
+    $c->go('index');
 }
 
 =head2 rss