Last modified: 2014-09-23 19:32:00 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 T34247, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 32247 - After uploading files, cannot go back or cancel uploads
After uploading files, cannot go back or cancel uploads
Status: NEW
Product: MediaWiki extensions
Classification: Unclassified
UploadWizard (Other open bugs)
unspecified
All All
: High normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
: 30687 34191 36583 (view as bug list)
Depends on: 29594
Blocks: 37144 42164
  Show dependency treegraph
 
Reported: 2011-11-07 21:29 UTC by Nux
Modified: 2014-09-23 19:32 UTC (History)
12 users (show)

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


Attachments
added navigation support to navigate through various steps (1.31 KB, patch)
2012-03-18 17:35 UTC, drecodeam
Details
added .patch file for the previous .diff file (1.31 KB, patch)
2012-03-18 17:39 UTC, drecodeam
Details

Description Nux 2011-11-07 21:29:55 UTC
Here is my scenario:
1. I choose many files - all of them are named P39283982.jpg
2. I upload them (not seeing thumbnails).
3. I release all rights.
4. OMG! I just remembered I cannot upload this one! Where is the back button!

At a very least the user should be able to remove the file or cancel the whole process. Currently you are left alone and don't even know where the files are and how to remove them.
Comment 1 Neil Kandalgaonkar 2011-11-07 22:52:28 UTC
Good point, we don't have a way to do that now. We should have a way to delete the file on any page, up to  the final upload.
Comment 2 Platonides 2011-12-23 00:34:35 UTC
The "available" solution is to close the window but yes, this is suboptimal.
Comment 3 drecodeam 2012-03-18 17:35:38 UTC
Created attachment 10256 [details]
added navigation support to navigate through various steps

Bind click handlers to step titles, to navigate to any step from any step. Click handler in turn invokes the moveToStep('<step name>') with the respective step.
Comment 4 drecodeam 2012-03-18 17:39:30 UTC
Created attachment 10257 [details]
added .patch file for the previous .diff file

added patch file for the 10256. No changes.
Comment 5 Sumana Harihareswara 2012-03-18 19:03:06 UTC
Thanks for the patch, drecodeam.  I've added some keywords so people know that your patch awaits review, as you can read about in  https://www.mediawiki.org/wiki/How_to_become_a_MediaWiki_hacker#Submit_your_changes , and I've given you permissions to add those keywords in the future yourself. :-)
Comment 6 Sumana Harihareswara 2012-03-22 19:40:06 UTC
Also, drecodeam, you can request a Git/Gerrit account

https://www.mediawiki.org/wiki/Project:Labsconsole_accounts

and submit your patch as a merge request to get it reviewed faster:

https://www.mediawiki.org/wiki/Git/Workflow

Thanks!
Comment 7 drecodeam 2012-04-13 17:54:52 UTC
The patch i submitted earlier has some issues with it, as the navigation works but the step divs are not ready right now for navigation. This patch needs more work, and hence is not suitable right now.
Comment 8 Mark A. Hershberger 2012-05-11 18:11:43 UTC
*** Bug 30687 has been marked as a duplicate of this bug. ***
Comment 9 Mark Holmquist 2012-05-15 00:16:57 UTC
*** Bug 36583 has been marked as a duplicate of this bug. ***
Comment 10 Mark Holmquist 2012-05-29 16:53:17 UTC
drecodeam, I'm not sure if you're still working on this, but I'm starting to think that it might be better to just add in remove controls on every page until "thanks", serving exactly the same purpose with exactly none of the complications of reverse-navigation. I really can't see a need for moving back to the upload step, since moving *forward* from it would necessarily involve re-uploading the files, which is not a good result. Also, trying to navigate backwards from "thanks" might be complicated, since by that point the files have been fully uploaded and any backwards navigation would involve deleting them from the server and then re-uploading them.

Does anyone have any thoughts on this? Maybe I'm blowing these complications out of proportion?
Comment 11 Mark Holmquist 2012-05-31 20:18:16 UTC
I have whipped up a quick patch that adds in "remove" functionality at the details step, which is really the only other place that makes sense (excluding the "upload" step which is obvious). It works really well, it's a very minor change, and I'd appreciate review.

Here it is: https://gerrit.wikimedia.org/r/9549
Comment 12 Mark Holmquist 2012-06-05 18:15:36 UTC
*** Bug 34191 has been marked as a duplicate of this bug. ***
Comment 13 Andre Klapper 2012-08-14 16:23:30 UTC
[Offtopic] Mark: Can somebody tell me where the keyword "patch-in-gerrit" was discussed (archives URL?) before it was introduced?
Comment 14 Mark Holmquist 2012-08-14 16:28:44 UTC
About ten minutes ago in #mediawiki :)

I had marked a bunch of bugs as patch-need-review, but the description (something like "patch attached, needs review") didn't match what I meant. So Chad added it for me.
Comment 15 Gerrit Notification Bot 2013-06-27 23:08:01 UTC
Change 9549 had a related patch set uploaded by MarkTraceur:
(bug 32247) Add "remove file" in the details step

https://gerrit.wikimedia.org/r/9549
Comment 16 Andre Klapper 2014-03-11 14:25:03 UTC
(In reply to Gerrit Notification Bot from comment #15)
> Change 9549 had a related patch set uploaded by MarkTraceur:
> https://gerrit.wikimedia.org/r/9549

Patch has been awaiting review for nine months...
Comment 17 Andre Klapper 2014-05-20 10:43:32 UTC
https://gerrit.wikimedia.org/r/9549

Patch has been awaiting review for eleven months...
Comment 18 Steinsplitter 2014-09-22 18:55:19 UTC
Open for too long, [ Please rebase your change and upload a new patchset.]

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


Navigation
Links