Bug 229574 - Moving tags doesn't update the "tag counter" next to the tag name
Summary: Moving tags doesn't update the "tag counter" next to the tag name
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Tags-Engine (show other bugs)
Version: 1.3.0
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-05 16:52 UTC by Andi Clemens
Modified: 2012-06-27 10:54 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 1.2.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andi Clemens 2010-03-05 16:52:07 UTC
Version:            (using KDE 4.4.1)
OS:                Linux
Installed from:    Archlinux Packages

The tag counter next to the tag name in the tag folder view is not updated when the tag is moved to another position / parent.

The counter reads "0" until digiKam is restarted.
Comment 1 Marcel Wiesweg 2010-03-06 14:51:45 UTC
Johannes, I remember you have done some changes to AlbumManager to cache the result of the count maps. This should have fixed problems like this?
Comment 2 Johannes Wienke 2010-03-06 16:54:50 UTC
I don't know what's going on here. Need to have a look at this but currently I don't have that much time any more as I started working on my master thesis.
Comment 3 Johannes Wienke 2010-03-06 18:15:05 UTC
SVN commit 1100051 by jwienke:

Update tag count after moving.

Marcel, can this be a performance problem? For me this was nearly instant. The thing is, normally we know the counts before moving, so we could manipulate the count map directly, but this is the more safe way to my mind.

CCMAIL: marcel.wiesweg@gmx.net
BUG: 229574

 M  +2 -4      NEWS  
 M  +3 -0      digikam/albummanager.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1100051
Comment 4 Marcel Wiesweg 2010-03-07 13:22:52 UTC
The tag map should not be changed at all when moving, shouldn't it? So there is no need to update the map from AlbumManager, if I did not miss something.
But yes, the parents in the model need to re-count their children.
Comment 5 Johannes Wienke 2010-03-07 13:26:10 UTC
I don't know. ;) I didn't really look into how this is computed. So you mean this is more an issue of AbstractCountingAlbumModel?
Comment 6 Marcel Wiesweg 2010-03-07 14:25:20 UTC
SVN commit 1100402 by mwiesweg:

No need to regenerate the map (which can be costly) but
just update the state with the stored map.

CCBUG: 229574

 M  +0 -3      digikam/albummanager.cpp  
 M  +15 -3     libs/models/abstractalbummodel.cpp  
 M  +4 -0      libs/models/abstractalbummodel.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1100402