Merge pull request #3148 from yratanov/feature/snapshot-serialize

add Snapshot#serialize method
emberjs · Jun 3, 2015 · 3897ea1 · 3897ea1
2 parents d6c46d8 + 50c4195
commit 3897ea1
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 1 deletion.
diff --git a/packages/ember-data/lib/system/snapshot.js b/packages/ember-data/lib/system/snapshot.js
@@ -393,6 +393,15 @@ Snapshot.prototype = {
     return get(this.record, keyName);
   },
 
+  /**
+    @method serialize
+    @param {Object} options
+    @return {Object} an object whose values are primitive JSON values only
+   */
+  serialize: function(options) {
+    return this.record.store.serializerFor(this.modelName).serialize(this, options);
+  },
+
   /**
     @method unknownProperty
     @param {String} keyName

diff --git a/packages/ember-data/lib/system/store.js b/packages/ember-data/lib/system/store.js
@@ -254,7 +254,7 @@ Store = Service.extend({
   */
   serialize: function(record, options) {
     var snapshot = record._internalModel.createSnapshot();
-    return this.serializerFor(snapshot.modelName).serialize(snapshot, options);
+    return snapshot.serialize(options);
   },
 
   /**

diff --git a/packages/ember-data/tests/integration/snapshot-test.js b/packages/ember-data/tests/integration/snapshot-test.js
@@ -637,6 +637,20 @@ test("snapshot.get() proxies property to record unless identified as id, attribu
   });
 });
 
+test("snapshot.serialize() serializes itself", function() {
+  expect(2);
+
+  run(function() {
+    var post = env.store.push('post', { id: 1, title: 'Hello World' });
+    var snapshot = post._createSnapshot();
+
+    post.set('title', 'New Title');
+
+    deepEqual(snapshot.serialize(), { author: undefined, title: 'Hello World' }, 'shapshot serializes correctly');
+    deepEqual(snapshot.serialize({ includeId: true }), { id: "1", author: undefined, title: 'Hello World' }, 'serialize takes options')
+  });
+});
+
 test('snapshot.typeKey is deprecated', function() {
   expect(1);