Page MenuHomePhabricator

Clicking in day view should create new event
ClosedPublic

Authored by lpriestley on May 24 2015, 5:33 PM.
Tags
None
Referenced Files
F15584447: D12990.id31314.diff
Thu, May 8, 3:43 AM
F15574529: D12990.id31315.diff
Tue, May 6, 1:57 AM
F15546163: D12990.id.diff
Sat, Apr 26, 2:09 PM
F15541812: D12990.diff
Fri, Apr 25, 1:39 PM
F15531972: D12990.id31313.diff
Wed, Apr 23, 2:01 PM
F15503110: D12990.id31314.diff
Mon, Apr 14, 6:57 AM
F15502650: D12990.id.diff
Mon, Apr 14, 2:52 AM
F15489603: D12990.id.diff
Fri, Apr 11, 10:37 AM
Subscribers

Details

Summary

Ref T8300, clicking in day view should create new event

Test Plan

Open day view, click in an empty slot, new event modal should open.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lpriestley retitled this revision from to Clicking in day view should create new event.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/calendar/controller/PhabricatorCalendarEventEditController.php
45–46

We might need an:

if (!$start->isValid()) {
  return new Aphront400Response();
}

...here.

For example, if you pass ?year=quack&month=platypus&day=orangejuice, I think $start->getEpoch() and $start->getDateTime() will return null, so we'll fatal a little later on.

This revision is now accepted and ready to land.May 24 2015, 5:42 PM
lpriestley marked an inline comment as done.
lpriestley edited edge metadata.

Preventing quackoclock start times

This revision was automatically updated to reflect the committed changes.