https://bugs.kde.org/show_bug.cgi?id=374191
--- Comment #21 from caulier.gil...@gmail.com ---
To comment #13 :
"Moreover, I found some function calls in the code that seem to be superfluous.
I did not remove them, but only set a comment @ODD."
I take a look to the patch #103144. The commented code with @ODD :
+ // @ODD : Why is singleton set to true? resetAlbumsAndTags already clears
the selection.
d->albumSelectors->setPAlbumSelected(album, true);
+ // @ODD : Why is singleton set to true? resetAlbumsAndTags already clears
the selection.
d->albumSelectors->setTAlbumSelected(album, true);
I think this kind of singleton selection exist due to some problem seen while
porting from Qt4 to Qt5. I don't remember exactly.
Check if this argument need to be set to true really, as the model have been
adjusted previously.
Check also where these methods are used outside similarity code to see if this
argument need really to exists.
Gilles Caulier
--
You are receiving this mail because:
You are watching all bug changes.
