From d0d511c3a55cd01453d9473aa6ea2912bcb587ed Mon Sep 17 00:00:00 2001 From: Christopher Joe Date: Wed, 30 Sep 2015 09:55:29 +1300 Subject: [PATCH 1/2] Remove '[' and ']' from jQuery selector for checkbox groups --- javascript/UserForm.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/javascript/UserForm.js b/javascript/UserForm.js index 452e05d..bb76fdb 100644 --- a/javascript/UserForm.js +++ b/javascript/UserForm.js @@ -125,7 +125,7 @@ jQuery(function ($) { // When a field becomes valid. success: function (error) { var errorId = $(error).attr('id'), - fieldId = errorId.substr(0, errorId.indexOf('-error')), + fieldId = errorId.substr(0, errorId.indexOf('-error')).replace(/[\\[\\]]/, ''), isCheckboxGroup = $(error).closest('.requiredField').hasClass('checkboxset'); // We need to escapse the field id if it's a checkboxfield From 6a8a82a287eddfd3786f1a7c25da06ad3e6b4b93 Mon Sep 17 00:00:00 2001 From: Christopher Joe Date: Thu, 1 Oct 2015 14:42:29 +1300 Subject: [PATCH 2/2] removed isCheckboxGroup check and replace --- javascript/UserForm.js | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/javascript/UserForm.js b/javascript/UserForm.js index bb76fdb..e913837 100644 --- a/javascript/UserForm.js +++ b/javascript/UserForm.js @@ -125,15 +125,11 @@ jQuery(function ($) { // When a field becomes valid. success: function (error) { var errorId = $(error).attr('id'), - fieldId = errorId.substr(0, errorId.indexOf('-error')).replace(/[\\[\\]]/, ''), - isCheckboxGroup = $(error).closest('.requiredField').hasClass('checkboxset'); + fieldId = errorId.substr(0, errorId.indexOf('-error')).replace(/[\\[\\]]/, ''); - // We need to escapse the field id if it's a checkboxfield - // because jQuery breaks when using selector that end with + // Remove square brackets since jQuery.validate.js uses idOrName, + // which breaks further on when using a selector that end with // square brackets. - if (isCheckboxGroup) { - fieldId = fieldId.replace('[]', '\\\\[\\\\]'); - } error.remove();