More correct fix - set default value array() for `getParameter` call.
ClosedPublic
Actions

Authored by lpriestley on Jun 17 2015, 3:29 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T8577: InvalidArgumentException when viewing upcoming calendar events

Commits

Restricted Diffusion Commit
rP9109cf62e464: More correct fix - set default value array() for `getParameter` call.

Summary

Ref T8577, More correct fix - set default value array() for getParameter call.

Test Plan

Make sure saved queries like Month View, Day View, and Upcoming Events, still work.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 32254.Jun 17 2015, 3:29 PM

lpriestley retitled this revision from to More correct fix - set default value array() for `getParameter` call..

lpriestley updated this object.

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

lpriestley added a reviewer: epriestley.

lpriestley added a task: T8577: InvalidArgumentException when viewing upcoming calendar events.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 17 2015, 3:29 PM

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

epriestley accepted this revision.Jun 18 2015, 12:32 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jun 18 2015, 12:32 PM

Closed by commit rP9109cf62e464: More correct fix - set default value array() for `getParameter` call. (authored by lpriestley, committed by lpriestley). · Explain WhyJun 18 2015, 4:06 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

calendar/

query/

PhabricatorCalendarEventSearchEngine.php

8 lines

Diff 32282

View Options

src/applications/calendar/query/PhabricatorCalendarEventSearchEngine.php

More correct fix - set default value array() for `getParameter` call.ClosedPublicActions