Fix `PhutilReadableSerializer` to quote strings.
ClosedPublic
Actions

Authored by joshuaspence on Jun 19 2014, 8:35 PM.

Tags

None

Referenced Files

	F14084033: D9632.diff
	Sat, Nov 23, 6:37 AM

	Unknown Object (File)
	Thu, Nov 21, 9:49 PM

	Unknown Object (File)
	Tue, Nov 19, 2:53 AM

	Unknown Object (File)
	Thu, Nov 14, 5:57 PM

	Unknown Object (File)
	Mon, Nov 11, 4:43 AM

	Unknown Object (File)
	Fri, Nov 8, 6:19 PM

	Unknown Object (File)
	Fri, Nov 8, 6:14 PM

	Unknown Object (File)
	Fri, Nov 8, 2:54 PM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rPHUd8935319d556: Fix `PhutilReadableSerializer` to quote strings.

Summary

See D9622#inline-37279 for explanation.

Test Plan

Modified and ran unit tests.

Diff Detail

Repository

rPHU libphutil

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

joshuaspence updated this revision to Diff 23100.Jun 19 2014, 8:35 PM

joshuaspence retitled this revision from to Fix `PhutilReadableSerializer` to quote strings..

joshuaspence updated this object.

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

joshuaspence added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 19 2014, 8:35 PM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

Harbormaster completed remote builds in B1245: Diff 23100.Jun 19 2014, 8:36 PM

epriestley accepted this revision.Jun 19 2014, 8:44 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jun 19 2014, 8:44 PM

Closed by commit rPHUd8935319d556 (authored by @joshuaspence).

Revision Contents
Changeset List

Path

Size

src/

readableserializer/

PhutilReadableSerializer.php

9 lines

__tests__/

PhutilReadableSerializerTestCase.php

2 lines

Diff 23101

View Options

src/readableserializer/PhutilReadableSerializer.php

View Options

src/readableserializer/tests/PhutilReadableSerializerTestCase.php

Fix `PhutilReadableSerializer` to quote strings.ClosedPublicActions