Page MenuHomePhabricator

Upload Wizard "Back to the old form": complete the list of altUploadForm in $wgUploadWizardConfig
Closed, ResolvedPublic

Description

Author: saibotrash

Description:
German interface:

Should link to [[Commons:Hochladen]], but links to [[Commons:Upload]]. That is NOT the old forms.


Version: unspecified
Severity: enhancement

Details

Reference
bz33513

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 12:08 AM
bzimport set Reference to bz33513.
bzimport added a subscriber: Unknown Object (MLST).

'mwe-upwiz-subhead-alt-upload'

That message contains a link, but the link *target* is provided as a parameter to it. Comes from somewhere else?

Seems to come from 'altUploadForm' in the UploadWizard config.

Reedy, are you recommending that all languages which have an alternate classic form just modify the mwe-upwiz-subhead-alt-upload message to ignore this configuration variable? Or is there another way to make that setting go to whatever is appropriate?

neilk wrote:

fixed in r109992, although this will require a config change too.

saibotrash wrote:

(In reply to comment #4)

fixed in r109992, although this will require a config change too.

I hope you did not only change it for German, did you? :-O

saibotrash wrote:

.. I will make the instructions more clear.. wait some minutes. :-)

saibotrash wrote:

Okay, I make it more clear what to do - I thought it was clear that it is not only German. :-)

Please just link to for every language. Any changes for some languages can be made on that page if required.
'//commons.wikimedia.org/w/index.php?title=Commons:Upload/autolang&uselang=' + mw.config.get( 'wgUserLanguage' );

Thank you!

saibotrash wrote:

(where is the comment edit function in bugzilla.. damn)

Btw: that page defaults to English if a language specific page is not available.

neilk wrote:

I did not change it only for German, I made it so you could configure any number of pages.

array(

'default'	=> 'Commons:Upload',
'de'		=> 'Commons:Hochladen',
'fr'          => 'The extra special upload page for french people'
...

);

But, that would be a site config change. Your autolang concept is more under control of the admins of the site, at the cost of an ugly title.

So, I could just revert the changes to the code and then put Commons:Upload/autolang as the the 'single' alternative upload form.

However, why are you suggesting that there be a uselang parameter? Isn't that already implicit?

saibotrash wrote:

You know that I would like to have as much as possible (best: all) of UW under control of the wiki. :-) Of course I do not like a config file solution which is really hard to maintain if UW doesn't pull the list of available language sub pages from the wiki.

Agree, "/autolang" is a but ugly.. right. Maybe "/autotranslated" or just "/auto" would be nicer - should I move?
I also had the idea of moving it to [[:commons:Commons:Upload]] and move the English page to /en (where is would usually belong in case of language subpages) https://commons.wikimedia.org/wiki/Commons_talk:Upload#Commons:Upload.2Fautolang The downside is that usually there was the English page and many many link will go there - so a change is not very likely.

I had a short discussion in IRC now: lets leave it at this page name for now. If it should be moved the link form UW would continue to work (redir) and can be changed if needed.
So please link to the link as mentioned in comment 7. Thanks!

Explanation of the uselang suggestion: If the UW was called with URL param uselang and you do not add it to the new link the language chosen by uselang will be lost. E.g. Commons:Upload/de is linked from some Wikipedia "how to upload images" (newbie oriented) pages. Commons:Upload/de features a link to UW with uselang=de. Sadly not all users set their language preference in their user preferences.

saibotrash wrote:

edit: s/but/bit

Since this problem appears to have been solved two ways, i.e., Saibo's autolang feature combined with a configuration change and the new array feature in the configuration file, I'm going to call this one "resolved".

  • Bug 29791 has been marked as a duplicate of this bug. ***

saibotrash wrote:

(In reply to comment #12)

Since this problem appears to have been solved two ways, i.e., Saibo's autolang
feature combined with a configuration change and the new array feature in the
configuration file, I'm going to call this one "resolved".

Did you test it? Not fixed.
https://commons.wikimedia.org/w/index.php?title=Special:UploadWizard&uselang=de

Zurück zum alten Formular

a page containing english text: "Where is the work from? (Click on the appropriate link)"

Once again, this is now a _configuration_ issue. Recategorizing. Can someone please make the relevant change outlined above in the Commons configuration?

masatia wrote:

I'm not Saibo. I tried in de but not work.

  • Bug 30791 has been marked as a duplicate of this bug. ***

(In reply to comment #15)

Once again, this is now a _configuration_ issue. Recategorizing. Can someone
please make the relevant change outlined above in the Commons configuration?

What's the configuration variable?
In that patch, the config was changed in UploadWizard.config.php, not in WMF's InitialiseSettings.php.

The patch above added handling for a config variable and an *example* in the default configuration file. If you look at the top of UploadWizard.config.php, it cautions you not to modify that file, but instead to use the LocalSettings.php file (or whatever you'd like to use in lieu of that), adding lines of the form

$wgUploadWizardConfig['name'] = 'value';

Thanks. I'm not very familiar with UW config and I guess many patch author candidates aren't either, so this helps.

Related URL: https://gerrit.wikimedia.org/r/63686 (Gerrit Change Ia3fa29b69b27378cd208b3c21203f9413506d84d)

Please note—this is /not/ the required config change :)

54 languages on Commons currently have their own localised version of [[:commons:Commons:Upload]].

Adding a definition for each of them to a config file is not scalable, wrong, and just crazy.

Created attachment 12322
A potential fix?

If I'm not awfully mistaken, this is how a potential patch fixing this could look like.

Attached:

(In reply to comment #24)

Created attachment 12322 [details]
A potential fix?

If I'm not awfully mistaken, this is how a potential patch fixing this could
look like.

Done. https://gerrit.wikimedia.org/r/#/c/64194/

Attached:

Related URL: https://gerrit.wikimedia.org/r/64200 (Gerrit Change Ifda8b61a8ae5ff04de5214925c23869e47e3bdd3)

https://gerrit.wikimedia.org/r/64200 (Gerrit Change Ifda8b61a8ae5ff04de5214925c23869e47e3bdd3) | change APPROVED and MERGED [by jenkins-bot]