https://bugs.kde.org/show_bug.cgi?id=374191
--- Comment #17 from Simon <freisi...@gmail.com> --- (In reply to Mario Frank from comment #16) > [...] > > Hi Simon, > > I am not completely happy with this solution. > But the AlbumTreeView from which the TagFolderView is derived only provides > me access to the list of selected tags as list of Album pointers. Using > overloaded methods is also my preferred way. But to do this, I would either > have to transform the list of Album pointers to a list of TAlbum pointers > which is an unnecessary complexity. Or I will have to adopt the > AlbumTreeView for this. > The latter is the way to go, I think. I agree. Even from the naming it is weird that in TagTreeView the selectedAlbum and albumForIndex methods return TAlbum, while selectedTags returns Album. So changing this seems natural. And selectedTags isn't used so widely, so not much adaption should be necessary. -- You are receiving this mail because: You are watching all bug changes.
