Page MenuHomePhabricator

Template-generated partial table cell markup incorrectly parsed (new - different - case)
Closed, ResolvedPublic

Description

Table CSS parameters are wrong parsed under certain circumstances. For example:
http://parsoid-lb.eqiad.wikimedia.org/idwiki/Nickelodeon_Kids%27_Choice_Awards?oldid=8237629

Comment about this from Subbu:
Kelson: your example is a bit different from the nlwiki scenario. Looks like we have an unhandled edge case. In the example below, the first row is handled, but the second isn't. Should be easy to fix.

{|
|-
!nowrap|Favorite Movie
| {{yes}}
|<!-- 1989 --> {{yes}}
|}

This bug is "similar" like #67850 bug and what reported their first:
https://bugzilla.wikimedia.org/show_bug.cgi?id=67850#c12

But the rootcause seems to be different (see following comment):
https://bugzilla.wikimedia.org/show_bug.cgi?id=67850#c13


Version: unspecified
Severity: normal

Details

Reference
bz72487

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 3:48 AM
bzimport added a project: Parsoid.
bzimport set Reference to bz72487.

gerritadmin wrote:

Change 168664 had a related patch set uploaded by Subramanya Sastry:
(Bug 72487) More robust handling of <td> split across page & template

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

Kelson, that fixes the page, but it also flagged a broken comment in one of the cells on the page .. php parser ignores it, but parsoid doesn't .. that is a wontfix on our end. you can edit the wikitext for that one cell and fix the comment.

gerritadmin wrote:

Change 168664 merged by jenkins-bot:
(Bug 72487) More robust handling of <td> split across page & template

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

ssastry set Security to None.