victorjonsson#614 Fixed bug in $.fn.validate

Author: victorjonsson

Gruntfile.js

@@ -95,9 +95,6 @@ function initializeGruntConfig(grunt) {
       }
     },
 
-    // watch for changes to source
-    // Better than calling grunt a million times
-    // (call 'grunt watch')
     watch: {
       files: [SRC_DIR + '/**'],
       tasks: ['test'],
@@ -202,7 +199,8 @@ module.exports = function (grunt) {
   grunt.loadNpmTasks('grunt-umd');
 
   grunt.registerTask("build-production", ["version", "test", "uglify"]);
-  grunt.registerTask('test', ['concat', 'copy', 'umd', 'cssmin','jshint', 'qunit']);
+  grunt.registerTask('build', ['concat', 'copy', 'umd', 'cssmin']);
+  grunt.registerTask('test', ['build','jshint', 'qunit']);
   grunt.registerTask("default", ["test", "watch"]);
   grunt.registerTask("prepublish", ["test", "uglify"]);
 };

src/main/jquery-plugins.js

@@ -137,8 +137,8 @@
     var language = $.extend({}, $.formUtils.LANG, lang || {});
     this.each(function() {
       var $elem = $(this),
-        $closestForm = $elem.closest('form').get(0) || {},
-        formDefaultConfig = $closestForm.validationConfig || {};
+        closestFormElem = $elem.closest('form').get(0) || {},
+        formDefaultConfig = closestFormElem.validationConfig || $.formUtils.defaultConfig();
 
       $elem.one('validation', function(evt, isValid) {
         if ( typeof cb === 'function' ) {

src/main/utils.js

@@ -128,6 +128,10 @@
       conf = conf || $.formUtils.defaultConfig();
       language = language || $.formUtils.LANG;
 
+      if (!$form.length) {
+        $form = $elem.parent();
+      }
+
       var value = this.getValue($elem);
 
       $elem
@@ -142,7 +146,6 @@
         })
         .trigger('beforeValidation', [value, language, conf]);
 
-
       var inputIsOptional = $elem.valAttr('optional') === 'true',
           skipBecauseItsEmpty = !value && inputIsOptional,
           validationRules = $elem.attr(conf.validationRuleAttribute),

test/qunit.html

@@ -167,20 +167,33 @@
 
         });
 
-        /*
-         * Test validation of hidden inputs
-         */
-         test('Validation of hidden inputs', function(assert) {
-          assert.expect(3);
-          var $input = input('--', {'':'required alphanumeric'});
-          equal(false, $input.closest('form').isValid({}, {}, false));
-          $input.hide();
-          equal(true, $input.closest('form').isValid({}, {}, false));
-          $input.validate(function(valid) {
-            assert.equal(false, valid);
-          }, {validateHiddenInputs: true});
-
-         });
+      /*
+      * Test $.fn.validate
+      */
+      test('Test $.fn.validate() ', function(assert) {
+        clearForm();
+        assert.expect(1);
+        var $input = input('--', {'':'required alphanumeric'});
+        $input.attr('id', 'some-input');
+        $('#some-input').validate(function(valid, elem) {
+          assert.equal(false, valid);
+        });
+      });
+
+      /*
+       * Test validation of hidden inputs
+       */
+      test('Validation of hidden inputs', function(assert) {
+        clearForm();
+        assert.expect(3);
+        var $input = input('--', {'':'required alphanumeric'});
+        assert.equal(false, $input.closest('form').isValid({}, {}, false));
+        $input.hide();
+        assert.equal(true, $input.closest('form').isValid({}, {}, false));
+        $input.validate(function(valid) {
+          assert.equal(false, valid);
+        }, {validateHiddenInputs: true});
+      });
 
         /*
          * Test error message