Page MenuHomePhabricator

Consolidate JSON formatting
ClosedPublic

Authored by joshuaspence on Dec 30 2014, 1:32 PM.
Tags
None
Referenced Files
F17953510: D11085.id26630.diff
Fri, Aug 1, 3:09 AM
F17941815: D11085.diff
Thu, Jul 31, 5:55 AM
F17870609: D11085.id.diff
Jul 28 2025, 2:25 PM
Unknown Object (File)
Jun 18 2025, 1:49 PM
Unknown Object (File)
Jun 16 2025, 2:11 AM
Unknown Object (File)
May 29 2025, 6:39 AM
Unknown Object (File)
May 21 2025, 3:27 AM
Unknown Object (File)
May 9 2025, 1:14 PM
Subscribers

Details

Summary

In some places {"foo" : "bar"} is used, whilst in other places {"foo": "bar"} is used, This diff attempts to consolidate these differences and adopt a more standard JSON format.

Test Plan
> print_r(id(new PhutilJSON())->encodeFormatted(array('foo' => 'bar')));
{
  "foo": "bar"
}

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence retitled this revision from to Consolidate JSON formatting.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Dec 30 2014, 2:28 PM
This revision was automatically updated to reflect the committed changes.