Page MenuHomePhabricator

Treat a line with WS and comments only as being transparent to all handlers
Closed, ResolvedPublic

Description

Bug 41756 fixed an old regression in the PHP preprocessor, so that a blank line containing nothing but two or more comments is ignored (does not break lists, etc).

This fix needs to be applied to Parsoid as well.


Version: unspecified
Severity: normal

Details

Reference
bz52762

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 2:08 AM
bzimport added a project: Parsoid.
bzimport set Reference to bz52762.

I am fixing the p-handler for now since it already had special-case code for the older pattern. But, a fix for 49974 would be more ideal that is more general.

Repurposing this bug for the general pattern. Will close 49974 as a dupe of this.

  • Bug 49974 has been marked as a duplicate of this bug. ***

Change 78907 had a related patch set uploaded by Subramanya Sastry:
P-handler specific fix of bug 52762

https://gerrit.wikimedia.org/r/78907

Change 78907 merged by jenkins-bot:
P-handler specific fix of bug 52762

https://gerrit.wikimedia.org/r/78907

Change 81014 had a related patch set uploaded by Subramanya Sastry:
(Bug 52762) Lines with 1+ comments and WS are transparent to handlers

https://gerrit.wikimedia.org/r/81014

Change 81014 merged by jenkins-bot:
(Bug 52762) Lines with 1+ comments and WS are transparent to handlers

https://gerrit.wikimedia.org/r/81014