diff options
author | Struan Donald <struan@exo.org.uk> | 2012-03-15 13:50:34 +0000 |
---|---|---|
committer | Struan Donald <struan@exo.org.uk> | 2012-03-15 13:50:34 +0000 |
commit | 4174f60e00c1615b7512f55f53e180426856a749 (patch) | |
tree | 9b46e9514ff72b0a400e0a55c7c1cc89678c00c4 /www/js | |
parent | b6a6c94631b0c714ee5a9e10e72f9b125b53587d (diff) |
make validation work properly
Diffstat (limited to 'www/js')
-rw-r--r-- | www/js/fixmystreet.js | 9 | ||||
-rw-r--r-- | www/js/mobile.js | 4 |
2 files changed, 10 insertions, 3 deletions
diff --git a/www/js/fixmystreet.js b/www/js/fixmystreet.js index de04ae2..30b35a5 100644 --- a/www/js/fixmystreet.js +++ b/www/js/fixmystreet.js @@ -186,8 +186,6 @@ $(function(){ if (form.submit_problem) { $('input[type=submit]', form).prop("disabled", true); } - - // form.submit(); }, // make sure we can see the error message when we focus on invalid elements showErrors: function( errorMap, errorList ) { @@ -197,7 +195,12 @@ $(function(){ this.defaultShowErrors(); submitted = false; }, - invalidHandler: function(form, validator) { submitted = true; } + invalidHandler: function(form, validator) { + form.stopImmediatePropagation(); + form.stopPropagation(); + form.preventDefault(); + submitted = true; + } }); $('input[type=submit]').click( function(e) { form_submitted = 1; } ); diff --git a/www/js/mobile.js b/www/js/mobile.js index 73bb10e..55fca3d 100644 --- a/www/js/mobile.js +++ b/www/js/mobile.js @@ -197,6 +197,10 @@ function fileUploadFail() { function postReport(e) { e.preventDefault(); + + // the .stopImmediatePropogation call in invalidHandler should render this + // redundant but it doesn't seem to work so belt and braces :( + if ( !$('#map_form').valid() ) { return; } var params = { service: 'iphone', |