Tags

None

Referenced Files

	F18944994: D9633.id.diff
	Tue, Nov 11, 5:28 PM

	F18839411: D9633.diff
	Mon, Oct 27, 4:10 PM

	F18829696: D9633.id.diff
	Sat, Oct 25, 12:00 AM

	F18822510: D9633.diff
	Thu, Oct 23, 7:09 AM

	F18757665: D9633.id23112.diff
	Oct 5 2025, 6:42 PM

	F18653951: D9633.diff
	Sep 21 2025, 10:45 PM

	F18651445: D9633.diff
	Sep 21 2025, 4:23 AM

	F18630881: D9633.id.diff
	Sep 16 2025, 1:31 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

Lint

Lint Skipped

Unit

Tests Skipped

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

Closed by commit rPHU3bd9044d42bb (authored by @joshuaspence).

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

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 23112

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

src/parser/tests/PhutilJSONParserTestCase.php

Fix the handling of empty keys in `PhutilJSONParser`.ClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 23112

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

src/parser/__tests__/PhutilJSONParserTestCase.php

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

Revision Contents
Changeset List

src/parser/tests/PhutilJSONParserTestCase.php