Differential D13212

Check if recurrence end date is disabled before saving it.
ClosedPublic
Actions

Authored by lpriestley on Jun 8 2015, 6:57 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T8357: Add recurrence end date

Commits

Restricted Diffusion Commit
rPace7505f2e98: Check if recurrence end date is disabled before saving it.

Summary

Ref T8357, Check if recurrence end date is disabled before saving it.

Test Plan

Create new event, before saving, leave "recurrence end date" unchecked, save, should not get an error.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 31928.Jun 8 2015, 6:57 PM

lpriestley retitled this revision from to Check if recurrence end date is disabled before saving it..

lpriestley updated this object.

lpriestley edited the test plan for this revision. (Show Details)

lpriestley added a reviewer: epriestley.

lpriestley added a task: T8357: Add recurrence end date.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 8 2015, 6:57 PM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

epriestley accepted this revision.Jun 8 2015, 6:57 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jun 8 2015, 6:57 PM

Closed by commit rPace7505f2e98: Check if recurrence end date is disabled before saving it. (authored by lpriestley, committed by lpriestley). · Explain WhyJun 8 2015, 6:58 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

calendar/

controller/

PhabricatorCalendarEventEditController.php

10 lines

Diff 31929

View Options

src/applications/calendar/controller/PhabricatorCalendarEventEditController.php

Check if recurrence end date is disabled before saving it.ClosedPublicActions