Page MenuHomePhabricator
Differential D13121

Cancelled recurring events should propogate to real child events
ClosedPublic
Actions

Authored by lpriestley on Jun 2 2015, 3:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 7, 5:31 AM
Unknown Object (File)
Fri, May 3, 3:12 AM
Unknown Object (File)
Mon, Apr 29, 2:41 PM
Unknown Object (File)
Thu, Apr 25, 5:56 PM
Unknown Object (File)
Wed, Apr 24, 10:43 PM
Unknown Object (File)
Sun, Apr 21, 5:29 PM
Unknown Object (File)
Sat, Apr 13, 5:33 PM
Unknown Object (File)
Apr 8 2024, 5:42 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8371: Canceling a recurrence should soft-cancel all exceptions to the recurrence.
Commits
Restricted Diffusion Commit
rP446611e2e323: Cancelled recurring events should propogate to real child events
Summary

Ref T8371, Cancelled recurring events should propogate to real child events

Test Plan

Create recurring event, create and exception to a ghost event, cancel recurring event, real ghost event should be treated as cancelled while the recurring event remains cancelled.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 31677.Jun 2 2015, 3:51 PM
lpriestley retitled this revision from to Cancelled recurring events should propogate to real child events.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T8371: Canceling a recurrence should soft-cancel all exceptions to the recurrence..
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 2 2015, 3:51 PM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
lpriestley updated this revision to Diff 31678.Jun 2 2015, 4:18 PM
lpriestley edited edge metadata.

Attempting to optimize getting parent events

lpriestley updated this revision to Diff 31679.EditedJun 2 2015, 4:23 PM

making use of lazy evaluation

lpriestley updated this revision to Diff 31680.Jun 2 2015, 4:33 PM

cleanup

lpriestley updated this revision to Diff 31699.Jun 2 2015, 11:58 PM

syncing most recent

lpriestley updated this revision to Diff 31703.Jun 3 2015, 1:31 AM

cleaning up lint complaints

lpriestley updated this revision to Diff 31704.Jun 3 2015, 1:44 AM

Ghost exceptions were not being unset before this.

lpriestley updated this revision to Diff 31705.Jun 3 2015, 2:02 AM

queries with no recurring events should not produce error.

lpriestley updated this revision to Diff 31706.Jun 3 2015, 2:17 AM

not filtering out ghost events.

lpriestley updated this revision to Diff 31707.Jun 3 2015, 2:38 AM

Fix problems with editing ghost events

lpriestley updated this revision to Diff 31708.Jun 3 2015, 2:40 AM

helpful comment

epriestley accepted this revision.Jun 3 2015, 2:42 AM
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/calendar/storage/PhabricatorCalendarEvent.php
351–353

This is never possible -- we either have a parent event or this throws because one wasn't attached.

This revision is now accepted and ready to land.Jun 3 2015, 2:42 AM
lpriestley updated this revision to Diff 31709.Jun 3 2015, 2:43 AM
lpriestley edited edge metadata.

addressing code review

lpriestley marked an inline comment as done.Jun 3 2015, 2:43 AM
Closed by commit rP446611e2e323: Cancelled recurring events should propogate to real child events (authored by lpriestley, committed by lpriestley). · Explain WhyJun 3 2015, 2:44 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
calendar/
controller/
2 lines
query/
61 lines
storage/
26 lines

Diff 31710

View Options

src/applications/calendar/controller/PhabricatorCalendarEventViewController.php

Loading...
View Options

src/applications/calendar/query/PhabricatorCalendarEventQuery.php

Loading...
View Options

src/applications/calendar/storage/PhabricatorCalendarEvent.php

Loading...
Phabricator · Built by Phacility