Page MenuHomePhabricator

Introduce idxv(), like idx()
ClosedPublic

Authored by epriestley on Feb 25 2016, 4:39 PM.
Tags
None
Referenced Files
F15454831: D15347.id37015.diff
Sat, Mar 29, 8:40 PM
F15450469: D15347.id.diff
Fri, Mar 28, 3:50 PM
F15448870: D15347.id37008.diff
Fri, Mar 28, 7:14 AM
F15447638: D15347.diff
Fri, Mar 28, 12:43 AM
F15444596: D15347.id37008.diff
Thu, Mar 27, 10:16 AM
F15427744: D15347.id37008.diff
Sun, Mar 23, 3:53 PM
F15412018: D15347.id37008.diff
Wed, Mar 19, 10:53 AM
F15398285: D15347.id37015.diff
Sun, Mar 16, 11:58 PM
Subscribers
None
Tokens
"Mountain of Wealth" token, awarded by yelirekim.

Details

Summary

Ref T10449. The new Conduit APIs are more powerful/flexible, but also more annoying to read data from since things are buried several levels deep.

Introduce idxv() to make it easier to do $conduit_result['data']['attachments']['fields']['name'] and similar safely without checking every single value.

Test Plan

See other diffs associated with T10449.

Diff Detail

Repository
rPHU libphutil
Branch
idxv
Lint
Lint Errors
SeverityLocationCodeMessage
Errorsrc/utils/utils.php:1220XHP45PHP Compatibility
Errorsrc/utils/utils.php:1220XHP45PHP Compatibility
Unit
Tests Passed
Build Status
Buildable 10847
Build 13373: Run Core Tests
Build 13372: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Introduce idxv(), like idx().
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Feb 25 2016, 4:58 PM
This revision was automatically updated to reflect the committed changes.