Paths

Table of Contentst

Diffusion Phorge 70c864914210

Use phutil_json_decode instead of json_decode
70c864914210
Actions

Description

Use phutil_json_decode instead of json_decode

Summary: Generally, phutil_json_decode should be preferred over json_decode.

Test Plan: Eyellballed.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D12680

Details

Provenance

Joshua Spence	Authored on May 5 2015, 3:20 AM
themackabu	Pushed on Mar 25 2025, 8:07 PM

Parents

rP05e745d9b81d: Minor documentation improvement

Branches

Unknown

Tags

Unknown

Event Timeline

Changes (34)

Path

Size

scripts/

celerity/

generate_sprites.php

src/

applications/

conduit/

controller/

PhabricatorConduitAPIController.php

ssh/

ConduitSSHWorkflow.php

console/

controller/

DarkConsoleDataController.php

differential/

__tests__/

DifferentialParseRenderTestCase.php

conduit/

DifferentialSetDiffPropertyConduitAPIMethod.php

diffusion/

DiffusionLintSaveRunner.php

controller/

DiffusionLastModifiedController.php

ssh/

__tests__/

DiffusionMercurialWireSSHTestCase.php

doorkeeper/

controller/

DoorkeeperTagsController.php

help/

controller/

PhabricatorHelpKeyboardShortcutController.php

herald/

adapter/

HeraldAdapter.php

controller/

HeraldRuleController.php

maniphest/

controller/

ManiphestBatchEditController.php

ManiphestTransactionPreviewController.php

metamta/

adapter/

PhabricatorMailImplementationMailgunAdapter.php

PhabricatorMailImplementationSendGridAdapter.php

contentsource/

PhabricatorContentSource.php

phame/

skins/

PhameSkinSpecification.php

phortune/

controller/

PhortunePaymentMethodCreateController.php

policy/

controller/

PhabricatorPolicyEditController.php

releeph/

differential/

DifferentialReleephRequestFieldSpecification.php

search/

engine/

PhabricatorElasticSearchEngine.php

xhprof/

controller/

PhabricatorXHProfProfileController.php

infrastructure/

customfield/

config/

PhabricatorCustomFieldConfigOptionType.php

standard/

PhabricatorStandardCustomFieldPHIDs.php

daemon/

bot/

PhabricatorBot.php

adapter/

PhabricatorBotBaseStreamingProtocolAdapter.php

PhabricatorBotFlowdockProtocolAdapter.php

PhabricatorCampfireProtocolAdapter.php

workers/

query/

PhabricatorWorkerLeaseQuery.php

edges/

query/

PhabricatorEdgeQuery.php

storage/

lisk/

LiskDAO.php

view/

phui/

PHUIIconView.php

rP70c864914210

View Options

scripts/celerity/generate_sprites.php

View Options

src/applications/conduit/controller/PhabricatorConduitAPIController.php

View Options

src/applications/conduit/ssh/ConduitSSHWorkflow.php

View Options

src/applications/console/controller/DarkConsoleDataController.php

View Options

src/applications/differential/tests/DifferentialParseRenderTestCase.php

View Options

src/applications/differential/conduit/DifferentialSetDiffPropertyConduitAPIMethod.php

View Options

src/applications/diffusion/DiffusionLintSaveRunner.php

View Options

src/applications/diffusion/controller/DiffusionLastModifiedController.php

View Options

src/applications/diffusion/ssh/tests/DiffusionMercurialWireSSHTestCase.php

View Options

src/applications/doorkeeper/controller/DoorkeeperTagsController.php

View Options

src/applications/help/controller/PhabricatorHelpKeyboardShortcutController.php

View Options

src/applications/herald/adapter/HeraldAdapter.php

View Options

src/applications/herald/controller/HeraldRuleController.php

View Options

src/applications/maniphest/controller/ManiphestBatchEditController.php

View Options

src/applications/maniphest/controller/ManiphestTransactionPreviewController.php

View Options

src/applications/metamta/adapter/PhabricatorMailImplementationMailgunAdapter.php

View Options

src/applications/metamta/adapter/PhabricatorMailImplementationSendGridAdapter.php

View Options

src/applications/metamta/contentsource/PhabricatorContentSource.php

View Options

src/applications/phame/skins/PhameSkinSpecification.php

View Options

