Page MenuHomePhabricator

class="wikitable wikitable" corrupted to class="wikitable"
Closed, ResolvedPublic1 Estimated Story Points

Description

Results in bugs like https://de.wikipedia.org/?diff=138315500 . This triggers the corruption warning in VE, and appears to be a bug in ve.dm.ClassAttributeNode

Event Timeline

Catrope raised the priority of this task from to Needs Triage.
Catrope updated the task description. (Show Details)
Catrope added a subscriber: Catrope.
Jdforrester-WMF set Security to None.

What's with the <nowiki></nowiki>s? I'm less concerned about the extra wikitable class going away.

@MZMcBride the <nowiki> are required if you want the contents of the table cell to be -. Note that there are existing cells with <nowiki> of the same sort.

Yeah, I saw that there were other similar cells. I misread <nowiki>-</nowiki> as <nowiki></nowiki>. My bad.

From quick testing, it seems !- creates a <th> cell containing a "-" and | - creates a <td> cell containing a "-". The latter seems more obvious to me and less clunky than wrapping nowiki tags around "-", but shrug.

I agree that serializing as "| -" might be more 'natural'. It's possible we could hack that into Parsoid as a special case. That's a separate bug/feature request though.

gerritbot added a subscriber: gerritbot.

Change 191037 had a related patch set uploaded (by Esanders):
Test for class attribute duplication bug

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

Patch-For-Review

Change 191028 had a related patch set uploaded (by Catrope):
Ignore duplicates when comparing class lists

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

Patch-For-Review

Change 191028 merged by jenkins-bot:
Ignore duplicates when comparing class lists

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

I can't understand why a duplicate class name is acceptable. Of course you must ignore these duplicates but the editor should also simply discard them

Change 191037 merged by jenkins-bot:
Test for class attribute duplication bug

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