work in progress

Author: Tronix117

dist/js-data-jsonapi-light.js

@@ -71,6 +71,16 @@ export class JsonApiAdapter extends HttpAdapter{
     // Ensure id is properly set
     props[mapper.idAttribute] = id;
 
+     if (!opts.replace) {
+        opts.method = opts.method || 'patch';
+      }
+
+    // We need the record to get changes
+    let record = (<any>mapper).datastore.get(mapper.name, id);
+    if (record) {
+      console.info(record.changes());
+    }
+
     return this.handleBeforeLifecycle(opts).then(() => {
       return HttpAdapter.prototype.update.call(this, mapper, id, props, opts)
     }).then(this.handleResponse(opts))

dist/js-data-jsonapi-light.js.map

@@ -20,6 +20,11 @@ export function jsonApiSerialize (mapper:any, data:any, opts:any){
   // Just cache a pointer to relations for the Resource
   mapperCacheRelationByField(mapper);
 
+  if (opts.changes && id && mapper.changes(id)) {
+    let changes = mapper.changes(id);
+    console.info('changes', changes);
+  }
+
   let relationships:any = {}
 
   for (let key in data) {
@@ -44,7 +49,9 @@ export function jsonApiSerialize (mapper:any, data:any, opts:any){
 
   // Work for update or create, if an id is given, server should accept it
   if (id) output.data.id = id;
-  if (Object.keys(relationships)) output.data.relationships = relationships;
+  if (Object.keys(relationships).length) {
+    output.data.relationships = relationships;
+  }
 
   return output;
 }

src/adapter.ts

@@ -79,8 +79,7 @@ describe('UPDATE', () => {
       reqPatch = reqPut = null;
     })
 
-    // @todo #9
-    xit('the request should receive only modified fields per default', () => {
+    it('the request should receive only modified fields per default', () => {
       return store.update('Article', ID, UPDATE_PARAMS).then((data) => {
         expect(reqPatch.body).to.deep.equal({
           data: {
@@ -102,7 +101,28 @@ describe('UPDATE', () => {
       })
     });
 
-    it('the request should receive all fields when used with replace: true', () => {
+    it('the request should only received changes when a record is saved', () => {
+      return store.find('Article', ID).then((record) => {
+        record.title = UPDATE_PARAMS.title;
+        return record.save();
+      }).then((data) => {
+        console.info(reqPatch.body);
+        expect(reqPatch.body).to.deep.equal({
+          data: {
+            type: MAPPER_NAME,
+            id: ID,
+            attributes: { title: UPDATE_PARAMS.title }
+          }
+        })
+
+        expect(data).to.be.an('object')
+        expect(data.id).to.equal(ID)
+        expect(data.title).to.equal(UPDATE_PARAMS.title)
+        expect(data.content).to.equal(FIND_RESPONSE.data.attributes.content)
+      })
+    });
+
+    it('the request should send relation when a record is saved.', () => {
       return store.update('Article', ID, UPDATE_PARAMS, {
         replace: true
       }).then((data) => {
@@ -123,6 +143,10 @@ describe('UPDATE', () => {
         expect(data.content).to.equal(FIND_RESPONSE.data.attributes.content)
       })
     });
+
+    it('the request should send all attributes and relationships when record is saved with option `replace`', () =>{
+      return true;
+    })
   })
 
   describe('when use of unsupported methods', () => {

src/serializer.ts

@@ -95,7 +95,7 @@ describe('Lifecycle', () => {
 
       return store.update('User', USER.ID, { 
         email: USER.EMAIL 
-      }, { beforeSerialize }).then(() => {
+      }, { beforeSerialize, replace: true }).then(() => {
         expect(beforeSerialize.calledOnce).to.be.true;
         expect(req.body.data.attributes).to.deep.equal({
           email: USER.EMAIL,
@@ -122,7 +122,7 @@ describe('Lifecycle', () => {
 
       return store.update('User', USER.ID, { 
         email: USER.EMAIL 
-      }, { afterSerialize }).then(() => {
+      }, { afterSerialize, replace: true }).then(() => {
         expect(afterSerialize.calledOnce).to.be.true;
         expect(req.body.data.attributes).to.deep.equal({
           email: USER.EMAIL,