add if statement and add passing test

vitrum · Apr 22, 2013 · 2caf748 · 2caf748
1 parent c3a1af0
commit 2caf748
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/backbone.js b/backbone.js
@@ -1183,7 +1183,7 @@
 
     // Make the request, allowing the user to override any Ajax options.
     var xhr = options.xhr = Backbone.ajax(_.extend(params, options));
-    model.trigger('request', model, xhr, options);
+    if(model) model.trigger('request', model, xhr, options);
     return xhr;
   };
 

diff --git a/test/sync.js b/test/sync.js
@@ -209,4 +209,10 @@ $(document).ready(function() {
     strictEqual(this.ajaxSettings.beforeSend(xhr), false);
   });
 
+  test("correctly check for model before triggering request event", 1, function() {
+    Backbone.emulateHTTP = true;
+    Backbone.sync('read', false, {url : "/someurl", data : ""}) 
+    ok(true, "No errors were thrown");
+  });
+
 });