Differential D21171

Skip "null" lines when constructing raw documents for DocumentEngine rendering
ClosedPublic
Actions

Authored by epriestley on Apr 26 2020, 12:30 AM.

Tags

None

Referenced Files

	F19505412: D21171.diff
	Fri, Jan 9, 11:28 AM

	F19010999: D21171.diff
	Nov 22 2025, 6:59 AM

	F18798821: D21171.id50411.diff
	Oct 17 2025, 9:00 AM

	F18775637: D21171.diff
	Oct 10 2025, 7:40 PM

	F18759507: D21171.id50411.diff
	Oct 6 2025, 4:17 AM

	F18623883: D21171.id.diff
	Sep 15 2025, 5:57 PM

	F18623643: D21171.id.diff
	Sep 15 2025, 4:42 PM

	F18611787: D21171.diff
	Sep 14 2025, 8:00 AM

Subscribers

None

Details

Reviewers: None
Commits: rPa1c978ef378a: (stable) Skip "null" lines when constructing raw documents for DocumentEngine…
rP7355bb7f2956: Skip "null" lines when constructing raw documents for DocumentEngine rendering

Summary

See https://discourse.phabricator-community.org/t/more-exceptions-when-viewing-diffs/3789. This is a similar issue with the same datastructure.

Test Plan

Works correctly under PHP 7.3, but this may not be the end of things.

Diff Detail

Repository

Branch

diff2

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 24229
Build 33371: Run Core Tests
Build 33370: arc lint + arc unit

Event Timeline

epriestley created this revision.Apr 26 2020, 12:30 AM

Harbormaster completed remote builds in B24229: Diff 50410.Apr 26 2020, 12:31 AM

epriestley requested review of this revision.Apr 26 2020, 12:31 AM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 26 2020, 12:34 AM

Closed by commit rP7355bb7f2956: Skip "null" lines when constructing raw documents for DocumentEngine rendering (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP7355bb7f2956: Skip "null" lines when constructing raw documents for DocumentEngine rendering.

epriestley added a commit: rPa1c978ef378a: (stable) Skip "null" lines when constructing raw documents for DocumentEngine….

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

parser/

DifferentialChangesetParser.php

6 lines

Diff 50410

src/applications/differential/parser/DifferentialChangesetParser.php

Loading...