Fixed issues around validation marionettejs#38

scott-w · Oct 1, 2016 · 6e7c648 · 6e7c648
1 parent b8e490e
commit 6e7c648
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/en/getting_started/tutorial/models.md b/en/getting_started/tutorial/models.md
@@ -242,10 +242,12 @@ var TodoList = Marionette.CompositeView.extend({
     this.model.set({
       assignee: this.ui.assignee.val(),
       text: this.ui.text.val()
-    }, {validate: true});
+    });
 
-    var items = this.model.pick('assignee', 'text');
-    this.collection.add(items);
+    if (this.model.isValid()) {
+      var items = this.model.pick('assignee', 'text');
+      this.collection.add(items);
+    }
   },
 
   itemAdded: function() {
@@ -345,10 +347,12 @@ var TodoList = Marionette.CompositeView.extend({
     this.model.set({
       assignee: this.ui.assignee.val(),
       text: this.ui.text.val()
-    }, {validate: true});
+    });
 
-    var items = this.model.pick('assignee', 'text');
-    this.collection.add(items);
+    if (this.model.isValid()) {
+      var items = this.model.pick('assignee', 'text');
+      this.collection.add(items);
+    }
   },
 
   itemAdded: function() {