Trying to fix the merge of priceConfig. It should just override the complete JSON.

Author: Vikas Agarwal

src/routes/projectTemplates/update.js

@@ -64,7 +64,7 @@ module.exports = [
         }
 
         // Merge JSON fields
-        entityToUpdate.scope = util.mergeJsonObjects(projectTemplate.scope, entityToUpdate.scope);
+        entityToUpdate.scope = util.mergeJsonObjects(projectTemplate.scope, entityToUpdate.scope, ['priceConfig']);
         entityToUpdate.phases = util.mergeJsonObjects(projectTemplate.phases, entityToUpdate.phases);
         // removes null phase templates
         entityToUpdate.phases = _.omitBy(entityToUpdate.phases, _.isNull);

src/util.js

@@ -379,12 +379,13 @@ _.assignIn(util, {
   * Merge two JSON objects. For array fields, the target will be replaced by source.
   * @param {Object} targetObj the target object
   * @param {Object} sourceObj the source object
+  * @param {Object} mergeExceptions list of keys which should be exempted from merge
   * @returns {Object} the merged object
   */
   // eslint-disable-next-line consistent-return
-  mergeJsonObjects: (targetObj, sourceObj) => _.mergeWith(targetObj, sourceObj, (target, source) => {
-    // Overwrite the array
-    if (_.isArray(source)) {
+  mergeJsonObjects: (targetObj, sourceObj, mergeExceptions) => _.mergeWith(targetObj, sourceObj, (target, source, key) => {
+    // Overwrite the array or merge exception keys
+    if (_.isArray(source) || (mergeExceptions && mergeExceptions.indexOf(key) !== -1)) {
       return source;
     }
   }),