revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-18 19:40:55 +01:00
Code Issues Releases Wiki Activity

Properly import all-day events in Calendar

Browse source 
Summary: Ref T10747. The transaction version of this copies the "all day" flag over properly, but this non-transaction version needs to copy it explicitly.

Test Plan: Imported an all-day event, saw it come in as all-day.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10747

Differential Revision: https://secure.phabricator.com/D16770
This commit is contained in:
epriestley 2016-10-30 08:13:08 -07:00
parent 5fd79479ec
commit 96b064b7e9
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/calendar/import/PhabricatorCalendarImportEngine.php
View file

@ -520,6 +520,8 @@ abstract class PhabricatorCalendarImportEngine
->setStartDateTime($start_datetime) ->setStartDateTime($start_datetime)
->setEndDateTime($end_datetime); ->setEndDateTime($end_datetime);
$event->setIsAllDay($start_datetime->getIsAllDay());
// TODO: This should be transactional, but the transaction only accepts // TODO: This should be transactional, but the transaction only accepts
// simple frequency rules right now. // simple frequency rules right now.
$rrule = $node->getRecurrenceRule(); $rrule = $node->getRecurrenceRule();