Bug 246727 - Add undo functionality to move to trash action
Summary: Add undo functionality to move to trash action
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Database-Trash (show other bugs)
Version: 1.4.0
Platform: Compiled Sources Linux
: NOR wishlist
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-04 22:02 UTC by Simon
Modified: 2019-07-13 05:41 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.0.0


Attachments
undotrash.png (319.11 KB, image/png)
2019-07-10 20:24 UTC, Maik Qualmann
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Simon 2010-08-04 22:02:09 UTC
Version:           1.4.0 (using KDE 4.4.5) 
OS:                Linux

because there seems to be no other way to recover images when putting them to the trash via Image > move to trash.

Reproducible: Always




OS: Linux (x86_64) release 2.6.34-0.slh.11-sidux-amd64
Compiler: cc
Comment 1 Johannes Wienke 2010-08-04 22:10:43 UTC
You can simply restore them using the normal trash restore operation.
Comment 2 Simon 2010-08-04 22:14:48 UTC
No.
This results in an error message.

And how to know which images I just deleted? Perhaps I've got a row of images of the same subject, and only one of them perfectly sharp. If I delete the unsharp ones first and then later the sharp ones accidentially, I can go through all images again.
Comment 3 Johannes Wienke 2010-08-04 22:18:50 UTC
What is the message? I can't reproduce this here.

To solve the problem of finding the images just deleted I would vote for a column in the list view of the trash that manages a deletion date. Like this the feature is available for every other use case as well.
Comment 4 Simon 2010-08-04 22:31:08 UTC
It complained about a folder not existing anymore. But
1. I didn't delete a folder in the meantime
2. All of the images I selected have been in folders that still exist.

A deletion date would be very helpful, that is true. I'd nevertheless vote for a built-in undo action (be it even only one step). E.g. if I'm deleting other files at the same time. Or is restoring from the trash complex?
Comment 5 Johannes Wienke 2010-08-04 22:46:28 UTC
For the deletion date please open a bug report against the kde lib providing the trash.

Undo seems to be possible using FileUndoManager from kio. We need to check this.
Comment 6 Simon 2010-08-04 23:26:41 UTC
#246740 added.
Comment 7 Jens 2015-09-03 09:45:51 UTC
Is this bug still active? If so, please extend this undo feature wish to all file system operations if there is already an API for file system level undo.

