Page MenuHomePhabricator

Upload link changed from local Special:Upload to Special:UploadWizard on Commons
Closed, ResolvedPublicBUG REPORT

Description

On some wikis, the file upload link in the sidebar has changed from local Special:Upload to upload to commons Special:UploadWizard. It should not be so.
For example try to upload file on https://ru.wikipedia.org/wiki/.

Event Timeline

Iniquity created this task.Wed, Jun 24, 3:56 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptWed, Jun 24, 3:56 PM
Iniquity changed the subtype of this task from "Task" to "Bug Report".Wed, Jun 24, 3:57 PM

@Aklapper I'm a little unsure which tags should be added, can you help? :)

MBH added a subscriber: MBH.Wed, Jun 24, 4:18 PM
Aklapper added a comment.EditedWed, Jun 24, 5:02 PM

Sounds a bit similar to T256106: Link to non-existent Commons:Upload on betacommons without JavaScript though kind of inverted.

There is no local overwrite for the URL of Загрузить файл at https://ru.wikipedia.org/wiki/MediaWiki:Upload-url .
No recent changes to https://phabricator.wikimedia.org/source/mediawiki-config/browse/master/wmf-config/config/ruwiki.yaml or https://phabricator.wikimedia.org/source/mediawiki-config/browse/master/dblists/commonsuploads.dblist either.
And https://phabricator.wikimedia.org/source/mediawiki-config/browse/master/wmf-config/InitialiseSettings.php$2888 defines
'commonsuploads' => '//commons.wikimedia.org/wiki/Special:UploadWizard?uselang=$lang',

So I'm rather confused why this has not always linked to Commons' Special:UploadWizard already before. Any idea when this changed?

Change 607586 had a related patch set uploaded (by Urbanecm; owner: Urbanecm):
[operations/mediawiki-config@master] Revert "IS: Cleanup some redundant rows."

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

Change 607586 merged by jenkins-bot:
[operations/mediawiki-config@master] Revert "IS: Cleanup some redundant rows."

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

Mentioned in SAL (#wikimedia-operations) [2020-06-24T18:29:06Z] <urbanecm@deploy1001> Synchronized wmf-config/InitialiseSettings.php: dea9214: Revert "IS: Cleanup some redundant rows." (T256279) (duration: 01m 05s)

Urbanecm closed this task as Resolved.Wed, Jun 24, 6:30 PM
Urbanecm claimed this task.
Urbanecm added a subscriber: Urbanecm.

This has been errorneously changed with 74e8295d7bc994272a17a12e8996d10174ea018c, which tried to remove some duplicates rows. However, those rows were actually not duplicates. I've reverted the patch in question, and it now seems to work.

Restricted Application added a project: User-Urbanecm. · View Herald TranscriptWed, Jun 24, 6:30 PM

I'm a little unsure why I'm pinged here TBH.