Last modified: 2012-04-12 13:59:48 UTC
Created attachment 7289 [details] Ignore exists warning when reuploading Users of commons find it annoying to have to double stage all file uploads or to have to always check "ignore all warnings". I was thinking it might be a good idea to always ignore the "fileexists" warning when the reupload option is used and the "fileexists" warning is the only warning. Possibly extended to only do this for registered users and given unregistered users the warning at all times. I have added a small patch to demo this behavior on a wiki.
I am pretty sure that this behaviour is a code update regression.
The "ignore all warnings" used to be checked in the uploadform by default in older versions during reupload. Personally I consider this a bad choice, because than it also ignores duplicates and filetype mismatches. As such though this might be a regression, I do think that the reason this has regressed was valid and should remain. It just needs more work I guess.
The real problem on Commons is not that you get a warning about the existing file, it's that the "Ignore warning and save file anyway" button on the confirmation page is disabled and unclickable, so you literally *can't* save the file except by selecting "Ignore any warnings" in advance. Note that using the Add Media Wizard (via the mwEmbed) gadget is apparently another way to work around this issue, although it has its own problems (it insists on prepending "== Summary == " to the upload summary, sometimes several times, for some reason).
Disabling JavaScript also seems to make the button usable. Haven't figured out yet which script is disabling it, though.
Ideally only the file-exists error is skipped and all other warnings are just passed through.
(In reply to comment #5) > Ideally only the file-exists error is skipped and all other warnings are just > passed through. Actually that is supposed to be already supported using a JavaScript called destWarningAck. Did Commons disable this somehow?
The issue is present on the English Wikipedia as well. Though we don't see the "disabled" button, so that must have another cause.
Ah, might this be related to the broken JavaScript bug?
Javascript from bug 22970 deployed. Does this fix the issue?
Should be fixed with r65025
Finally fixed with r65029