Merge pull request #2994 from Getriax/TRIM_MODEL

Trim content builder model attributes

packages/strapi-plugin-content-type-builder/services/ContentTypeBuilder.js

@@ -6,7 +6,12 @@ const _ = require('lodash');
 const generator = require('strapi-generate');
 const { fromJS, List, Map } = require('immutable');
 const Manager = require('../utils/Manager.js');
-const { createManager, removeColsLine, reorderList } = require('../utils/helpers.js');
+const {
+  createManager,
+  removeColsLine,
+  reorderList,
+  deepTrimObject,
+} = require('../utils/helpers.js');
 
 module.exports = {
   appearance: async (attributes, model, plugin) => {
@@ -369,8 +374,9 @@ module.exports = {
     });
 
     Object.assign(attributesNotConfigurable, attrs);
+    const trimmedNotConfigurableAttributes = deepTrimObject(attributesNotConfigurable);
 
-    return [attributesNotConfigurable, errors];
+    return [trimmedNotConfigurableAttributes, errors];
   },
 
   clearRelations: (model, source, force) => {

packages/strapi-plugin-content-type-builder/utils/helpers.js

@@ -65,11 +65,31 @@ const escapeNewlines = (content, placeholder = '\n') => {
   return content.replace(/[\r\n]+/g, placeholder);
 }
 
+const deepTrimObject = attribute => {
+  if (Array.isArray(attribute)) {
+    return attribute.map(deepTrimObject);
+  }
+
+  if (typeof attribute === 'object') {
+    return Object.entries(attribute)
+      .reduce((acc, [key, value]) => {
+        const trimmedObject = deepTrimObject(value);
+
+        return { ...acc, [key]: trimmedObject };
+      }, {});
+  }
+
+  return typeof attribute === 'string'
+    ? attribute.trim()
+    : attribute;
+}
+
 module.exports = {
   createArrayOfLastEls,
   createManager,
   getElementsOnALine,
   removeColsLine,
   reorderList,
-  escapeNewlines
+  escapeNewlines,
+  deepTrimObject
 };