CRM-17802 Fix default end and start times and switch to use !empty rather than isset
authorSeamus Lee <seamuslee001@gmail.com>
Mon, 11 Jan 2016 22:51:59 +0000 (22:51 +0000)
committerSeamus Lee <seamuslee001@gmail.com>
Mon, 11 Jan 2016 23:26:27 +0000 (23:26 +0000)
CRM/Contact/Form/Search/Custom/ContributionAggregate.php

index 2b8e2d7f9b91ecbe2b1f43df6d217f1f096da010..a86bbc2a20a348008f0e7ad60913e4a4cb27d451 100644 (file)
@@ -200,8 +200,8 @@ civicrm_contact AS contact_a {$this->_aclFrom}
     $clauses[] = "contrib.contact_id = contact_a.id";
     $clauses[] = "contrib.is_test = 0";
 
-    $startTime = isset($this->_formValues['start_date_time']) ? $this->_formValues['start_date_time'] : '12:00AM';
-    $endTime = isset($this->_formValues['end_date_time']) ? $this->_formValues['end_date_time'] : '11:59PM';
+    $startTime = !empty($this->_formValues['start_date_time']) ? $this->_formValues['start_date_time'] : '12:00:00';
+    $endTime = !empty($this->_formValues['end_date_time']) ? $this->_formValues['end_date_time'] : '23:59:59';
 
     $startDate = CRM_Utils_Date::processDate($this->_formValues['start_date'], $startTime);
     if ($startDate) {