Removed in built validation and added explicit validation on startDate and endDate so that we can pass them without each other

Author: Vikas Agarwal

src/routes/phases/create.js

@@ -14,7 +14,7 @@ const addProjectPhaseValidations = {
     param: Joi.object().keys({
       name: Joi.string().required(),
       status: Joi.string().required(),
-      startDate: Joi.date().max(Joi.ref('endDate')).optional(),
+      startDate: Joi.date().optional(),
       endDate: Joi.date().optional(),
       duration: Joi.number().min(0).optional(),
       budget: Joi.number().min(0).optional(),
@@ -52,6 +52,11 @@ module.exports = [
           err.status = 404;
           throw err;
         }
+        if (data.startDate !== null && data.endDate !== null && data.startDate > data.endDate) {
+          const err = new Error('startDate must not be after endDate.');
+          err.status = 400;
+          throw err;
+        }
         return models.ProjectPhase
           .create(data)
           .then((_newProjectPhase) => {