Last modified: 2014-10-09 14:23:02 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 T56750, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 54750 - UploadWizard should not allowing upload two different files with same name at same time
UploadWizard should not allowing upload two different files with same name at...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
UploadWizard (Other open bugs)
unspecified
All All
: Normal normal with 2 votes (vote)
: ---
Assigned To: Rainer Rillke @commons.wikimedia
:
: 31208 34230 56678 64883 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-09-29 00:03 UTC by Bawolff (Brian Wolff)
Modified: 2014-10-09 14:23 UTC (History)
12 users (show)

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


Attachments

Description Bawolff (Brian Wolff) 2013-09-29 00:03:49 UTC
Steps to reproduce:
*Start upload wizard
*Upload 2 files
*Give them the same name on the describe step
*hit done

Expected behaviour:
*Upload wizard should give an error to the user, as this is almost certainly not what the user meant to do. Presumably this was a typo on the part of the user, and they did not mean for the files to have same title (or didn't realize there is 1:1 relation of titles to files. In any case, if user is uploading 2 files at the same time, they probably want them to be separate files.

Actual behaviour:
*Upload wizard uploads the first file, and then uploads the second file over top of the first. 

Additional actual behaviour:
*Race conditions galore in the backend cause bad things to happen, and data-loss on one of the files. See bug 54736. However, even if that didn't happen, this is still a behaviour of upload wizard that I think should be fixed.
Comment 1 Nemo 2013-09-30 12:06:10 UTC
*** Bug 34230 has been marked as a duplicate of this bug. ***
Comment 2 Nemo 2013-09-30 12:10:55 UTC
Thanks for mentioning this in bug 54776, I had seen the summary in my search results but I thought it was another thing (the bug I just duplicated to this).
Comment 3 Bawolff (Brian Wolff) 2013-11-06 17:38:52 UTC
*** Bug 56678 has been marked as a duplicate of this bug. ***
Comment 4 Pikne 2013-11-06 18:23:19 UTC
*** Bug 31208 has been marked as a duplicate of this bug. ***
Comment 5 Lokal_Profil 2014-04-22 09:33:18 UTC
Would it be possible to check a Commons dump to see how often this occurs? It should be detectable as a upload over an existing file with the edit comment "User created page with UploadWizard" [1]. Since the wizard does not allow re-uploading this should only ever occur in this case.

[1] https://commons.wikimedia.org/w/index.php?title=Special:Log&page=File%3AInstrument+-+Karin+Ohlin%2C+2.jpg
Comment 6 Bawolff (Brian Wolff) 2014-05-19 20:59:16 UTC
Note, potential dupe at bug 64883 (But this one was here first).

(In reply to Lokal_Profil from comment #5)
> Would it be possible to check a Commons dump to see how often this occurs?
> It should be detectable as a upload over an existing file with the edit
> comment "User created page with UploadWizard" [1]. Since the wizard does not
> allow re-uploading this should only ever occur in this case.
> 
> [1]
> https://commons.wikimedia.org/w/index.php?title=Special:
> Log&page=File%3AInstrument+-+Karin+Ohlin%2C+2.jpg

MariaDB [commonswiki_p]> select count(*) from logging_logindex where log_type = 'upload' and log_action = 'overwrite' and log_comment = 'User created page with UploadWizard' ;
+----------+
| count(*) |
+----------+
|    19511 |
+----------+
1 row in set (2 min 55.42 sec)
Comment 7 Rainer Rillke @commons.wikimedia 2014-05-19 21:41:44 UTC
*** Bug 64883 has been marked as a duplicate of this bug. ***
Comment 8 Gerrit Notification Bot 2014-05-19 21:42:21 UTC
Change 133434 had a related patch set uploaded by Rillke:
UploadWizard: Check for duplicate titles

https://gerrit.wikimedia.org/r/133434
Comment 9 Gerrit Notification Bot 2014-05-20 00:20:14 UTC
Change 133434 merged by jenkins-bot:
UploadWizard: Check for duplicate titles

https://gerrit.wikimedia.org/r/133434
Comment 10 Mark A. Hershberger 2014-10-09 14:23:02 UTC
Couldn't trivially backport.

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


Navigation
Links