Bug 166437

Summary: Deleting images in Image Editor not working properly
Product: [Applications] digikam Reporter: Gandalf Lechner <gandalflechner>
Component: Database-TrashAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles
Priority: NOR    
Version: 0.10.0   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In: 0.10.0

Description Gandalf Lechner 2008-07-13 12:45:46 UTC
Version:           0.10.0-beta2 (rev.: 831774) (using 4.00.83 (KDE 4.0.83 (KDE 4.1 Beta2), Kubuntu packages)
Compiler:          gcc
OS:                Linux (x86_64) release 2.6.24-17-generic

In the image editor, when you delete an image, the main view is not refreshed, but still shows the erased image. Going forward and backward one image gives an error of the type "Image X could not be loaded", since the file is gone now. Apparantly the deleted image is not removed from the list.

In the album view, I see after deleting an image still the default picture icon in the place where the thumbnail was.
Comment 1 Andi Clemens 2008-08-15 21:46:51 UTC
I can confirm this, actually the image will be removed from the album icon view and also from the thumbbar in the image editor. But if you click on the thumbnail that "replaces" the erased image (for example if you delete the first image, the second image will become the first in the thumbbar), then it will still show the old erased image in the main view of the editor. If you click "last", it displays the last image but clicking "first" after that will load the "ghost" image again, although the selected thumbnail should show another image.
Comment 2 Marcel Wiesweg 2008-08-25 14:44:46 UTC
SVN commit 852158 by mwiesweg:

Manage file deletion better with the help of signals from database.

When deleting from editor, do it asynchronously (if db information is present)
Connect to CollectionImageChangeset to react when a file is deleted
outside of editor.
Fix some wrong indices in the find next/previous image code (now in removeItem),
fixing bug 166437.


 M  +101 -34   imagewindow.cpp  
 M  +3 -0      imagewindow.h