Fix for datepicker and issue victorjonsson#80

victorjonsson · victorjonsson · commit 9dd1e7ec5b60 · 2014-03-09T08:47:20.000+01:00
diff --git a/README.md b/README.md
@@ -294,7 +294,11 @@ it calls jQ func **$.formUtils.validateInput** to validate the single input when
 
 ## Changelog
 
-#### 3.1.36
+#### 2.1.40
+* Incorrect error-styling when using datepicker or suggestions is now fixed
+* Incorrect error-styling of select elements is now fixed
+
+#### 2.1.36
 * Now possible to use the native reset() function to clear error messages and error styling of the input elements
 
 #### 2.1.34
diff --git a/form-validator/date.dev.js b/form-validator/date.dev.js
@@ -10,7 +10,7 @@
  *
  * @website http://formvalidator.net/#location-validators
  * @license Dual licensed under the MIT or GPL Version 2 licenses
- * @version 2.1.41
+ * @version 2.1.42
  */
 (function($) {
 
diff --git a/form-validator/file.dev.js b/form-validator/file.dev.js
@@ -10,7 +10,7 @@
  *
  * @website http://formvalidator.net/
  * @license Dual licensed under the MIT or GPL Version 2 licenses
- * @version 2.1.41
+ * @version 2.1.42
  */
 (function($, window) {
 
diff --git a/form-validator/form-test.html b/form-validator/form-test.html
@@ -3,7 +3,8 @@
 <head>
     <meta charset="utf-8" />
     <title>Form Test</title>
-    <link rel="stylesheet" href="http://netdna.bootstrapcdn.com/bootstrap/3.0.0-wip/css/bootstrap.min.css" />
+    <link rel="stylesheet" href="//netdna.bootstrapcdn.com/bootstrap/3.0.0-wip/css/bootstrap.min.css" />
+    <link rel="stylesheet" href="//code.jquery.com/ui/1.10.4/themes/black-tie/jquery-ui.css" />
     <style>
 
         /* Form and inputs */
@@ -68,6 +69,22 @@
             <input name="Swedish county suggestion" data-validation="swecounty" type="text" id="swedish-county-suggestions" class="form-control" />
         </div>
 
+        <div class="form-group">
+            <label class="control-label">Year</label>
+            <input name="birth" class="form-control"
+                   data-validation="date"
+                   data-validation-format="yyyy-mm-dd"
+                   data-suggestions="2014-01-15,2014-01-16,2014-01-17" />
+        </div>
+
+        <div class="form-group">
+            <label class="control-label">Datepicker</label>
+            <input name="birth2" class="form-control"
+                   data-validation="date"
+                   data-validation-format="mm/dd/yyyy"
+                   id="datepicker" />
+        </div>
+
         <div class="form-group password-strength">
             <label class="control-label" for="password">Display password strength (only strong)</label>
             <input name="password" type="password" id="password" class="form-control" data-validation="strength" data-validation-strength="3" />
@@ -109,7 +126,7 @@
         <div class="form-group">
             <label class="control-label">Text</label>
             (<span id="max-len">20</span> chars left)<br />
-            <textarea id="text-area" name="some-text"></textarea>
+            <textarea id="text-area" class="form-control" name="some-text"></textarea>
         </div>
 
         <div class="form-group">
@@ -189,19 +206,31 @@
             <label class="control-label">Country</label>
             <input name="test" data-validation="country" data-validation-error-msg="No valid country given" />
         </div>
+        <div class="form-group">
+            <label class="control-label">Alphanumeric (will only be validated if the checkbox is checked)</label>
+            <input name="test2"
+                    data-validation="alphanumeric"
+                    data-validation-error-msg="Invalid..."
+                    data-validation-if-checked="checker" />
+            <br />
+            <input type="checkbox" name="checker" />
+        </div>
         <p>
             <input type="submit" class="button">
             <input type="reset" class="button">
         </p>
     </form>
 </div>
 <script src="//ajax.googleapis.com/ajax/libs/jquery/1/jquery.min.js"></script>
+<script src="//code.jquery.com/ui/1.10.4/jquery-ui.min.js"></script>
 <script src="jquery.form-validator.js"></script>
 <script>
 (function($, window) {
 
     var dev = window.location.hash.indexOf('dev') > -1 ? '.dev' : '';
 
+    // setup datepicker
+    $("#datepicker").datepicker();
 
     // Add a new validator
     $.formUtils.addValidator({
@@ -273,7 +302,15 @@
             console.log('About to validate input "'+this.name+'"');
         })
         .on('validation', function(evt, isValid) {
-            console.log('Input '+this.name+' is '+( isValid ? 'VALID':'NOT VALID'));
+            var validationResult = '';
+            if( isValid === null ) {
+                validationResult = 'not validated';
+            } else if( isValid ) {
+                validationResult = 'VALID';
+            } else {
+                validationResult = 'INVALID';
+            }
+            console.log('Input '+this.name+' is '+validationResult);
         });
 
 })(jQuery, window);
diff --git a/form-validator/jquery.form-validator.js b/form-validator/jquery.form-validator.js
@@ -5,7 +5,7 @@
 *
 * @website http://formvalidator.net/
 * @license Dual licensed under the MIT or GPL Version 2 licenses
-* @version 2.1.41
+* @version 2.1.42
 */
 (function($) {
 
@@ -165,6 +165,23 @@
         if(!eventContext)
             eventContext = 'blur';
 
+        if( (this.valAttr('suggestion-nr') || this.hasClass('hasDatepicker') ) && !window.postponedValidation ) {
+            // This validation has to be postponed 
+            var _self = this;
+            window.postponedValidation = function() {
+                console.log('running the waiting validation');
+                _self.validateInputOnBlur(language, conf, attachKeyupEvent);
+                window.postponedValidation = false;
+            };
+            setTimeout(function() {
+                if( window.postponedValidation ) {
+                    window.postponedValidation();
+                }
+            }, 200);
+
+            return this;
+        }
+
         language = $.extend({}, $.formUtils.LANG, language || {});
         _removeErrorStyle(this, conf);
 
@@ -179,7 +196,7 @@
                             eventContext
                         );
 
-        $elem.trigger('validation', [validation===true]);
+        $elem.trigger('validation', [validation===null ? null : validation===true]);
 
         if(validation === true) {
             $elem
diff --git a/form-validator/jquery.form-validator.min.js b/form-validator/jquery.form-validator.min.js
diff --git a/form-validator/location.dev.js b/form-validator/location.dev.js
@@ -10,7 +10,7 @@
  *
  * @website http://formvalidator.net/#location-validators
  * @license Dual licensed under the MIT or GPL Version 2 licenses
- * @version 2.1.41
+ * @version 2.1.42
  */
 (function($) {
 
diff --git a/form-validator/security.dev.js b/form-validator/security.dev.js
@@ -12,7 +12,7 @@
  *
  * @website http://formvalidator.net/#security-validators
  * @license Dual licensed under the MIT or GPL Version 2 licenses
- * @version 2.1.41
+ * @version 2.1.42
  */
 (function($) {
 
diff --git a/form-validator/sweden.dev.js b/form-validator/sweden.dev.js
@@ -13,7 +13,7 @@
  *
  * @website http://formvalidator.net/#swedish-validators
  * @license Dual licensed under the MIT or GPL Version 2 licenses
- * @version 2.1.41
+ * @version 2.1.42
  */
 (function($, window) {
 
diff --git a/form-validator/uk.dev.js b/form-validator/uk.dev.js
@@ -9,7 +9,7 @@
  *
  * @website http://formvalidator.net/#uk-validators
  * @license Dual licensed under the MIT or GPL Version 2 licenses
- * @version 2.1.41
+ * @version 2.1.42
  */
 $.formUtils.addValidator({
     name : 'ukvatnumber',

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@`
`10`	`10`	`*`
`11`	`11`	`* @website http://formvalidator.net/#location-validators`
`12`	`12`	`* @license Dual licensed under the MIT or GPL Version 2 licenses`
`13`		`- * @version 2.1.41`
	`13`	`+ * @version 2.1.42`
`14`	`14`	`*/`
`15`	`15`	`(function($) {`
`16`	`16`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@`
`12`	`12`	`*`
`13`	`13`	`* @website http://formvalidator.net/#security-validators`
`14`	`14`	`* @license Dual licensed under the MIT or GPL Version 2 licenses`
`15`		`- * @version 2.1.41`
	`15`	`+ * @version 2.1.42`
`16`	`16`	`*/`
`17`	`17`	`(function($) {`
`18`	`18`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@`
`13`	`13`	`*`
`14`	`14`	`* @website http://formvalidator.net/#swedish-validators`
`15`	`15`	`* @license Dual licensed under the MIT or GPL Version 2 licenses`
`16`		`- * @version 2.1.41`
	`16`	`+ * @version 2.1.42`
`17`	`17`	`*/`
`18`	`18`	`(function($, window) {`
`19`	`19`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`*`
`10`	`10`	`* @website http://formvalidator.net/#uk-validators`
`11`	`11`	`* @license Dual licensed under the MIT or GPL Version 2 licenses`
`12`		`- * @version 2.1.41`
	`12`	`+ * @version 2.1.42`
`13`	`13`	`*/`
`14`	`14`	`$.formUtils.addValidator({`
`15`	`15`	`name : 'ukvatnumber',`