Broken links to scripts and styles on Special:Gadgets
Closed, ResolvedPublic
Actions

Description

Observed on 1.27.0-wmf.8 Wikimedia projects. An additional MediaWiki: prefix was prepended to every target link pointing to a JS script/CSS stylesheet on Special:Gadgets, resulting in page titles like MediaWiki:MediaWiki:Gadget-citations.js (see test.wikipedia.org).

Details

	Project	Branch	Lines +/-	Subject
	mediawiki/extensions/Gadgets	wmf/1.27.0-wmf.8	+1 -1	Fix double "MediaWiki:MediaWiki:" namespace on Special:Gadgets
	mediawiki/extensions/Gadgets	master	+1 -1	Fix double "MediaWiki:MediaWiki:" namespace on Special:Gadgets

Customize query in gerrit

Event Timeline

PeterBowman created this task.Dec 10 2015, 12:59 AM

PeterBowman raised the priority of this task from to Needs Triage.

PeterBowman updated the task description. (Show Details)

PeterBowman added projects: MediaWiki-extensions-Gadgets, Regression.

PeterBowman added a subscriber: PeterBowman.

Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald TranscriptDec 10 2015, 12:59 AM

Actually, it is pre-pended and I confirm this is a regression that has been recently added.

In T121047#1868381, @Rillke wrote:

Actually, it is pre-pended and I confirm this is a regression that has been recently added.

Oops, sorry. It's fixed now in the task description.

zhuyifei1999 added a subscriber: zhuyifei1999.Dec 10 2015, 5:57 AM

https://gerrit.wikimedia.org/r/228781 is responsible for this bug.

Aklapper triaged this task as High priority.Dec 10 2015, 11:33 AM

https://gerrit.wikimedia.org/r/228781 is responsible for this bug.

Why?

foreach ( $gadget->getScriptsAndStyles() as $codePage ) {
   $t = Title::makeTitleSafe( NS_MEDIAWIKI, $codePage );
   // ...
   $lnk[] = Linker::link( $t, htmlspecialchars( $t->getText() );

So you say $t at this point is already a botched title? This in turn would mean that $codePage is wrong?

https://gerrit.wikimedia.org/r/228781 is responsible for this bug.

Why?

I besected the problem to this change.

foreach ( $gadget->getScriptsAndStyles() as $codePage ) {
   $t = Title::makeTitleSafe( NS_MEDIAWIKI, $codePage );
   // ...
   $lnk[] = Linker::link( $t, htmlspecialchars( $t->getText() );
So you say $t at this point is already a botched title? This in turn would mean that $codePage is wrong?

I didn't analyzed the intend of the change.

Yeah, this is my fault, sorry.

In T121047#1869856, @Rillke wrote:
https://gerrit.wikimedia.org/r/228781 is responsible for this bug.

Why?
foreach ( $gadget->getScriptsAndStyles() as $codePage ) {
   $t = Title::makeTitleSafe( NS_MEDIAWIKI, $codePage );
   // ...
   $lnk[] = Linker::link( $t, htmlspecialchars( $t->getText() );
So you say $t at this point is already a botched title? This in turn would mean that $codePage is wrong?

Yes, $t is wrong here, because $codePage already has the namespace, so it should use Title::newFromText().

Change 258204 had a related patch set uploaded (by Legoktm):
Fix double "MediaWiki:MediaWiki:" namespace on Special:Gadgets

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

gerritbot added a project: Patch-For-Review.Dec 10 2015, 8:06 PM

Change 258204 merged by jenkins-bot:
Fix double "MediaWiki:MediaWiki:" namespace on Special:Gadgets

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

Change 258209 had a related patch set uploaded (by Legoktm):
Fix double "MediaWiki:MediaWiki:" namespace on Special:Gadgets

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

ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2015-12-15_(1.27.0-wmf.9)).Dec 10 2015, 9:00 PM

Change 258209 merged by jenkins-bot:
Fix double "MediaWiki:MediaWiki:" namespace on Special:Gadgets

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

Backported and deployed.

ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2015-12-08_(1.27.0-wmf.8)).Dec 10 2015, 10:00 PM

Broken links to scripts and styles on Special:GadgetsClosed, ResolvedPublicActions

Description

Details

Event Timeline

Broken links to scripts and styles on Special:Gadgets
Closed, ResolvedPublic
Actions