Permalink
Browse files

save with non-model success

  • Loading branch information...
1 parent a24544b commit c396adb2dde5265a6a76e8affc75d5bf47d0ab4a @tgriesser tgriesser committed Dec 26, 2012
Showing with 13 additions and 1 deletion.
  1. +1 −1 backbone.js
  2. +12 −0 test/model.js
View
@@ -461,7 +461,7 @@
model.attributes = attributes;
var serverAttrs = model.parse(resp, options);
if (options.wait) serverAttrs = _.extend(attrs || {}, serverAttrs);
- if (!model.set(serverAttrs, options)) return false;
+ if (typeof serverAttrs === 'object' && !model.set(serverAttrs, options)) return false;
if (success) success(model, resp, options);
};
View
@@ -423,6 +423,18 @@ $(document).ready(function() {
equal(model.get('title'), 'Twelfth Night');
});
+ test("save with non-object success response", 1, function () {
+ var model = new Backbone.Model();
+ model.sync = function(method, model, options) {
+ options.success(model, '', options);
+ };
+ model.save({testing:'empty'}, {
+ success: function (model) {
+ deepEqual(model.attributes, {testing:'empty'});
+ }
+ });
+ });
+
test("fetch", 2, function() {
doc.fetch();
equal(this.syncArgs.method, 'read');

0 comments on commit c396adb

Please sign in to comment.