Protect against null value for these date fields.

Potentially fixes broken date values being sent to EAS clients, thus breaking sync.
horde · Jan 15, 2018 · e145146 · e145146
1 parent 593c228
commit e145146
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/lib/Task.php b/lib/Task.php
@@ -1491,21 +1491,27 @@ public function toASTask(array $options = array())
 
         /* Completion */
         if ($this->completed) {
-            $message->datecompleted = new Horde_Date($this->completed_date);
+            if ($this->completed_date) {
+                $message->datecompleted = new Horde_Date($this->completed_date);
+            }
             $message->complete = Horde_ActiveSync_Message_Task::TASK_COMPLETE_TRUE;
         } else {
             $message->complete = Horde_ActiveSync_Message_Task::TASK_COMPLETE_FALSE;
         }
 
         /* Due Date */
         if (!empty($this->due)) {
-            $message->utcduedate = new Horde_Date($this->due);
+            if ($this->due) {
+                $message->utcduedate = new Horde_Date($this->due);
+            }
             $message->duedate = clone($message->utcduedate);
         }
 
         /* Start Date */
         if (!empty($this->start)) {
-            $message->utcstartdate = new Horde_Date($this->start);
+            if ($this->start) {
+                $message->utcstartdate = new Horde_Date($this->start);
+            }
             $message->startdate = clone($message->utcstartdate);
         }