victorjonsson#599 quick-fix for async validators when used together with toggleDisabled

Author: victorjonsson

src/modules/security.js

@@ -348,17 +348,21 @@
 
   var requestServer = function (serverURL, $element, val, conf, callback) {
       var reqParams = $element.valAttr('req-params') || $element.data('validation-req-params') || {},
+        inputName = $element.valAttr('param-name') || $element.attr('name'),
         handleResponse = function (response, callback) {
           callback(response);
         };
 
+      if (!inputName) {
+        throw new Error('Missing input name used for http requests made by server validator');
+      }
       if (!reqParams) {
         reqParams = {};
       }
       if (typeof reqParams === 'string') {
         reqParams = $.parseJSON(reqParams);
       }
-      reqParams[$element.valAttr('param-name') || $element.attr('name')] = val;
+      reqParams[inputName] = val;
 
       $.ajax({
         url: serverURL,
@@ -583,7 +587,7 @@
             callback: function (result) {
               $form.find('[data-validation~="recaptcha"]')
                 .trigger('validation', (result && result !== ''));
-              
+
             },
             'expired-callback': function() {
               $form.find('[data-validation~="recaptcha"]').trigger('validation', false);

src/modules/toggleDisabled.js

@@ -82,6 +82,12 @@
             }
           });
 
+      // Notice! Async validator can't be validated onkeyup
+      // We can not determine if an input has an async validator by looking at the markup.
+      // The solution here would be to add the flag `async` to addValidator configuration
+      // for now we will have to hard code known async validators
+      $formsToDisable.find('[data-validation~="server"],[data-validation~="dimension"]')
+        .valAttr('event', 'change');
 
       // Make all inputs validated on keyup, require validateOnEvent in validation config
       toggleFormState($formsToDisable, 'disabled');