Page MenuHomeDevCentral

Better Phabricator API requests handling
ClosedPublic

Authored by dereckson on Dec 25 2015, 19:45.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 11, 10:39
Unknown Object (File)
Thu, Nov 27, 05:50
Unknown Object (File)
Mon, Nov 17, 12:59
Unknown Object (File)
Mon, Nov 17, 10:51
Unknown Object (File)
Nov 12 2025, 10:46
Unknown Object (File)
Nov 11 2025, 20:13
Unknown Object (File)
Nov 10 2025, 23:13
Unknown Object (File)
Nov 10 2025, 21:11
Subscribers
None

Details

Reviewers
dereckson
Commits
Unknown Object (Diffusion Commit)
rNOTIF06d52a204908: Better Phabricator API requests handling
Summary

We don't try to guess if the API reply will be an object with the
PHID as key, or an array with 0 as offset for our reply, or an object
with 0 as property.

Instead, we call PhabricatorAPI::getFirstResult() method.

Test Plan

With this change, the development server fired notifications
for pending requests.

19:42:09 < Wearg> dereckson accepted D117: PHPMD: composer phpmd.

Diff Detail

Repository
rNOTIF Notifications center
Lint
No Lint Coverage
Unit
No Test Coverage
Branch
save-payload
Build Status
Buildable 145
Build 145: arc lint + arc unit

Event Timeline

dereckson retitled this revision from to Better Phabricator API requests handling.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
dereckson edited edge metadata.
This revision is now accepted and ready to land.Dec 25 2015, 19:47
This revision was automatically updated to reflect the committed changes.