src/applications/phortune/controller/PhortunePaymentMethodCreateController.php

View Options

src/applications/policy/controller/PhabricatorPolicyEditController.php

View Options

src/applications/releeph/differential/DifferentialReleephRequestFieldSpecification.php

View Options

src/applications/search/engine/PhabricatorElasticSearchEngine.php

View Options

src/applications/xhprof/controller/PhabricatorXHProfProfileController.php

View Options

src/infrastructure/customfield/config/PhabricatorCustomFieldConfigOptionType.php

View Options

src/infrastructure/customfield/standard/PhabricatorStandardCustomFieldPHIDs.php

View Options

src/infrastructure/daemon/bot/PhabricatorBot.php

View Options

src/infrastructure/daemon/bot/adapter/PhabricatorBotBaseStreamingProtocolAdapter.php

View Options

src/infrastructure/daemon/bot/adapter/PhabricatorBotFlowdockProtocolAdapter.php

View Options

src/infrastructure/daemon/bot/adapter/PhabricatorCampfireProtocolAdapter.php

View Options

src/infrastructure/daemon/workers/query/PhabricatorWorkerLeaseQuery.php

View Options

src/infrastructure/edges/query/PhabricatorEdgeQuery.php

View Options

src/infrastructure/storage/lisk/LiskDAO.php

View Options

src/view/phui/PHUIIconView.php

Use phutil_json_decode instead of json_decode70c864914210Actions

Description

Details

Event Timeline

Changes (34)

rP70c864914210

scripts/celerity/generate_sprites.php

src/applications/conduit/controller/PhabricatorConduitAPIController.php

src/applications/conduit/ssh/ConduitSSHWorkflow.php

src/applications/console/controller/DarkConsoleDataController.php

src/applications/differential/__tests__/DifferentialParseRenderTestCase.php

src/applications/differential/conduit/DifferentialSetDiffPropertyConduitAPIMethod.php

src/applications/diffusion/DiffusionLintSaveRunner.php

src/applications/diffusion/controller/DiffusionLastModifiedController.php

src/applications/diffusion/ssh/__tests__/DiffusionMercurialWireSSHTestCase.php

src/applications/doorkeeper/controller/DoorkeeperTagsController.php

src/applications/help/controller/PhabricatorHelpKeyboardShortcutController.php

src/applications/herald/adapter/HeraldAdapter.php

src/applications/herald/controller/HeraldRuleController.php

src/applications/maniphest/controller/ManiphestBatchEditController.php

src/applications/maniphest/controller/ManiphestTransactionPreviewController.php

src/applications/metamta/adapter/PhabricatorMailImplementationMailgunAdapter.php

src/applications/metamta/adapter/PhabricatorMailImplementationSendGridAdapter.php

src/applications/metamta/contentsource/PhabricatorContentSource.php

src/applications/phame/skins/PhameSkinSpecification.php

src/applications/phortune/controller/PhortunePaymentMethodCreateController.php

src/applications/policy/controller/PhabricatorPolicyEditController.php

src/applications/releeph/differential/DifferentialReleephRequestFieldSpecification.php

src/applications/search/engine/PhabricatorElasticSearchEngine.php

src/applications/xhprof/controller/PhabricatorXHProfProfileController.php

src/infrastructure/customfield/config/PhabricatorCustomFieldConfigOptionType.php

src/infrastructure/customfield/standard/PhabricatorStandardCustomFieldPHIDs.php

src/infrastructure/daemon/bot/PhabricatorBot.php

src/infrastructure/daemon/bot/adapter/PhabricatorBotBaseStreamingProtocolAdapter.php

src/infrastructure/daemon/bot/adapter/PhabricatorBotFlowdockProtocolAdapter.php

src/infrastructure/daemon/bot/adapter/PhabricatorCampfireProtocolAdapter.php

src/infrastructure/daemon/workers/query/PhabricatorWorkerLeaseQuery.php

src/infrastructure/edges/query/PhabricatorEdgeQuery.php

src/infrastructure/storage/lisk/LiskDAO.php

src/view/phui/PHUIIconView.php

Use phutil_json_decode instead of json_decode
70c864914210
Actions

src/applications/differential/tests/DifferentialParseRenderTestCase.php

src/applications/diffusion/ssh/tests/DiffusionMercurialWireSSHTestCase.php