Page MenuHomePhabricator
Differential D9640

Update callsites of `phutil_json_decode`.
ClosedPublic
Actions

Authored by joshuaspence on Jun 20 2014, 8:21 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 17, 7:06 AM
Unknown Object (File)
Sun, Dec 15, 5:38 AM
Unknown Object (File)
Sat, Dec 7, 11:20 AM
Unknown Object (File)
Thu, Nov 28, 9:00 PM
Unknown Object (File)
Nov 16 2024, 8:01 PM
Unknown Object (File)
Oct 24 2024, 12:01 PM
Unknown Object (File)
Oct 24 2024, 9:47 AM
Unknown Object (File)
Oct 21 2024, 1:51 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rPa4a8cfa6d632: Update callsites of `phutil_json_decode`.
Summary

Depends on D9634. phutil_json_decode now throws an exception on invalid JSON.

Test Plan

arc unit

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence updated this revision to Diff 23129.Jun 20 2014, 8:21 AM
joshuaspence retitled this revision from to Update callsites of `phutil_json_decode`..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
joshuaspence added a parent revision: D9634: Modify `phutil_json_decode` to use `PhutilJSONParser`..
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 20 2014, 8:21 AM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
Harbormaster completed remote builds in B1263: Diff 23129.Jun 20 2014, 8:22 AM
epriestley accepted this revision.Jun 20 2014, 12:37 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 20 2014, 12:37 PM
joshuaspence closed this revision.Jun 20 2014, 2:39 PM
joshuaspence updated this revision to Diff 23136.

Closed by commit rPa4a8cfa6d632 (authored by @joshuaspence).

Revision Contents
Changeset List

PathSize
src/
applications/
config/
custom/
17 lines
dashboard/
paneltype/
5 lines
differential/
customfield/
6 lines
6 lines
parser/
__tests__/
13 lines

Diff 23136

View Options

src/applications/config/custom/PhabricatorConfigJSONOptionType.php

Loading...
View Options

src/applications/dashboard/paneltype/PhabricatorDashboardPanelTypeTabs.php

Loading...
View Options

src/applications/differential/customfield/DifferentialAuditorsField.php

Loading...
View Options

src/applications/differential/customfield/DifferentialJIRAIssuesField.php

Loading...
View Options

src/applications/differential/parser/__tests__/DifferentialCommitMessageParserTestCase.php

Loading...
Phabricator · Built by Phacility