Page MenuHomePhabricator

Conpherence - fix a bug where users have no events
ClosedPublic

Authored by btrahan on Jun 16 2015, 9:35 PM.
Tags
None
Referenced Files
F15422558: D13314.diff
Sat, Mar 22, 9:21 AM
F15418704: D13314.id32224.diff
Fri, Mar 21, 12:58 AM
F15411870: D13314.diff
Wed, Mar 19, 10:05 AM
F15408418: D13314.id.diff
Tue, Mar 18, 10:54 PM
F15402226: D13314.diff
Mon, Mar 17, 9:27 PM
F15396319: D13314.id.diff
Sun, Mar 16, 1:28 PM
Unknown Object (File)
Feb 12 2025, 10:19 AM
Unknown Object (File)
Feb 8 2025, 11:13 AM
Subscribers

Details

Summary

Fixes T8518. Defaulting $events to null creates carnage further down the stack. Instead, default it to array() so foreach, array_select_keys, etc. don't barf.

Test Plan

logic and will ask the reporting user to verify since they can repro 100%

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Conpherence - fix a bug where users have no events.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 16 2015, 9:36 PM
This revision was automatically updated to reflect the committed changes.