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
F13048610: D12990.diff
Thu, Apr 18, 9:39 PM
Unknown Object (File)
Wed, Apr 17, 12:42 PM
Unknown Object (File)
Sun, Apr 7, 4:19 PM
Unknown Object (File)
Sun, Mar 31, 4:16 AM
Unknown Object (File)
Sun, Mar 31, 2:49 AM
Unknown Object (File)
Sun, Mar 31, 2:49 AM
Unknown Object (File)
Feb 17 2024, 7:47 AM
Unknown Object (File)
Feb 6 2024, 2:36 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
Branch
calendarneweventjs
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6235
Build 6257: [Placeholder Plan] Wait for 30 Seconds

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.