Page MenuHomePhabricator

Using ] (closing square bracket) in title tag for external link breaks expected parsing
Closed, ResolvedPublic

Description

When I use 'span title' in external link, MediaWiki are recognized MediaWiki syntax instead of the brackets and showed strange if '&#91; and &#93;' are included. e.g. [http://zoo.com/wiki/Talk:Main page <span title="Main Page talk &#91;alt-shift-t&#93;">Discussions about Main Page</span>]

[http://zoo.com/wiki/Talk:Main page <span title="Main Page talk &#91;alt-shift-t&#93;">Discussions about Main Page</span>]

is parsed as

<div class="mw-parser-output"><p><a rel="nofollow" class="external text" href="http://zoo.com/wiki/Talk:Main">page <span title="Main Page talk &#91;alt-shift-t</a>">Discussions about Main Page</span>]
</p></div>
[http://zoo.com/wiki/Talk:Main page <span title="Main Page talk [alt-shift-t]">Discussions about Main Page</span>]

has the same result.

Event Timeline

bzimport raised the priority of this task from to Lowest.Nov 22 2014, 1:56 AM
bzimport added a project: MediaWiki-Parser.
bzimport set Reference to bz49672.
bzimport added a subscriber: Unknown Object (MLST).

https://gerrit.wikimedia.org/r/391135 solves this problem. The wikitext is rendered then as

<div class="mw-parser-output"><p><a rel="nofollow" class="external text" href="http://zoo.com/wiki/Talk:Main">page <span title="Main Page talk &#91;alt-shift-t&#93;">Discussions about Main Page</span></a>
</p></div>

Change 391135 had a related patch set uploaded (by Fomafix; owner: MaxSem):
[mediawiki/core@master] Sanitizer::safeEncodeAttribute(): also encode ]

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

Change 391135 merged by jenkins-bot:
[mediawiki/core@master] Sanitizer::safeEncodeAttribute(): also encode ]

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

Fomafix assigned this task to MaxSem.
Fomafix raised the priority of this task from Lowest to Low.
Fomafix removed a project: Patch-For-Review.