Last modified: 2013-11-26 19:27:32 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 T50280, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 48280 - Change the default value for the concurrent uploads limit to be named "default"
Change the default value for the concurrent uploads limit to be named "default"
Status: RESOLVED INVALID
Product: MediaWiki extensions
Classification: Unclassified
UploadWizard (Other open bugs)
master
All All
: Low enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
: easy
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-05-08 23:09 UTC by Mark Holmquist
Modified: 2013-11-26 19:27 UTC (History)
3 users (show)

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


Attachments

Description Mark Holmquist 2013-05-08 23:09:46 UTC
A recent patch introduced a weird usability issue in UploadWizard where the 0th element in the array of valid options is 'default'. We should probably either

A) Use 0 for both the index and the value, and explain that 0 means "default" in an i18n-able message or

B) Use 'default' for both the index and the value and get cursed at by the i18n people.

If there are other options I'm open to them, but getting 0 out of that array would be desirable.
Comment 1 Derk-Jan Hartman 2013-05-25 08:37:57 UTC
@Mark, can you be more specific, because I know this code quite well, and with the above description I still don't know how to pinpoint the piece of code you are discussing :D
Comment 2 Andre Klapper 2013-06-05 14:18:05 UTC
mtraceur: Could you answer comment 1, please?
Comment 3 Mark Holmquist 2013-11-26 19:27:32 UTC
Sorry about the slow response, guys!

I honestly don't even know what I was talking about, I'll close this for now.

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


Navigation
Links