Page MenuHomePhabricator

External links syntax is ended by quotes
Open, LowestPublic

Description

Now quotes stop external links:

[http://test.wikipedia.org/w/api.php?action=opensearch&search="Foo+Bar" Foo Bar]
http://test.wikipedia.org/w/api.php?action=opensearch&search="Foo+Bar"

Produce:

<a href="http://test.wikipedia.org/w/api.php?action=opensearch&search=">"Foo+Bar" Foo Bar</a>
<a href="http://test.wikipedia.org/w/api.php?action=opensearch&search=">http://test.wikipedia.org/w/api.php?action=opensearch&search=</a>"Foo+Bar"

If you try to escape them with &quot; you don't get " or &quot; at the url, but \" (urlescaped)

[http://test.wikipedia.org/w/api.php?action=opensearch&search=&quot;Foo+Bar&quot; Foo Bar]
http://test.wikipedia.org/w/api.php?action=opensearch&search=&quot;Foo+Bar&quot;

Produce:

<a href="http://test.wikipedia.org/w/api.php?action=opensearch&amp;search=%5C%22Foo+Bar%5C%22">Foo Bar</a>
<a href="http://test.wikipedia.org/w/api.php?action=opensearch&amp;search=%5C%22Foo+Bar&amp;quot">http://test.wikipedia.org/w/api.php?action=opensearch&amp;search=%5C%22Foo+Bar&amp;quot</a>

Details

Reference
bz15661

Event Timeline

bzimport raised the priority of this task from to Lowest.Nov 21 2014, 10:18 PM
bzimport set Reference to bz15661.
bzimport added a subscriber: Unknown Object (MLST).

I have tried to reproduce this at http://test.wikipedia.org/wiki/User:Qgil but honestly I get lost in the details.

Could you please help triage your own ol' bug report? :) Thank you.

Still reproducible. Can this bug be marked as EASY?

matmarex updated the task description. (Show Details)
matmarex removed a subscriber: wikibugs-l-list.
matmarex renamed this task from External links syntax breakage to External links syntax is ended by quotes.Mar 3 2017, 10:55 PM
matmarex added a subscriber: matmarex.

The first part is still reproducible, the quote still ends the link.

The second part appears to have been fixed, using &quot; to escape works correctly, no backslash appears.

Parsoid produces identical rendering.