Page MenuHomePhabricator
Create Task
Maniphest T170454

Instead of "None of the uploads were successful." display what went wrong (an error message)
Closed, DeclinedPublic
Actions

Description

When an upload with the Commons Upload Wizard fails it should display an error message about what exactly went wrong such as "a file with this title already exists". As of right now it only displays "None of the uploads were successful." with a button to "Retry failed uploads" which is useless for actually correcting the error.

Furthermore if a file with that same name already exists it should display link to said image and preferably its thumbnail so users can check whether it's the same image.

Event Timeline

Fixuture created this task.Jul 12 2017, 6:20 PM
Restricted Application added a project: Multimedia. · View Herald TranscriptJul 12 2017, 6:20 PM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
matmarex added a subscriber: matmarex.Jul 12 2017, 6:25 PM

It already does both of these things. Here's a screenshot of what I see:

image.png (968×1 px, 123 KB)

If you see something else, please elaborate (and upload a screenshot too).

Fixuture added a comment.Jul 12 2017, 6:34 PM

Here's what I see:

s1.png (685×1 px, 85 KB)

matthiasmullie added a subscriber: matthiasmullie.Jul 13 2017, 1:19 PM

On your screenshot, it explicitly says that the image is a duplicate of an existing image, with a link to that other image.
In this specific case, we could probably get rid of the "retry" button, since it's not really a retryable error (unless you were to remove the duplicate, in which case you could retry).
But apart from that, I'm not sure what else we could do here? The detailed error information is there already.

Aklapper added a comment.Jul 20 2017, 10:33 PM
In T170454#3435222, @matthiasmullie wrote:

But apart from that, I'm not sure what else we could do here? The detailed error information is there already.

@Fixuture: Could you clarify?

dr0ptp4kt changed the task status from Open to Stalled.Aug 7 2017, 4:24 PM
dr0ptp4kt triaged this task as Low priority.
dr0ptp4kt added a subscriber: dr0ptp4kt.

Updating status and priority until further information received.

matmarex removed a subscriber: matmarex.Aug 13 2017, 6:11 PM
Aklapper closed this task as Declined.Aug 24 2017, 12:43 PM
In T170454#3458485, @Aklapper wrote:

@Fixuture: Could you clarify?

Unfortunately closing this report as no further information has been provided.
@Fixuture: After you have provided the information asked for and if this still happens, please set the status of this report back to "Open" via the Add Action... → Change Status dropdown. Thanks!

Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL