Add server side code to expose readonly property and use it in the frontend

Author: willdurand

app/Document.js

@@ -9,6 +9,7 @@ export default class Document extends Record({
   last_modified: null, // defined by the server
   last_modified_locally: null,
   template: '',
+  readonly: false,
 }) {
 
   hasDefaultContent() {
@@ -26,4 +27,8 @@ export default class Document extends Record({
   isDefault() {
     return this.hasDefaultContent() && this.hasNeverBeenSync() && this.hasNoLocalChanges();
   }
+
+  isReadOnly() {
+    return true === this.readonly;
+  }
 }

app/__tests__/server-test.js

@@ -37,7 +37,8 @@ describe('Express app', () => {
         .expect(200, {
           content: { ba: 'bar' },
           last_modified: 1459441561629,
-          uuid: '9950e80b-f214-45d0-a98c-bffee2582c71'
+          uuid: '9950e80b-f214-45d0-a98c-bffee2582c71',
+          readonly: false,
         }, done);
     });
 
@@ -49,6 +50,7 @@ describe('Express app', () => {
           content: { ba: 'bar' },
           last_modified: 1459441561629,
           uuid: READONLY_DOCUMENT_UUID,
+          readonly: true,
         }, done);
     });
   });
@@ -128,7 +130,7 @@ describe('Express app', () => {
           content: content,
           uuid: EXISTING_DOCUMENT_UUID,
           template: '',
-          last_modified: 'date'
+          last_modified: 'date',
         }, done);
     });
 
@@ -143,6 +145,7 @@ describe('Express app', () => {
           },
           last_modified: 1459441561629,
           uuid: READONLY_DOCUMENT_UUID,
+          readonly: true,
         }, done);
     });
   });

app/modules/monod.js

@@ -76,6 +76,7 @@ export function load(id, secret) {
             content: res.body.content,
             last_modified: res.body.last_modified,
             template: res.body.template || '', // avoid BC break
+            readonly: res.body.readonly || false, // avoid BC break
           }));
         })
         .catch((err) => {

app/modules/persistence.js

@@ -76,6 +76,7 @@ export function serverPersist() {
           last_modified: res.body.last_modified,
           last_modified_locally: null,
           template: res.body.template || '',
+          readonly: res.body.readonly || false,
         });
 
         return Promise.resolve(current);
@@ -107,6 +108,7 @@ export function serverPersist() {
             last_modified: res.body.last_modified,
             last_modified_locally: null,
             template: res.body.template || '',
+            readonly: res.body.readonly || false,
           });
 
           dispatch(warning(config.READONLY_MESSAGE));

app/modules/sync.js

@@ -66,6 +66,7 @@ export function synchronize() { // eslint-disable-line import/prefer-default-exp
           content: res.body.content,
           last_modified: res.body.last_modified,
           template: res.body.template || '', // avoid BC break
+          readonly: res.body.readonly || false, // avoid BC break
         }));
       })
       .catch((err) => {
@@ -140,6 +141,7 @@ export function synchronize() { // eslint-disable-line import/prefer-default-exp
                 content: serverDoc.get('content'),
                 last_modified: serverDoc.get('last_modified'),
                 template: serverDoc.get('template'),
+                readonly: serverDoc.get('readonly'),
               });
 
               return db

server.js

@@ -47,10 +47,7 @@ if ('production' === process.env.NODE_ENV || 'test' === process.env.NODE_ENV) {
       }
 
       const document = JSON.parse(data);
-
-      if (document.readonly) {
-        delete(document.readonly);
-      }
+      document.readonly = !!document.readonly;
 
       return res.json(document);
     });
@@ -70,8 +67,6 @@ if ('production' === process.env.NODE_ENV || 'test' === process.env.NODE_ENV) {
       const document = readErr ? {} : JSON.parse(data);
 
       if (!!document.readonly) {
-        delete(document.readonly);
-
         return res.status(403).json(document);
       }