tonyair/silverstripe-userforms
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-userforms.git synced 2024-10-22 17:05:42 +02:00
Code Issues Projects Releases Wiki Activity

focus on first invalid element after form validation

Browse Source 
Checkbox/radio groups now get caught by validation
This commit is contained in:
scott1702 2015-08-26 17:19:29 +12:00
parent be90ffac35
commit 7f31dbe4ff
1 changed files with 28 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
javascript/UserForm.js
View File

@ -43,7 +43,7 @@ jQuery(function ($) {
// Add an error container which displays a list of invalid steps on form submission.
this.errorContainer = new ErrorContainer(this.$el.children('.error-container'));
// Listen for events triggered my form steps.
// Listen for events triggered by form steps.
this.$el.on('userform.action.prev', function (e) {
self.prevStep();
});
@ -63,6 +63,13 @@ jQuery(function ($) {
this.$el.validate(this.validationOptions);
// Ensure checkbox groups are validated correctly
$('.optionset.requiredField input').each(function() {
$(this).rules('add', {
required: true
});
});
return this;
}
@ -78,10 +85,18 @@ jQuery(function ($) {
if (element.is(':radio') || element.parents('.checkboxset').length > 0) {
error.insertAfter(element.closest('ul'));
} else if (element.parents('.checkbox').length > 0) {
error.insertAfter(element.next('label'));
} else {
error.insertAfter(element);
}
},
invalidHandler: function (event, validator) {
//setTimeout 0 so it runs after errorPlacement
setTimeout(function () {
validator.currentElements.filter('.error').first().focus();
}, 0);
},
// Callback for handling the actual submit when the form is valid.
// Submission in the jQuery.validate sence is handled at step level.
// So when the final step is submitted we have to also check all previous steps are valid.
@ -109,12 +124,21 @@ jQuery(function ($) {
},
// When a field becomes valid.
success: function (error) {
var errorId = $(error).attr('id');
var errorId = $(error).attr('id'),
fieldId = errorId.substr(0, errorId.indexOf('-error')),
isCheckboxGroup = $(error).closest('.requiredField').hasClass('checkboxset');
// We need to escapse the field id if it's a checkboxfield
// because jQuery breaks when using selector that end with
// square brackets.
if (isCheckboxGroup) {
fieldId = fieldId.replace('[]', '\\\\[\\\\]');
}
error.remove();
// Pass the field's ID with the event.
userform.$el.trigger('userform.form.valid', [errorId.substr(0, errorId.indexOf('-error'))]);
userform.$el.trigger('userform.form.valid', [fieldId]);
}
};