Tables: Invalid attributes in wikitext rows aren't roundtripped properly
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	ssastry
	Apr 5 2015, 9:38 PM

Description

[subbu@earth tests] cat /tmp/wt
{|
|- title="foo" | [[Foo]]
|}
[subbu@earth tests] node parse --wt2wt < /tmp/wt
{|
|- title="foo" |   [ Foo]]
|}

	Subject	Repo	Branch	Lines +/-
	Content on table start / row is all attributes	mediawiki/services/parsoid	master	+75 -42

Mentioned In: rGPAR65c72b14778a: Content on table start / row is all attributes
Mentioned Here: rGPAR4d1551535c5a: Handle broken_table_attribute_name_char in table_attributes

ssastry raised the priority of this task from to Low.

ssastry updated the task description. (Show Details)

ssastry subscribed.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 5 2015, 9:38 PM

[[Foo]]
{|
|- title="foo" | 
|}

which is arguably more correct.

We have a test for this called, "RT-ed inter-element separators should be valid separators".

Change 237182 had a related patch set uploaded (by Arlolra):
Content on table start / row is all attributes

T95131#1618958 was very wrong. Content on table start / row is all attributes. With that patch,

λ (T95131) cat t | node parse --wt2wt
{|
|- title="foo" | [[Foo]]
|}

Change 237182 merged by jenkins-bot:
Content on table start / row is all attributes

Arlolra closed this task as Resolved.Sep 10 2015, 7:47 PM

Arlolra claimed this task.