playlist - count and totaltime not updated when adding a song

[Eve00000]

Bug Description

When adding a song to a playlist the playlist - count and -totaltime are not updated, both dragging song to playlist and right click.
After restart of Mixxx the counter and time are right.

extra, after de-selection (other song) and re-slecting song the playlist is bold (=song is in playlist)

Version

2.5-alpha-229-g7b3e48544a (main)

OS

Win 10

[ronso0]

Can confirm the playcount issue, repaint on selection change (bold) works though.

Crates are not affected, so it's probably some issue in PlaylistFeature::slotPlaylistTableChanged and/or PlaylistFeature::constructChildModel

[m0dB]

(Thanks @ronso0 for spotting the duplicate bug report)

As I have been watching this behaviour closely while working on #12020 , I am sure this was working correctly not too long ago.

[m0dB]

I am removing the Qt6 tag, as I am sure it worked correctly with Qt6, so I doubt it is Qt version related.

[ronso0]

Can confirm the playcount issue, repaint on selection change (bold) works though.

Crates are not affected, so it's probably some issue in PlaylistFeature::slotPlaylistTableChanged and/or PlaylistFeature::constructChildModel

or

mixxx/src/library/trackset/playlistfeature.cpp

Line 203 in 99bc5c2

QString PlaylistFeature::fetchPlaylistLabel(int playlistId) {

or there's an issue in PlaylistDAO.

[ronso0]

Damn, stupid typo 🤦‍♂️
8b79278

I'll push a fix asap

[ronso0]

but something else is wrong... I'll take a look.

[m0dB]

Thanks for looking into this, @ronso0. If you get stuck, let me know; while I am certainly not an expert on this part of the code base, I have been looking at it quite a bit recently :-)

[Eve00000]

Oh, there(s something going on here :-)
not as easy as you thought?

[ronso0]

I'm still not sure how this broke (currently bisecting), but a simple m_pSidebarModel->triggerRepaint() after updating the tree item label fixes it. Seems setting the label does not make the model emit dataChanged(index).