Page MenuHomePhabricator

Calendar events should now auto-invite the creator
ClosedPublic

Authored by lpriestley on Apr 29 2015, 8:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 8, 5:02 PM
Unknown Object (File)
Fri, Dec 6, 12:04 AM
Unknown Object (File)
Wed, Dec 4, 1:39 AM
Unknown Object (File)
Wed, Dec 4, 1:39 AM
Unknown Object (File)
Wed, Dec 4, 1:39 AM
Unknown Object (File)
Wed, Dec 4, 1:39 AM
Unknown Object (File)
Wed, Dec 4, 1:39 AM
Unknown Object (File)
Wed, Dec 4, 1:30 AM
Subscribers

Details

Summary

Closes T7935, Calendar events should now auto-invite the creator.

Test Plan

Create event, save, event should now show creator as an invitee.

Diff Detail

Repository
rP Phabricator
Branch
calendareventinvitees
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5594
Build 5613: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

lpriestley retitled this revision from to Calendar events should now auto-invite the creator.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley edited edge metadata.

Removing commented out code.

epriestley edited edge metadata.
epriestley added inline comments.
src/applications/calendar/query/PhabricatorCalendarEventInviteeQuery.php
55

Minor, but prefer to check these strictly against null, e.g.:

if ($this->ids !== null) { ... }

This prevents a bug where callers do:

->withIDs(array())

...by mistake and accidentally select every result.

src/applications/calendar/storage/PhabricatorCalendarEventTransaction.php
303–305

This might be worth breaking out separately at some point since I could imagine a user caring about the event getting rescheduled but not caring about the invitee list changing. Perfectly fine for now, though.

This revision is now accepted and ready to land.Apr 29 2015, 8:36 PM
lpriestley edited edge metadata.

Checking strictly null.

src/applications/calendar/query/PhabricatorCalendarEventInviteeQuery.php
63–89

Oh, sorry, for all of these cases.

This revision was automatically updated to reflect the committed changes.