Fix `PhutilReadableSerializer` to quote strings.
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F18829325: D9632.id.diff
	Fri, Oct 24, 9:09 PM

	F18824680: D9632.diff
	Thu, Oct 23, 5:20 PM

	F18741132: D9632.id23101.diff
	Oct 2 2025, 8:45 AM

	F18730242: D9632.diff
	Sep 30 2025, 12:33 PM

	F18703256: D9632.id.diff
	Sep 28 2025, 3:26 AM

	F18628903: D9632.id23100.diff
	Sep 16 2025, 7:32 AM

	F18616488: D9632.diff
	Sep 14 2025, 5:45 PM

	F18557883: D9632.id.diff
	Sep 8 2025, 8:10 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