Page MenuHomePhabricator

file upload error messages don't work sanely
Closed, ResolvedPublic

Description

After specifying the name of a deleted file ("jar2.jpg") on my local install I saw:

There was a file by this name, "File:Jar.jpg", but it was deleted and you
can not reupload the file. If your file is different, try renaming it.

So I gave it "jar.jpg" (the file that had been moved to "jar2.jpg" and then deleted) and I saw this text:

A file with this name exists already.There was a file by this name,
"File:Jar.jpg", but it was deleted and you can not reupload the file. If
your file is different, try renaming it.

That is, "A File with this name exists already." was just added to the beginning of the previous message w/o erasing it.


Version: unspecified
Severity: normal

Details

Reference
bz32469

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:58 PM
bzimport added a project: UploadWizard.
bzimport set Reference to bz32469.
bzimport added a subscriber: Unknown Object (MLST).

I tried reproducing this and am getting an alert with "please wait, still checking the title for uniqueness..." instead of any error in the UI, after which nothing happens :/

Actual sequence of events before comment 0:

  1. Uploaded jar.jpg to local wiki with UW.
  2. Moved jar.jpg to jar2.jpg and left a redirect.
  3. Deleted jar2.jpg.

Let me know if that does not let you reproduce this.

I tried again, those 3 steps, then uploading jar.jpg. And again I'm getting "please wait, still checking the title for uniqueness...".

Looks like we got 2 competing bugs here, fighting for the same turf :)

Mark and Jeroen, can I ask you (though it was a long time ago) what browser(s) you used to test this? I have a sneaking suspicion that IE does something to break how UW does validation.

Jeroen, I think this patch might fix your problem, anyway: https://gerrit.wikimedia.org/r/7608

If not, let me know, because I need to figure out what's causing the issue.

ping me in another day and I'll look at this. I don't have time right now.

Mark, I have totally dropped the ball on this one. Can you take a glance at this sometime soon?

(also, updating keyword to be more better)

FWIW, I managed to perform the steps in the original report without issue, so I think we may have gotten a free fix somewhere.

Fixed in https://gerrit.wikimedia.org/r/#/c/21211/ and some previous change (don't recall the other one).

Gilles raised the priority of this task from Medium to Unbreak Now!.Dec 4 2014, 10:27 AM
Gilles moved this task from Untriaged to Done on the Multimedia board.
Gilles lowered the priority of this task from Unbreak Now! to Medium.Dec 4 2014, 11:22 AM