Bug 180205 - deleted files inserted into dynamic playlists
Summary: deleted files inserted into dynamic playlists
Status: RESOLVED DUPLICATE of bug 171418
Alias: None
Product: amarok
Classification: Applications
Component: general (show other bugs)
Version: 2.0
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Amarok Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-01-10 02:21 UTC by Brandon Rader
Modified: 2009-02-03 01:17 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Brandon Rader 2009-01-10 02:21:20 UTC
Version:           2.0 (using KDE 4.1.3)
OS:                Linux
Installed from:    Ubuntu Packages

I've deleted some songs from my collection, and these still periodically show up in the dynamic playlist, although they do not show up under the collection tab.  They show up on the playlist with a blank album cover, the track number, and song title.  The artist is blank, and the time shows as 0:00.  When it tries to play them, playback just stops until I skip the song.

To reproduce it, delete a chunk of your collection, then start a dynamic playlist.  Just repopulate the playlist until some of these deleted tracks are inserted into the playlist.
Comment 1 Mikko C. 2009-01-10 08:31:29 UTC

*** This bug has been marked as a duplicate of bug 171418 ***
Comment 2 Dan Meltzer 2009-02-03 00:44:38 UTC
SVN commit 920478 by dmeltzer:

Fix tracks being counted twice, and track playback in the engine controller.

The enginecontroller is a mess.  This commit fixes things for me (which probably breaks things for at least one other person...)  Remove slotPrefinishMarkReached (which only called slotAboutToFinish) as it resulted in duplicate stat calculation.  Also, Phonon::MediaSource can have remote media enqueued, and in fact needs to to make playback work for http/lastfm streams. Please give this some testing and if it doesn't break anything I'll backport it to stable in a few days

BUG: 180205
CCMAIL: amarok-devel@kde.org

 M  +12 -21    EngineController.cpp  
 M  +0 -1      EngineController.h  
 M  +7 -0      playlist/PlaylistActions.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=920478
Comment 3 Dan Meltzer 2009-02-03 01:17:10 UTC
Such a misfire :x

*** This bug has been marked as a duplicate of bug 171418 ***