Last modified: 2012-08-14 17:36:01 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 T34353, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 32353 - UploadWizard multi-file select broken on Firefox 3.6
UploadWizard multi-file select broken on Firefox 3.6
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
UploadWizard (Other open bugs)
unspecified
All All
: Normal major with 1 vote (vote)
: ---
Assigned To: Jeroen De Dauw
:
: 32481 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-11-10 23:46 UTC by Neil Kandalgaonkar
Modified: 2012-08-14 17:36 UTC (History)
8 users (show)

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


Attachments
Patch to disable Firefox 3.6's multiple file inputs (726 bytes, patch)
2012-05-09 21:28 UTC, Mark Holmquist
Details

Description Neil Kandalgaonkar 2011-11-10 23:46:31 UTC
It may be broken on FF 3.6 period, but this is the first report & the first way I could confirm the issue.

One can select multiple files, but all files except the first one fail with a "the file you submitted was empty" error. 

This means that the upload was submitted, but was perceived as 0 bytes long at the server.
Comment 1 Neil Kandalgaonkar 2011-11-10 23:48:58 UTC
Possibly related:

Saibo:
error numer 3: I said I tried to upload two files of which only one worked. The strange thing is: the file which worked is named now https://commons.wikimedia.org/wiki/File:Burning_eyes_smileyi.png but it is locally on my HDD burning eyes smileyb.png  unless I am completely crazy
Comment 2 Brion Vibber 2011-11-11 00:19:01 UTC
I can confirm that only one file makes it with Firefox 3.6.24 32-bit build on my Ubuntu 11.10 64-bit box.

Looking at the POSTs in Firebug, all of them do indeed submit empty files except for one...

Possibly something is mixing up the file objects...?


All files make it ok on a more current Firefox 7.0.2.
Comment 3 Erik Moeller 2011-12-28 22:20:31 UTC
Can reproduce in FF 3.6 on Windows and Ubuntu.

If we can't fix this, we should blacklist multi-file selection for the 3.X branches.

Increasing priority to "Normal".
Comment 4 Saibo 2011-12-28 23:11:35 UTC
That would need a probably simple blacklist and is unresolved for over 1.5 months despite I had directly on the deployment told this to Neil in IRC (AFAIK). Please only deploy new features if you have the manpower to support it it least a bit.
Comment 5 Mark Holmquist 2012-05-09 21:28:28 UTC
Created attachment 10571 [details]
Patch to disable Firefox 3.6's multiple file inputs

This fixes the problem by disallowing the multiple-file inputs on 3.6, after plenty of effort to fix it by enabling them.
Comment 6 Mark Holmquist 2012-05-09 21:33:56 UTC
For posterity: The bug is *not* that only one file is sent, but that only one POST request has any data in it--all of the files are sent in that request, in the same field, but the server-side only accepts one file at a time. The problem here appears to be that, for whatever UploadWizard is doing to separate the file inputs, Firefox 3.6 had not yet fully implemented the necessary features in either FileAPI or something else.

This interesting caveat brings about such crazy results as, for example, the filename of the first file being preserved, but that file actually being populated with the contents of the last file, because the server side only reads that one.
Comment 7 Mark Holmquist 2012-05-09 22:58:10 UTC
Code review started: https://gerrit.wikimedia.org/r/7120
Comment 8 Andre Klapper 2012-05-10 19:34:34 UTC
I assume this is also still a problem in 10.0.4 and 12.0? 
(As 3.6 is mentioned in the bug summary but 3.6 is EOLed.)
Comment 9 Mark Holmquist 2012-05-10 19:39:48 UTC
No, 3.6 appears to be the only problematic browser. Regardless of its status at Mozilla, there are still a lot of people using it--http://upsdell.com/BrowserNews/stat.htm says between 0.5% and 12% still use Firefox 3, which is still a pretty big number.
Comment 10 Mark Holmquist 2012-05-10 22:18:41 UTC
*** Bug 32481 has been marked as a duplicate of this bug. ***
Comment 11 Sumana Harihareswara 2012-05-11 15:19:50 UTC
Adding patch and patch-need-review keywords. Thanks, Mark.
Comment 12 Mark A. Hershberger 2012-05-11 17:51:49 UTC
(In reply to comment #9)
> No, 3.6 appears to be the only problematic browser. Regardless of its status at
> Mozilla, there are still a lot of people using
> it--http://upsdell.com/BrowserNews/stat.htm says between 0.5% and 12% still use
> Firefox 3, which is still a pretty big number.

http://stats.wikimedia.org/wikimedia/squids/SquidReportClients.htm shows 2.37% for FF 3.6, more than the largest Safari version.
Comment 13 Brion Vibber 2012-05-22 23:28:24 UTC
Merged in commit 7763f2fb3f089e2843e3c3b641c1d2014df6336d

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


Navigation
Links