@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
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
+3
-1
+3
-1
src/applications/calendar/storage/PhabricatorCalendarEvent.php
+3
-1
src/applications/calendar/storage/PhabricatorCalendarEvent.php
···
584
584
$min_date = $start->newPHPDateTime();
585
585
$max_date = $end->newPHPDateTime();
586
586
587
-
if ($show_end) {
587
+
if ($this->getIsAllDay()) {
588
588
// Subtract one second since the stored date is exclusive.
589
589
$max_date = $max_date->modify('-1 second');
590
+
}
590
591
592
+
if ($show_end) {
591
593
$min_day = $min_date->format('Y m d');
592
594
$max_day = $max_date->format('Y m d');
593
595