Removes null values from phases key of project templates

Author: Vikas Agarwal

src/routes/projectTemplates/update.js

@@ -66,6 +66,8 @@ module.exports = [
         // Merge JSON fields
         entityToUpdate.scope = util.mergeJsonObjects(projectTemplate.scope, entityToUpdate.scope);
         entityToUpdate.phases = util.mergeJsonObjects(projectTemplate.phases, entityToUpdate.phases);
+        // removes null phase templates
+        entityToUpdate.phases = _.omitBy(entityToUpdate.phases, _.isNull)
 
         return projectTemplate.update(entityToUpdate);
       })

src/util.js

@@ -379,14 +379,11 @@ _.assignIn(util, {
   * @returns {Object} the merged object
   */
   // eslint-disable-next-line consistent-return
-  mergeJsonObjects: (targetObj, sourceObj) => _.mergeWith(targetObj, sourceObj, (target, source, key) => {
+  mergeJsonObjects: (targetObj, sourceObj) => _.mergeWith(targetObj, sourceObj, (target, source) => {
     // Overwrite the array
     if (_.isArray(source)) {
       return source;
     }
-    if (source === null) {
-      _.unset(target, key);
-    }
   }),
 });