diff options
author | Struan Donald <struan@exo.org.uk> | 2013-06-18 16:34:45 +0100 |
---|---|---|
committer | Struan Donald <struan@exo.org.uk> | 2013-06-18 16:35:04 +0100 |
commit | 87e23ed72191aa9b3ce94cd511872df96842a089 (patch) | |
tree | 9b8af97ab950b9b477d456d05c412a66fbbd24fa /www/js/views/submit.js | |
parent | 74320342c0e5e99b71da71228030bfd05fc22336 (diff) |
check that errors keys look about correct
Diffstat (limited to 'www/js/views/submit.js')
-rw-r--r-- | www/js/views/submit.js | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/www/js/views/submit.js b/www/js/views/submit.js index 9060e46..fcc9f26 100644 --- a/www/js/views/submit.js +++ b/www/js/views/submit.js @@ -77,14 +77,16 @@ onReportInvalid: function(model, err, options) { var errors = err.errors; var errorList = '<ul><li class="plain">Invalid report</li>'; + var validErrors = [ 'password', 'category', 'name' ]; for ( var k in errors ) { - error = ''; - if ( k === 'password' ) { - error = FMS.strings.password_problem; - } else { - error = errors[k]; + if ( validErrors.inidexOf(k) >= 0 || errors[k].match(/required/) ) { + if ( k === 'password' ) { + error = FMS.strings.password_problem; + } else if ( k !== '') { + error = errors[k]; + } + errorList += '<li>' + error + '</li>'; } - errorList += '<li>' + error + '</li>'; } errorList += '</ul>'; $('#errors').html(errorList).show(); |