Delete pictures and folders are not removed from table-view.
Steps to Reproduce:
1. Select a picture
2. Delete it or move it to trash
Preview jumps to next picture but the deleted picture is not removed from preview / table view. When reselecting delete picture an error message is displayed saying picture does not exist.
Deleted pictures should be removed from of lists / table views or light room views.
This feature is know unusable for productive use.
I remember we had this problem in an earlier version. No need to reintroduce old bugs!
Digikam was a good working horse. Was! Would be nice if you make less but better tested changes.
Thanks for your feedback.
I confirm this bug. I shall look into it.
I think there is some inherent problem with updating the views. In all the three modes, there is NO code to handle the updation of the views. There is only code for moving the selection to the next image. I can reproduce it in icon, preview and table mode. See if you can do it as well.
See the relevant code here :
Also, there is a 'signalItemsChanged()' in TableView, which by name, I thought would take care of this. But there is NO code to catch this signal in the entire project. Am I missing anything here?
Table View have been written by Michael. I CC him...
signalImagesChanged is caught here:
Deletion works fine on my system using digiKam 3.2 and I think I made hardly any changes after that version, so tableview code is probably not be to blame here. Also, tableview mostly monitors the ImageModel, which takes care of deletion on its own. Maybe using git bisect may help to find the problem...
Of course I mean signalItemsChanged ;-)
Can you please update your digiKam to git/master and see if you can reproduce the bug?
Current git/master works fine. I didn't realise that I was on dev/tagmngr branch.
@Veaceslav, by git bisect, I have been able to narrow down the problem to these two commits.
Kindly see if you can reproduce the bug. If yes, then fix it. :-)
Thank you Smit, I was able to track it down. It was only my mistake when porting to QList, I didn't find any other bugs related to this issue.
And I can't reproduce this bug anymore...
Git commit 606db1c6a50ab5644f6870d6050f9bb49911197d by Smit Mehta.
Committed on 11/08/2013 at 23:58.
Pushed by smitmehta into branch 'master'.
M +2 -1 NEWS
Well, this is odd... after all is not necessary to change something in TableView to break it.
Also, what I learnt from tracking the bug on my branch, that changes in ImageAlbumModel can be a source of bugs...
P.s. Anyone with Suse rpms that can confirm this bug?