no longer skipping a custom validator if its attr has a falsy value. fixes #475

Author: 1000hz

js/tests/unit/validator.js

@@ -533,6 +533,10 @@ $(function () {
       +   '<input type="text" id="bar" data-foo="foo" value="bar">'
       +   '<div class="help-block with-errors"></div>'
       + '</div>'
+      + '<div class="form-group">'
+      +   '<input type="text" id="baz" data-foo value="baz">'
+      +   '<div class="help-block with-errors"></div>'
+      + '</div>'
       + '<button type="submit">Submit</button>'
       + '</form>'
 
@@ -552,6 +556,7 @@ $(function () {
     assert.ok($('#foo').data('bs.validator.errors').length === 0, 'foo input is valid')
     assert.ok($('#bar').data('bs.validator.errors').length === 1, 'bar input is invalid')
     assert.ok($('#bar').data('bs.validator.errors')[0] === 'not equal to foo', 'bar error is custom error')
+    assert.ok($('#baz').data('bs.validator.errors').length === 1, 'baz ran validator even though data-foo has no attr value')
   })
 
   QUnit.test('should update set of fields', function (assert) {

js/validator.js

@@ -204,7 +204,7 @@
     $.each(this.validators, $.proxy(function (key, validator) {
       var error = null
       if ((getValue($el) || $el.attr('required')) &&
-          ($el.attr('data-' + key) || key == 'native') &&
+          ($el.attr('data-' + key) !== undefined || key == 'native') &&
           (error = validator.call(this, $el))) {
          error = getErrorMessage(key) || error
         !~errors.indexOf(error) && errors.push(error)