Page MenuHomePhabricator
Differential D12680

Use phutil_json_decode instead of json_decode
ClosedPublic
Actions

Authored by joshuaspence on May 3 2015, 9:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 13, 8:19 PM
Unknown Object (File)
Fri, Dec 13, 8:42 AM
Unknown Object (File)
Wed, Dec 4, 7:14 PM
Unknown Object (File)
Thu, Nov 28, 3:07 PM
Unknown Object (File)
Nov 19 2024, 12:04 PM
Unknown Object (File)
Nov 7 2024, 1:10 PM
Unknown Object (File)
Oct 23 2024, 6:15 AM
Unknown Object (File)
Oct 14 2024, 4:57 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP70c864914210: Use phutil_json_decode instead of json_decode
Summary

Generally, phutil_json_decode should be preferred over json_decode.

Test Plan

Eyellballed.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5706
Build 5725: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence updated this revision to Diff 30445.May 3 2015, 9:15 PM
joshuaspence retitled this revision from to Use phutil_json_decode instead of json_decode.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 3 2015, 9:15 PM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
epriestley accepted this revision.May 3 2015, 9:21 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 3 2015, 9:21 PM
joshuaspence updated this revision to Diff 30510.May 4 2015, 9:11 PM
joshuaspence edited edge metadata.

Found some more

Closed by commit rP70c864914210: Use phutil_json_decode instead of json_decode (authored by joshuaspence, committed by joshuaspence). · Explain WhyMay 5 2015, 11:07 AM
This revision was automatically updated to reflect the committed changes.
joshuaspence mentioned this in D12714: Fix a lisk issue.May 5 2015, 12:02 PM
joshuaspence mentioned this in rP8d128646b31a: Fix a lisk issue.May 5 2015, 12:11 PM
epriestley mentioned this in rP31a89bb94d2a: Revert a json_decode() which decodes possible scalars.May 5 2015, 6:08 PM
epriestley mentioned this in rP5f7ee505b6b7: Revert a third json_decode() which decodes possible scalars.May 5 2015, 8:03 PM
epriestley mentioned this in rP336ccfeea12e: Revert a fourth json_decode() which decodes possible scalars.May 5 2015, 8:13 PM

Revision Contents
Changeset List

PathSize
src/
applications/
conduit/
controller/
15 lines
ssh/
8 lines
console/
controller/
6 lines
differential/
__tests__/
9 lines
diffusion/
8 lines
help/
controller/
5 lines
herald/
adapter/
16 lines
metamta/
adapter/
10 lines
phame/
skins/
13 lines
phortune/
controller/
6 lines
policy/
controller/
9 lines
search/
engine/
11 lines
xhprof/
controller/
9 lines
infrastructure/
daemon/
bot/
9 lines

Diff 30445

View Options

src/applications/conduit/controller/PhabricatorConduitAPIController.php

Loading...
View Options

src/applications/conduit/ssh/ConduitSSHWorkflow.php

Loading...
View Options

src/applications/console/controller/DarkConsoleDataController.php

Loading...
View Options

src/applications/differential/__tests__/DifferentialParseRenderTestCase.php

Loading...
View Options

src/applications/diffusion/DiffusionLintSaveRunner.php

Loading...
View Options

src/applications/help/controller/PhabricatorHelpKeyboardShortcutController.php

Loading...
View Options

src/applications/herald/adapter/HeraldAdapter.php

Loading...
View Options

src/applications/metamta/adapter/PhabricatorMailImplementationMailgunAdapter.php

Loading...
View Options

src/applications/phame/skins/PhameSkinSpecification.php

Loading...
View Options

src/applications/phortune/controller/PhortunePaymentMethodCreateController.php

Loading...
View Options

src/applications/policy/controller/PhabricatorPolicyEditController.php

Loading...
View Options

src/applications/search/engine/PhabricatorSearchEngineElastic.php

Loading...
View Options

src/applications/xhprof/controller/PhabricatorXHProfProfileController.php

Loading...
View Options

src/infrastructure/daemon/bot/PhabricatorBot.php

Loading...
Phabricator · Built by Phacility