Maniphest T208470

Parsoid should nowiki-escape '}' in a table cell or insert a whitespace character, as appropriate
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Psychoslave
	Oct 31 2018, 9:32 PM

Description

While a - is well embeded in a nowiki anchor when a table cell is created with - as only content, that is not the case when a cell is created with only }. The resulting code will be a |} at the beggining of the line wich will end the table.

Details

	Subject	Repo	Branch	Lines +/-
	html2wt: Handle } in table cells properly	mediawiki/services/parsoid	master	+36 -3

Customize query in gerrit

Related Objects

Mentioned In: T204477: Create punjabi.wikimedia.org for Punjabi Wikimedians User Group
T205546: Create Wiktionary Cantonese
T205710: Create Wikinews Limburgish
T206777: Create Wikipedia Shan
T207286: Time profiling: Replace millisecond granularity timers with microsecond granularity timers
T187142: Deduplicate template styles in Parsoid
T184755: Consider not removing multiple blank lines/white space between paragraphs
T210437: Sanitizer::stripAllTags shouldn't expand legacy "semicolon-less" HTML5 entities
T209236: "&params" URL parameter (used in a link parameter in [[File]] markup) incorrectly parsed as "¶ms" (%C2%B6ms)
Mentioned Here: rGPAR18a98afcd12c: Don't normalize mw-empty-elt empty paragraphs that come from source
T184755: Consider not removing multiple blank lines/white space between paragraphs
T187142: Deduplicate template styles in Parsoid
T204477: Create punjabi.wikimedia.org for Punjabi Wikimedians User Group
T205546: Create Wiktionary Cantonese
T205710: Create Wikinews Limburgish
T206777: Create Wikipedia Shan
T207286: Time profiling: Replace millisecond granularity timers with microsecond granularity timers
T209236: "&params" URL parameter (used in a link parameter in [[File]] markup) incorrectly parsed as "¶ms" (%C2%B6ms)
T210437: Sanitizer::stripAllTags shouldn't expand legacy "semicolon-less" HTML5 entities

Event Timeline

Psychoslave created this task.Oct 31 2018, 9:32 PM

Restricted Application added subscribers: jeblad, Aklapper. · View Herald TranscriptOct 31 2018, 9:32 PM

matmarex edited projects, added Parsoid; removed VisualEditor.Nov 1 2018, 1:20 AM

ssastry renamed this task from [VisualEditor:] Creating a cell table with a single `}` create misparsed table to Parsoid should nowiki-escape '}' in a table cell or insert a whitespace character, as appropriate.Nov 1 2018, 3:59 AM

ssastry triaged this task as Medium priority.

ssastry edited projects, added Parsoid-Serializer, Parsoid-Nowiki, Parsoid-Edit-Support; removed Parsoid.

Change 472379 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] html2wt: Handle } in table cells properly

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

gerritbot added a project: Patch-For-Review.Nov 8 2018, 5:59 AM

ssastry claimed this task.Nov 8 2018, 6:00 AM

Change 472379 merged by jenkins-bot:
[mediawiki/services/parsoid@master] html2wt: Handle } in table cells properly

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

ssastry closed this task as Resolved.Nov 8 2018, 5:16 PM