From 5ddaf5289862cce3957c5630eb11df31fb0786b8 Mon Sep 17 00:00:00 2001 From: Chad Little Date: Sat, 9 Apr 2016 11:50:16 -0700 Subject: [PATCH] (stable) Set time and date on Calendar Date Control form Summary: Recurring events will fatal a Calendar with this not set. `newDateTime` requires a date and time to be called property. I think this is correct fix? Fixes T10766 Test Plan: Build a recurring event, pull up /calendar/, see recurring events as expected. Previously, fatal. Reviewers: lpriestley, epriestley Reviewed By: epriestley Subscribers: CodeMouse92, Korvin Maniphest Tasks: T10766 Differential Revision: https://secure.phabricator.com/D15666 --- src/view/form/control/AphrontFormDateControlValue.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/view/form/control/AphrontFormDateControlValue.php b/src/view/form/control/AphrontFormDateControlValue.php index 114340006e..db2c7235c4 100644 --- a/src/view/form/control/AphrontFormDateControlValue.php +++ b/src/view/form/control/AphrontFormDateControlValue.php @@ -279,7 +279,7 @@ final class AphrontFormDateControlValue extends Phobject { } public function getDateTime() { - return $this->newDateTime(); + return $this->newDateTime($this->valueDate, $this->valueTime); } private function getTimezone() {