[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 Jeroen De Dauw jeroen_ded...@yahoo.com changed: What|Removed |Added Status|REOPENED|ASSIGNED --- Comment #9 from Jeroen De Dauw jeroen_ded...@yahoo.com 2011-12-07 15:17:25 UTC --- 1) Ok, did not notice this myself during testing :/ 2) So the only thing you want me to do is add in total to the message? I could also go for something like You tried to upload 5 files while already having 3, so 3 of those have been removed. if that's clearer. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 --- Comment #10 from Neil Kandalgaonkar ne...@wikimedia.org 2011-12-07 16:30:17 UTC --- (In reply to comment #9) 2) So the only thing you want me to do is add in total to the message? No, I also changed add to upload. I just think that You tried to add 8 files seems wrong, as it implies that I *just* tried to add 8 files. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 Jeroen De Dauw jeroen_ded...@yahoo.com changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED --- Comment #11 from Jeroen De Dauw jeroen_ded...@yahoo.com 2011-12-07 17:03:39 UTC --- Should be fixed by r105441 -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 --- Comment #12 from Neil Kandalgaonkar ne...@wikimedia.org 2011-12-07 21:52:57 UTC --- Looks good. Thanks for the quick fix. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 Neil Kandalgaonkar ne...@wikimedia.org changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED | --- Comment #8 from Neil Kandalgaonkar ne...@wikimedia.org 2011-12-07 01:51:15 UTC --- The current fix seems to have an off-by-one error and a few other problems. For all below cases, in LocalSettings.php $wgUploadWizardConfig['maxUploads'] = 5 1) Off by one error: - Uploading files, selected File1..10.jpg - Dialog triggered correctly, saying I had 10 files and 5 would be dropped - However, only File1..4 were shown. - At this point, the Add another upload button was still enabled - Adding another upload brings the total to 5. No dialog is shown saying any will be dropped. Add another upload button is disabled. So that seems to work. 2) Does not track current total - Add 3 files to UploadWizard, successful as usual - Add 5 more - Dialog triggered, incorrectly stating: You can only upload 5 files at once. You tried to add 8 files, so 3 files have been removed. It would be more correct to say: You can only upload 5 files at once. You tried to upload 8 files in total, so 3 files have been removed. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 Jeroen De Dauw jeroen_ded...@yahoo.com changed: What|Removed |Added Status|REOPENED|RESOLVED Resolution||FIXED --- Comment #7 from Jeroen De Dauw jeroen_ded...@yahoo.com 2011-11-24 14:44:51 UTC --- Dialog added in r104167 -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 Jeroen De Dauw jeroen_ded...@yahoo.com changed: What|Removed |Added Status|NEW |RESOLVED Resolution||FIXED --- Comment #1 from Jeroen De Dauw jeroen_ded...@yahoo.com 2011-11-23 23:38:22 UTC --- Fixed by r104103. Maybe it can be done in a nicer way, but I did not see any after poking at the code a bit. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 Neil Kandalgaonkar ne...@wikimedia.org changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED | --- Comment #2 from Neil Kandalgaonkar ne...@wikimedia.org 2011-11-23 23:46:42 UTC --- This will work but I think the solution needs to be a bit better. 1 - it's really inelegant to do all the work of adding uploads and then removing them. There are all sorts of side effects (for one, there are a few static counters in various classes). Multiple files are launched around line 333 of mw.UploadWizardUpload.js, maybe start from there? 2 - after someone goes over the limit, we need to throw up a dialog to explain to them what happened. Otherwise they will probably not notice. By the way, after they are over the limit, check that the 'new upload' button is properly disabled -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 --- Comment #4 from Jeroen De Dauw jeroen_ded...@yahoo.com 2011-11-24 00:34:31 UTC --- Second attempt in r104117 - still need to do the warning thing though. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 --- Comment #5 from Jeroen De Dauw jeroen_ded...@yahoo.com 2011-11-24 00:35:59 UTC --- Any suggestion on how to display such a warning? Is there any appropriate warning display thinghy yet, or do I need to create something new? -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 --- Comment #6 from Neil Kandalgaonkar ne...@wikimedia.org 2011-11-24 00:38:03 UTC --- (In reply to comment #5) Any suggestion on how to display such a warning? Is there any appropriate warning display thinghy yet, or do I need to create something new? do a modal dialog with jQuery UI dialog. There are some examples in the codebase. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 --- Comment #3 from Jeroen De Dauw jeroen_ded...@yahoo.com 2011-11-24 00:09:12 UTC --- Right. Multiple files are launched around line 333 of mw.UploadWizardUpload.js, maybe start from there? Ah, that seems more promising then where I was looking :) -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 Neil Kandalgaonkar ne...@wikimedia.org changed: What|Removed |Added AssignedTo|iba...@wikimedia.org|jeroen_ded...@yahoo.com -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 Neil Kandalgaonkar ne...@wikimedia.org changed: What|Removed |Added Priority|Unprioritized |High -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 31341] multiple file selection doesn't respect maxUpload config
https://bugzilla.wikimedia.org/show_bug.cgi?id=31341 Neil Kandalgaonkar ne...@wikimedia.org changed: What|Removed |Added AssignedTo|wikibugs-l@lists.wikimedia. |iba...@wikimedia.org |org | -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l