Paths

Table of Contentst

Differential D12678

Use phutil_json_decode instead of json_decode
ClosedPublic
Actions

Authored by joshuaspence on May 3 2015, 9:14 PM.

Tags

None

Referenced Files

	F15475505: D12678.diff
	Sun, Apr 6, 8:02 PM

	F15461251: D12678.id.diff
	Tue, Apr 1, 6:39 AM

	F15444670: D12678.id30469.diff
	Thu, Mar 27, 10:40 AM

	F15439273: D12678.id.diff
	Wed, Mar 26, 6:42 AM

	F15439202: D12678.id30443.diff
	Wed, Mar 26, 6:20 AM

	F15435571: D12678.diff
	Tue, Mar 25, 8:57 AM

	F15433643: D12678.id30443.diff
	Tue, Mar 25, 12:13 AM

	F15424236: D12678.id30469.diff
	Sat, Mar 22, 9:03 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Commits

rARCa4d33ef117aa: Use phutil_json_decode instead of json_decode

Summary

Generally, phutil_json_decode should be preferred over json_decode.

Test Plan

Eyeballed.

Diff Detail

Repository

Branch

master

Lint

Lint Passed

Unit

Test Failures

Build Status

Buildable 5704
Build 5723: [Placeholder Plan] Wait for 30 Seconds

Unit TestsFailed
View All

	Time	Test
	855 ms	ArcanistJSHintLinterTestCase::testLinter
	1,721 ms	ArcanistPuppetLintLinterTestCase::testLinter
	278 ms	ArcanistXMLLinterTestCase::testLinter
	1,941 ms	ArcanistCSSLintLinterTestCase::testLinter
	410 ms	ArcanistCSSLintLinterTestCase::testVersion
		View Full Test Results (3 Failed · 59 Passed · 5 Skipped)

Event Timeline

joshuaspence updated this revision to Diff 30443.May 3 2015, 9:14 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:14 PM

Herald added subscribers: epriestley, Korvin, aurelijus. · View Herald Transcript

epriestley accepted this revision.May 3 2015, 9:14 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.May 3 2015, 9:14 PM

Closed by commit rARCa4d33ef117aa: Use phutil_json_decode instead of json_decode (authored by joshuaspence, committed by joshuaspence). · Explain WhyMay 4 2015, 12:36 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

lint/

linter/

ArcanistCSharpLinter.php

2 lines

ArcanistJSHintLinter.php

14 lines

unit/

parser/

ArcanistPhpunitTestResultParser.php

5 lines

workflow/

ArcanistCallConduitWorkflow.php

5 lines

Diff 30443

src/lint/linter/ArcanistCSharpLinter.php

Loading...

src/lint/linter/ArcanistJSHintLinter.php

Loading...

src/unit/parser/ArcanistPhpunitTestResultParser.php

Loading...

src/workflow/ArcanistCallConduitWorkflow.php

Loading...