Fix jashkenas#1179 - isValid returns true in the absence of validate.

jandas · Apr 4, 2012 · 137b7fd · 137b7fd
1 parent a2ed079
commit 137b7fd
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/backbone.js b/backbone.js
@@ -529,7 +529,7 @@
     // Check if the model is currently in a valid state. It's only possible to
     // get into an *invalid* state if you're using silent changes.
     isValid: function() {
-      return !this.validate(this.attributes);
+      return !this.validate || !this.validate(this.attributes);
     },
 
     // Run validation against the next complete set of model attributes,

diff --git a/test/model.js b/test/model.js
@@ -784,4 +784,10 @@ $(document).ready(function() {
     }
   });
 
+  test("#1179 - isValid returns true in the absence of validate.", function() {
+    var model = new Backbone.Model();
+    model.validate = null;
+    ok(model.isValid());
+  });
+
 });