works; need to improve design greatly

jashkenas · Dec 17, 2011 · 8079618 · 8079618
1 parent d4df49d
commit 8079618
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 27 deletions.
diff --git a/examples/todos-requirejs-2/collections/TodoList.js b/examples/todos-requirejs-2/collections/TodoList.js
@@ -27,8 +27,6 @@ define(function () {
         // GUID in the database. This generates the next order number for new items.
         nextOrder: function() {
             console.log('nextOrder');
-            console.log(this.last().get('order'));
-
 
             if (!this.length) return 1;
             return this.last().get('order') + 1;

diff --git a/examples/todos-requirejs-2/models/Todo.js b/examples/todos-requirejs-2/models/Todo.js
@@ -1,29 +1,32 @@
 define(function () {     
     'use strict';
 
-    var module = {}, Todo;
-
-    // Todo Model
-    // ----------
-
-    // Our basic **Todo** model has `text`, `order`, and `done` attributes.
-    Todo = Backbone.Model.extend({
-
-        // Default attributes for a todo item.
-        defaults: {
-            done:  false
-        },        
-
-        // Toggle the `done` state of this todo item.
-        toggle: function() {
-            console.log('toggle');
-            this.save({done: !this.get("done")});
-        }
+    var module = {};
+
+    module.create = function (params) {
+	    // Todo Model
+	    // ----------
+
+	    // Our basic **Todo** model has `text`, `order`, and `done` attributes.
+	    var Todo = Backbone.Model.extend({
+
+	        // Default attributes for a todo item.
+	        defaults: function() {
+		    	return {
+		        	done:  false,
+		        	order: params.collection.nextOrder()
+		    	};
+	        },
+
+	        // Toggle the `done` state of this todo item.
+	        toggle: function() {
+	            console.log('toggle');
+	            this.save({done: !this.get("done")});
+	        }
 
-    });   
-
-    module.create = function (properties) {
-        return new Todo(properties);            
+    	});         
+
+        return Todo;            
     };  
 
     return module;

diff --git a/examples/todos-requirejs-2/todos.js b/examples/todos-requirejs-2/todos.js
@@ -1,12 +1,14 @@
 // Load the application once the DOM is ready, using `jQuery.ready`:
-define (['collections/TodoList', 'views/App'], function (todoCollection, appView) {
+define (['collections/TodoList', 'views/App', 'models/Todo'], function (todoCollection, appView, todoModel) {
     'use strict';
 
     var module = {};
 
     module.init = function () {       
-        var collection = todoCollection.create();            
-
+        var collection = todoCollection.create(), 
+        	model = todoModel.create({ 'collection' : collection });  
+
+		collection.model = model;
         return appView.create({ 'collection' : collection });             
     };