Paths

Table of Contentst

Differential D13787

Add encodeAsList to PhutilJSON and make formatting prettier
ClosedPublic
Actions

Authored by epriestley on Aug 4 2015, 7:28 PM.

Tags

None

Referenced Files

	F13091432: D13787.diff
	Thu, Apr 25, 2:56 AM

	Unknown Object (File)
	Wed, Apr 17, 3:22 PM

	Unknown Object (File)
	Tue, Apr 9, 2:35 AM

	Unknown Object (File)
	Sat, Apr 6, 11:39 PM

	Unknown Object (File)
	Fri, Apr 5, 10:03 PM

	Unknown Object (File)
	Mar 26 2024, 2:59 AM

	Unknown Object (File)
	Mar 6 2024, 4:11 AM

	Unknown Object (File)
	Mar 1 2024, 4:44 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T7419: Provide a straightforward way to create and update CI test statuses for a revision
T8973: Encoding JSON for presentation could use more human-friendly escaping rules

Commits

rPHUb8420e193467: Add encodeAsList to PhutilJSON and make formatting prettier

Summary

Ref T8973. Ref T7419. Two issues:

In documenting T7419, I want to pretty print a JSON list value (an example value for lint and unit when calling harbormaster.sendmessage). Expose a public method for this.
T8973 discusses PHP's default behavior of escaping forward slashes in JSON. This is legal, but cumbersome. When pretty-printing for human users, disable it if we have a recent-enough PHP version.

Test Plan

See next diffs.
Pretty-printed lists with values that contain slashes, got friendly/helpful results:

Screen Shot 2015-08-04 at 12.27.12 PM.png (337×622 px, 38 KB)

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 33301.Aug 4 2015, 7:28 PM

epriestley retitled this revision from to Add encodeAsList to PhutilJSON and make formatting prettier.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: chad.

epriestley added tasks: T8973: Encoding JSON for presentation could use more human-friendly escaping rules, T7419: Provide a straightforward way to create and update CI test statuses for a revision.

chad accepted this revision.Aug 4 2015, 7:39 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Aug 4 2015, 7:39 PM

Closed by commit rPHUb8420e193467: Add encodeAsList to PhutilJSON and make formatting prettier (authored by epriestley, committed by epriestley). · Explain WhyAug 4 2015, 8:05 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

parser/

21 lines

Diff 33306

src/parser/PhutilJSON.php

Loading...