Page MenuHomePhabricator
Differential D9633

Fix the handling of empty keys in `PhutilJSONParser`.
ClosedPublic
Actions

Authored by joshuaspence on Jun 20 2014, 12:21 AM.
Tags
None
Referenced Files
F19133741: D9633.diff
Dec 10 2025, 7:31 AM
F19017052: D9633.diff
Nov 23 2025, 6:26 AM
F18994428: D9633.diff
Nov 19 2025, 11:07 AM
F18944994: D9633.id.diff
Nov 11 2025, 5:28 PM
F18839411: D9633.diff
Oct 27 2025, 4:10 PM
F18829696: D9633.id.diff
Oct 25 2025, 12:00 AM
F18822510: D9633.diff
Oct 23 2025, 7:09 AM
F18757665: D9633.id23112.diff
Oct 5 2025, 6:42 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rPHU3bd9044d42bb: Fix the handling of empty keys in `PhutilJSONParser`.
Summary

Because the empty string is not a valid property of stdClass, the original JsonLint library replaced empty keys with '_empty_'. After D9623, all instances of stdClass have been replaced by arrays, so we can remove this nonsense.

Depends on D9623.

Test Plan

Added a test case and ran arc unit.

Diff Detail

Repository
rPHU libphutil
Branch
json2.5
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1251
Build 1251: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence updated this revision to Diff 23110.Jun 20 2014, 12:21 AM
joshuaspence retitled this revision from to Fix the handling of empty keys in `PhutilJSONParser`..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
joshuaspence added a parent revision: D9623: Modify the JsonLint library to return `array` instead of `stdClass`..
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 20 2014, 12:21 AM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
Harbormaster completed remote builds in B1251: Diff 23110.Jun 20 2014, 12:22 AM
epriestley accepted this revision.Jun 20 2014, 12:26 AM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 20 2014, 12:26 AM
joshuaspence closed this revision.Jun 20 2014, 12:27 AM
joshuaspence updated this revision to Diff 23112.

Closed by commit rPHU3bd9044d42bb (authored by @joshuaspence).

Revision Contents
Changeset List

PathSize
externals/
jsonlint/
src/
Seld/
JsonLint/
4 lines
src/
parser/
__tests__/
1 line

Diff 23110

View Options

externals/jsonlint/src/Seld/JsonLint/JsonParser.php

Loading...
View Options

src/parser/__tests__/PhutilJSONParserTestCase.php

Loading...
Phabricator · Built by Phacility