From fbebb8643c1af3d126efc311c441d6cac14315ae Mon Sep 17 00:00:00 2001
From: louietyj <louietyj@gmail.com>
Date: Mon, 7 Nov 2016 13:49:21 +0800
Subject: [PATCH] Refactor validateParams

---
 src/main/java/seedu/todo/controllers/UpdateController.java | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/main/java/seedu/todo/controllers/UpdateController.java b/src/main/java/seedu/todo/controllers/UpdateController.java
index 43b40c6bf021..5578b4880c07 100644
--- a/src/main/java/seedu/todo/controllers/UpdateController.java
+++ b/src/main/java/seedu/todo/controllers/UpdateController.java
@@ -251,11 +251,7 @@ private boolean validateParams(boolean isTask, CalendarItem record, String name,
             LocalDateTime newDateFrom = (dateFrom == null) ? event.getStartDate() : dateFrom;
             LocalDateTime newDateTo = (dateTo == null) ? event.getEndDate() : dateTo;
 
-            if (newDateFrom == null || newDateTo == null) {
-                return false;
-            }
-
-            if (newDateTo.isBefore(newDateFrom)) {
+            if (newDateFrom == null || newDateTo == null || newDateTo.isBefore(newDateFrom)) {
                 return false;
             }
         }