Last modified: 2014-01-29 18:56:21 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T33244, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 31244 - LQT should escape thread titles before providing a link to the user
LQT should escape thread titles before providing a link to the user
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
LiquidThreads (Other open bugs)
unspecified
All All
: Normal normal with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
gci2013
: easy, javascript
: 32391 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-09-29 12:13 UTC by Helder
Modified: 2014-01-29 18:56 UTC (History)
10 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Helder 2011-09-29 12:13:55 UTC
When the page
https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Thread:Talk:Article_feedback/Bug%3F
showed up in [[mw:Special:NewMessages]] I wanted to get a link for it and clicked on "Link to". The first link provided is wrong, because it points to

https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Thread:Talk:Article_feedback/Bug?

which opens another thread (since the "?" is not escaped as it should).

I'm tagging this with the "javascript" keyword because the link provided when I disable JavaScript is correct.
Comment 1 Valerie Juarez 2013-03-19 20:34:58 UTC
Hi Helder! Sorry that nobody has taken a look at this report yet and given feedback. I can confirm that this report is still valid.
Comment 2 Helder 2013-03-19 20:46:17 UTC
No problem.

I'm adding keyword "easy", since escaping things shouldn't be hard to do...
Comment 3 Jesús Martínez Novo (Ciencia Al Poder) 2013-06-07 14:51:48 UTC
*** Bug 32391 has been marked as a duplicate of this bug. ***
Comment 4 Nemo 2013-12-27 09:20:54 UTC
OK for GCI.
Comment 5 Gerrit Notification Bot 2014-01-28 23:32:14 UTC
Change 110101 had a related patch set uploaded by Alex Monk:
Use mw.util.getUrl for generating Link URLs

https://gerrit.wikimedia.org/r/110101
Comment 6 Gerrit Notification Bot 2014-01-29 18:55:36 UTC
Change 110101 merged by jenkins-bot:
Use mw.util.getUrl for generating Link URLs

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

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links