Last modified: 2013-05-16 23:33:29 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 T35513, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 33513 - Upload Wizard "Back to the old form": complete the list of altUploadForm in $wgUploadWizardConfig
Upload Wizard "Back to the old form": complete the list of altUploadForm in $...
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Site requests (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
: i18n, shell
: 29791 30791 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-01-04 14:46 UTC by Saibo
Modified: 2013-05-16 23:33 UTC (History)
9 users (show)

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


Attachments
A potential fix? (5.03 KB, patch)
2013-05-15 19:31 UTC, Tomasz W. Kozlowski
Details

Description Saibo 2012-01-04 14:46:44 UTC
German interface:

Should link to [[Commons:Hochladen]], but links to [[Commons:Upload]]. That is NOT the old forms.
Comment 1 Sam Reed (reedy) 2012-01-04 15:39:02 UTC
'mwe-upwiz-subhead-alt-upload'
Comment 2 Brion Vibber 2012-01-04 19:12:49 UTC
That message contains a link, but the link *target* is provided as a parameter to it. Comes from somewhere else?
Comment 3 Brion Vibber 2012-01-04 19:16:32 UTC
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?
Comment 4 Neil Kandalgaonkar 2012-01-25 03:07:49 UTC
fixed in r109992, although this will require a config change too.
Comment 5 Saibo 2012-01-25 16:51:10 UTC
(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
Comment 6 Saibo 2012-01-25 17:02:39 UTC
.. I will make the instructions more clear.. wait some minutes. :-)
Comment 7 Saibo 2012-01-25 17:14:28 UTC
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!
Comment 8 Saibo 2012-01-25 17:16:01 UTC
(where is the comment edit function in bugzilla.. damn)

Btw: that page defaults to English if a language specific page is not available.
Comment 9 Neil Kandalgaonkar 2012-01-25 21:40:49 UTC
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?
Comment 10 Saibo 2012-01-25 23:20:58 UTC
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.
Comment 11 Saibo 2012-01-25 23:22:48 UTC
edit: s/but/bit
Comment 12 Mark Holmquist 2012-05-29 17:33:26 UTC
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".
Comment 13 Mark Holmquist 2012-06-05 00:00:58 UTC
*** Bug 29791 has been marked as a duplicate of this bug. ***
Comment 14 Saibo 2012-06-05 23:06:12 UTC
(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)"
Comment 15 Mark Holmquist 2012-06-05 23:11:52 UTC
Once again, this is now a _configuration_ issue. Recategorizing. Can someone please make the relevant change outlined above in the Commons configuration?
Comment 16 Ferry Roland 2012-09-30 00:36:38 UTC
I'm not Saibo. I tried in de but not work.
Comment 17 Andre Klapper 2013-01-29 20:11:26 UTC
*** Bug 30791 has been marked as a duplicate of this bug. ***
Comment 18 Nemo 2013-03-06 16:41:46 UTC
(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.
Comment 19 Mark Holmquist 2013-03-06 16:47:15 UTC
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';
Comment 20 Nemo 2013-03-06 17:11:37 UTC
Thanks. I'm not very familiar with UW config and I guess many patch author candidates aren't either, so this helps.
Comment 21 Gerrit Notification Bot 2013-05-14 17:28:32 UTC
Related URL: https://gerrit.wikimedia.org/r/63686 (Gerrit Change Ia3fa29b69b27378cd208b3c21203f9413506d84d)
Comment 22 Tomasz W. Kozlowski 2013-05-14 18:23:24 UTC
Please note—this is /not/ the required config change :)
Comment 23 Tomasz W. Kozlowski 2013-05-15 19:30:26 UTC
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.
Comment 24 Tomasz W. Kozlowski 2013-05-15 19:31:58 UTC
Created attachment 12322 [details]
A potential fix?

If I'm not awfully mistaken, this is how a potential patch fixing this could look like.
Comment 25 Sam Reed (reedy) 2013-05-16 20:51:33 UTC
(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/
Comment 26 Gerrit Notification Bot 2013-05-16 21:06:51 UTC
Related URL: https://gerrit.wikimedia.org/r/64200 (Gerrit Change Ifda8b61a8ae5ff04de5214925c23869e47e3bdd3)
Comment 27 Gerrit Notification Bot 2013-05-16 23:33:29 UTC
https://gerrit.wikimedia.org/r/64200 (Gerrit Change Ifda8b61a8ae5ff04de5214925c23869e47e3bdd3) | change APPROVED and MERGED [by jenkins-bot]

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


Navigation
Links