Comment 2 for bug 1014842

Revision history for this message
Max Linke (max-linke) wrote :

I've had the error on Windows/Linux for 1.11-a/1.10/1.09

Ok I've did some triaging today. wTrackTableView::dropEvent() works almost fine(more on that in p.s.)
PlaylistTablemodel::addTracks() seems to works fine the problems lies in playlistdao.
PTM calls trackdao.addTracks first and this works fine but the call to playlistdao.addtracks afterwards fails and returns 0 added tracks, it does not matter if the track is already in the library or not.

p.s. I found these two comments on wtracktableview.cpp

TODO: Filter out invalid URLs (eg. files that aren't supported audio filetypes, etc.)
// XXX: Crappy, assumes all URLs are valid songs. Should filter
              // out invalid URLs at the start of this function.

select() happens only for crates, adding select() in LTM::addTracks() should fix this.