From 6e5565b3ff6d1c730f2a2eedbeab755917eeb98a Mon Sep 17 00:00:00 2001 From: epriestley Date: Tue, 15 Nov 2016 13:49:25 -0800 Subject: [PATCH] Clean up a little more Calendar display logic Summary: Uh, non-all-day-events said 1:30 - 2:29 PM, which is real silly. Test Plan: Looked at a non-all-day-event. Reviewers: chad Reviewed By: chad Differential Revision: https://secure.phabricator.com/D16874 --- .../calendar/storage/PhabricatorCalendarEvent.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/applications/calendar/storage/PhabricatorCalendarEvent.php b/src/applications/calendar/storage/PhabricatorCalendarEvent.php index adcaa6f316..75d573c79e 100644 --- a/src/applications/calendar/storage/PhabricatorCalendarEvent.php +++ b/src/applications/calendar/storage/PhabricatorCalendarEvent.php @@ -584,10 +584,12 @@ final class PhabricatorCalendarEvent extends PhabricatorCalendarDAO $min_date = $start->newPHPDateTime(); $max_date = $end->newPHPDateTime(); - if ($show_end) { + if ($this->getIsAllDay()) { // Subtract one second since the stored date is exclusive. $max_date = $max_date->modify('-1 second'); + } + if ($show_end) { $min_day = $min_date->format('Y m d'); $max_day = $max_date->format('Y m d');