Page MenuHomePhabricator
Files F15388006

D16787.id40431.diff
No OneTemporary
Actions

D16787.id40431.diff
View Options

diff --git a/src/applications/calendar/import/PhabricatorCalendarImportEngine.php b/src/applications/calendar/import/PhabricatorCalendarImportEngine.php
--- a/src/applications/calendar/import/PhabricatorCalendarImportEngine.php
+++ b/src/applications/calendar/import/PhabricatorCalendarImportEngine.php
@@ -225,6 +225,7 @@
$xactions[$full_uid] = $this->newUpdateTransactions($event, $node);
$update_map[$full_uid] = $event;
+ $attendee_map[$full_uid] = array();
$attendees = $node->getAttendees();
$private_index = 1;
foreach ($attendees as $attendee) {
@@ -526,7 +527,7 @@
->setStartDateTime($start_datetime)
->setEndDateTime($end_datetime);
- $event->setIsAllDay($start_datetime->getIsAllDay());
+ $event->setIsAllDay((int)$start_datetime->getIsAllDay());
// TODO: This should be transactional, but the transaction only accepts
// simple frequency rules right now.
@@ -551,11 +552,18 @@
PhabricatorUser $viewer,
PhabricatorCalendarImport $import) {
- $any_event = id(new PhabricatorCalendarEventQuery())
- ->setViewer($viewer)
- ->withImportSourcePHIDs(array($import->getPHID()))
- ->setLimit(1)
- ->execute();
+ $table = new PhabricatorCalendarEvent();
+ $conn = $table->establishConnection('r');
+
+ // Using a CalendarEventQuery here was failing oddly in a way that was
+ // difficult to reproduce locally (see T11808). Just check the table
+ // directly; this is significantly more efficient anyway.
+
+ $any_event = queryfx_all(
+ $conn,
+ 'SELECT phid FROM %T WHERE importSourcePHID = %s LIMIT 1',
+ $table->getTableName(),
+ $import->getPHID());
return (bool)$any_event;
}
diff --git a/src/applications/calendar/query/PhabricatorCalendarEventQuery.php b/src/applications/calendar/query/PhabricatorCalendarEventQuery.php
--- a/src/applications/calendar/query/PhabricatorCalendarEventQuery.php
+++ b/src/applications/calendar/query/PhabricatorCalendarEventQuery.php
@@ -126,7 +126,7 @@
return array(
'start' => array(
'table' => $this->getPrimaryTableAlias(),
- 'column' => 'dateFrom',
+ 'column' => 'utcInitialEpoch',
'reverse' => true,
'type' => 'int',
'unique' => false,

File Metadata

Mime Type
text/plain
Expires
Sun, Mar 16, 3:11 AM (3 w, 21 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7705421
Default Alt Text
D16787.id40431.diff (2 KB)

Event Timeline

Phabricator · Built by Phacility