Auto DJ Playlist with double-entrys is displayed wrong

Bug #796832 reported by Daniel Schürmann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
High
Daniel Schürmann

Bug Description

If you have a track several times in auto DJ Playlist, all this tracks are deleted (only) from View if the first one is played.

Steps to reproduce the problem
1.) Load auto DJ playlist with tree times the same Track
2.) Enable auto DJ

* After playing the track the first time only one song is displayed.
* All songs are played as desired
* if you try to add the same track again it is also not displayed in the View.
* after MIxxx restart, all track are back

It was tested on ubuntu lucid 32 bit with current trunk 2836.
MIxxx 1.9.0 from ppa is not effected.

I have tried to debug the problem and found out that there must be an issue with WTrackTableView and its model PlaylistTableModel. The model has always the correct content but the view suppresses the additional tracks.

If the function PlaylistTableModel::removeTrack() is called from dlgautodj.cpp we have the problem. If it is called from WTrackTableView::slotRemove() evrything works fine.

Tags: autodj library

Related branches

Changed in mixxx:
assignee: nobody → Daniel Schürmann (daschuer)
status: New → In Progress
Revision history for this message
Daniel Schürmann (daschuer) wrote :

This bug also effects other playlist with double-entries. After one of those tracks was played and you try to sort the playlist, the track is only displayed one time.

Changed in mixxx:
status: In Progress → Fix Committed
Revision history for this message
Daniel Schürmann (daschuer) wrote :

The bug was caused by BaseSqlTableModel when the sorting of the table was updated from the track cache.
The double-entries where deleted and only one entry was put back to the table.

Revision history for this message
Daniel Schürmann (daschuer) wrote :

Fix is not commited to trunk yet

Changed in mixxx:
status: Fix Committed → In Progress
Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Is this fixed now in trunk? A bunch of fixes went into the library and playlist code this week.

Changed in mixxx:
importance: Undecided → High
RJ Skerry-Ryan (rryan)
tags: added: autodj library
Revision history for this message
Daniel Schürmann (daschuer) wrote :

m_recordCache is now removed, together with this bug.
I have also tried to reproduce it with lp:mixx 2858. Result: Bug resolved.

Changed in mixxx:
status: In Progress → Fix Committed
jus (jus)
Changed in mixxx:
milestone: none → 1.10.0
milestone: 1.10.0 → none
Revision history for this message
jus (jus) wrote :

Cleanup, cant target a fix to already been published milestone.
Fix released in 1.10.0

Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/5925

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.