From 80bd4ccfe8e42a2af3bf408cd9be860561e8004b Mon Sep 17 00:00:00 2001
From: Jure Triglav <juretriglav@gmail.com>
Date: Tue, 9 Jun 2020 16:01:40 +0200
Subject: [PATCH] feat(formbuilder): write forms as pretty json

---
 server/formbuilder/src/resolvers.js | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/server/formbuilder/src/resolvers.js b/server/formbuilder/src/resolvers.js
index 85d6939427..160853d1f6 100644
--- a/server/formbuilder/src/resolvers.js
+++ b/server/formbuilder/src/resolvers.js
@@ -3,6 +3,10 @@ const fs = require('fs')
 const { readFiles, mkdirp } = require('./util')
 const form = require('../../../app/storage/forms/submit.json')
 
+const writeJson = (path, object) => {
+  return fs.writeFileSync(path, JSON.stringify(object, null, 2))
+}
+
 const mergeFiles = path =>
   readFiles(path).then(files => {
     const forms = []
@@ -17,7 +21,6 @@ const mergeFiles = path =>
 const resolvers = {
   Mutation: {
     async deleteForms(_, { formId }, ctx) {
-      // DONE
       try {
         const folderPath = `${config.get(
           'pubsweet-component-xpub-formbuilder.path',
@@ -36,7 +39,6 @@ const resolvers = {
       }
     },
     async deleteFormElement(_, { formId, elementId }, ctx) {
-      // DONE
       try {
         const folderPath = `${config.get(
           'pubsweet-component-xpub-formbuilder.path',
@@ -48,7 +50,7 @@ const resolvers = {
         if (forms.children) {
           const children = forms.children.filter(el => el.id !== elementId)
           forms.children = children
-          fs.writeFileSync(path, JSON.stringify(forms))
+          writeJson(path, forms)
         }
 
         const form = await mergeFiles(folderPath)
@@ -58,7 +60,6 @@ const resolvers = {
       }
     },
     async createForm(_, { form }, ctx) {
-      // DONE
       try {
         form = JSON.parse(form)
         const folderPath = `${config.get(
@@ -68,7 +69,7 @@ const resolvers = {
 
         if (!fs.existsSync(path)) {
           mkdirp(folderPath)
-          fs.writeFileSync(path, JSON.stringify(form))
+          writeJson(path, form)
         }
 
         const forms = await mergeFiles(folderPath)
@@ -97,7 +98,7 @@ const resolvers = {
           }
         }
 
-        fs.writeFileSync(path, JSON.stringify(form))
+        writeJson(path, form)
 
         const forms = await mergeFiles(folderPath)
 
@@ -126,7 +127,7 @@ const resolvers = {
           forms.children.push(children)
         }
 
-        fs.writeFileSync(path, JSON.stringify(forms))
+        writeJson(path, forms)
         const form = await mergeFiles(folderPath)
         return form
       } catch (err) {
-- 
GitLab