Page MenuHomePhabricator

Consolidate JSON formatting
ClosedPublic

Authored by joshuaspence on Dec 30 2014, 1:32 PM.
Tags
None
Referenced Files
F13980378: D11085.diff
Sat, Oct 19, 10:17 AM
F13963917: D11085.id26608.diff
Tue, Oct 15, 6:38 PM
Unknown Object (File)
Sep 20 2024, 2:50 PM
Unknown Object (File)
Aug 26 2024, 12:08 AM
Unknown Object (File)
Aug 25 2024, 10:37 PM
Unknown Object (File)
Aug 19 2024, 1:03 PM
Unknown Object (File)
Aug 12 2024, 4:47 PM
Unknown Object (File)
Aug 10 2024, 10:38 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
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 3386
Build 3393: [Placeholder Plan] Wait for 30 Seconds

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.