I recently accidentally moved about 1500 photos scattered over hundreds of albums on my NAS into a single album by searching for "similar" images and collecting them, not realizing that Digikam would also find images on the mounted NAS image collections... I just wanted to do this for local images. Now how would I get those moved images back into the right folders? :-(
Comment 8 bert 2018-10-17 06:52:33 UTC
I hope it is still active. To clarify. Usecase: You delete a bunch of photos, one by one and delete one too many (sound familiar?:-) A simple Ctrl-z to undo the last one would be a very quick and userfriendly way to solve such a mistake.

Adding ctrl-z to more operations is appreciated too. And of course a stack mechanism to undo not only the last but several operations would be great too.
Comment 9 Maik Qualmann 2018-11-01 17:28:13 UTC
*** Bug 400551 has been marked as a duplicate of this bug. ***
Comment 10 Maik Qualmann 2018-11-03 21:56:14 UTC
Git commit 27bf9d3c50e8484822ad6358856d696593626aee by Maik Qualmann.
Committed on 03/11/2018 at 21:52.
Pushed by mqualmann into branch 'master'.

add undo button to the trash view to restore items
this restore all deleted items since startup
FIXED-IN: 6.0.0

M  +2    -1    NEWS
M  +89   -17   core/app/views/trashview.cpp
M  +1    -0    core/app/views/trashview.h
M  +5    -0    core/libs/dtrash/dtrashiteminfo.cpp
M  +2    -0    core/libs/dtrash/dtrashiteminfo.h
M  +12   -0    core/libs/dtrash/dtrashitemmodel.cpp
M  +5    -0    core/libs/dtrash/dtrashitemmodel.h

https://commits.kde.org/digikam/27bf9d3c50e8484822ad6358856d696593626aee
Comment 11 Christian Gruber 2018-11-03 22:20:11 UTC
This poses a problem in the other direction: When there is no undo possibility, it is indeed a problem when you accidentally delete a photo. On the other hand, when you "restore all deleted items since startup", it is also a problem: Imagine that you delete 1000 photos one day. Then you hibernate the computer and wake it up one week later, in order to go on editing. You then delete accidentally one picture and perform an undo. As a result, 1001 pictures would be undeleted. With bad luck, this would be unrecognized, and when you recognize it, probably the same 1000 pictures cannot be remembered, throwing away one day of editing.
It would be a much less surprising behavior if the undo was like you are used to it in other programs, undeleting one by one ordered by deletion time.
Comment 12 Maik Qualmann 2018-11-03 23:22:20 UTC
I'll find a solution tomorrow...

Maik
Comment 13 Maik Qualmann 2018-11-04 07:59:47 UTC
Git commit 58ed099e3ed5093ceaec96ddcc00c926f83c7b61 by Maik Qualmann.
Committed on 04/11/2018 at 07:58.
Pushed by mqualmann into branch 'master'.

undo last deleted items and use constant delete time

M  +13   -13   core/app/views/trashview.cpp
M  +2    -2    core/libs/database/utils/ifaces/dio.cpp
M  +20   -12   core/libs/dtrash/dtrash.cpp
M  +14   -8    core/libs/dtrash/dtrash.h
M  +2    -2    core/libs/iojobs/iojob.cpp
M  +8    -1    core/libs/iojobs/iojobdata.cpp
M  +4    -2    core/libs/iojobs/iojobdata.h

https://commits.kde.org/digikam/58ed099e3ed5093ceaec96ddcc00c926f83c7b61
Comment 14 Christian Gruber 2018-11-04 12:47:30 UTC
Is the number of undeleted files configurable or determinable, at least afterwards?
Comment 15 Maik Qualmann 2018-11-04 15:44:11 UTC
I do not understand the question. Every undo step undoes the last deletion, that can only be one file or even thousands of files. Until the trash is empty. This works even after the restart.

Maik
Comment 16 Christian Gruber 2018-11-04 15:55:21 UTC
OK, then I misunderstood this. I thought that the last n files are undeleted. As it seems, n=1, so no problem arises here.
Comment 17 bert 2018-12-23 07:23:18 UTC
Hello Maik,

Thanks for solving this bug. Great work. I will try it asap.
Merry Christmas!
Comment 18 Christian Gruber 2019-06-19 18:32:02 UTC
In digikam-6.2.0-git-20190613T125237-qtwebkit-x86-64.appimage, where can I find it?
Comment 19 Maik Qualmann 2019-06-19 19:27:03 UTC
In the trash view the lower left button labeled "Undo"?

Maik
Comment 20 Christian Gruber 2019-06-20 11:50:26 UTC
What is the "trash view" and how do I activate it?
Comment 21 Christian Gruber 2019-07-10 20:11:40 UTC
I can't find the Undo button anywhere.
Comment 22 Maik Qualmann 2019-07-10 20:24:22 UTC
Created attachment 121451 [details]
undotrash.png

I had deliberately not responded because I could not imagine that it would not be found where the function is "hidden". ((:-)) I left my screenshot in German...

Maik
Comment 23 Christian Gruber 2019-07-10 20:33:19 UTC
Indeed, now I have seen it. I thought that this is again an issue with the different Linux distributions...
By the way, what is the difference between "Rückgängig" and "Wiederherstellen"?
Comment 24 Maik Qualmann 2019-07-10 20:43:09 UTC
With "Rückgängig" (Undo) all files of the last deletion will be restored, which can be 1 or many image(s). With "Wiederherstellen" (Restore), the manually selected images are restored from the table view.

Maik
Comment 25 Christian Gruber 2019-07-10 20:44:37 UTC
And another thing: When you are in the phase of cleaning up, images are deleted with one keypress ("Entf" or "Del"). The undo should also be possible with one keypress (Ctl-Z). Here, you have to stop what you are doing (presumably the light table), go to the trash bin and select "Undo", and then return to what you just did. This completely interrupts the work flow.
Comment 26 Maik Qualmann 2019-07-13 05:41:15 UTC
An undo of all last images is now possible. An undo with Ctrl-Z would presuppose that all other actions (apply metadata) can also be undone. Only deleting images would be illogical. For this purpose, however, already exists a bug entry. I close this bug now.

Maik