Auto generate phases if not provided

Author: ThomasKranitsas

src/services/ChallengeService.js

@@ -722,13 +722,20 @@ async function validateChallengeData (challenge) {
  * @param {String} timelineTemplateId the timeline template id
  */
 async function populatePhases (phases, startDate, timelineTemplateId) {
-  if (!phases || phases.length === 0) {
-    return
-  }
   if (_.isUndefined(timelineTemplateId)) {
     throw new errors.BadRequestError(`Invalid timeline template ID: ${timelineTemplateId}`)
   }
   const template = await helper.getById('TimelineTemplate', timelineTemplateId)
+  if (!phases || phases.length === 0) {
+    // auto populate phases
+    phases = []
+    for (const p in template.phases) {
+      phases.push({
+        phaseId: p.phaseId,
+        duration: p.defaultDuration
+      })
+    }
+  }
   const phaseDefinitions = await helper.scan('Phase')
   // generate phase instance ids
   for (let i = 0; i < phases.length; i += 1) {
@@ -861,7 +868,11 @@ async function createChallenge (currentUser, challenge, userToken) {
       throw new errors.BadRequestError(`trackId and typeId are required to create a challenge`)
     }
   }
-  if (challenge.timelineTemplateId && challenge.phases && challenge.phases.length > 0) {
+
+  if (challenge.timelineTemplateId) {
+    if (!challenge.phases) {
+      challenge.phases = []
+    }
     await populatePhases(challenge.phases, challenge.startDate, challenge.timelineTemplateId)
   }
 
@@ -1306,7 +1317,7 @@ async function update (currentUser, challengeId, data, userToken, isFull) {
         }
       }
     }
-    const newPhases = challenge.phases
+    const newPhases = challenge.phases || []
     const newStartDate = data.startDate || challenge.startDate
 
     await helper.validatePhases(newPhases)