Bug 131736 - Queue Track and Stop Playing After Track do not work with streams
Summary: Queue Track and Stop Playing After Track do not work with streams
Status: RESOLVED FIXED
Alias: None
Product: amarok
Classification: Applications
Component: Playlist (show other bugs)
Version: 1.4.2-beta1
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Amarok Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-08-02 18:45 UTC by Arend van Beelen jr.
Modified: 2006-12-31 15:10 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Arend van Beelen jr. 2006-08-02 18:45:18 UTC
Version:           1.4.2-beta1 (using KDE KDE 3.5.2)
Installed from:    SuSE RPMs

It would be nice and more consistent if functions like Queue Track and Stop Playing After Track worked sensibly in combination with streams.

For example, I'm playing a stream and want the stream to stop after it has finished its current song, so I select Stop Playing After Track. Currently this has no effect at all and is just simply useless. I think it would make sense to just stop playing after the stream has finished its current song in this case. You may even consider to rename the action to Stop Playing After Song to reflect this if you implement this. If you do not want to implement this, please at least remove the option from the context menu.

Something similar can be said for the action Queue Track. If you're currently playing a stream and then queue a track from your playlist, it would make sense if Amarok jumped to the queued track as soon as the stream finishes its current song.

Finally, does it really make sense to have Repeat Track in the context menu of streams? At least the last.fm streams do not allow to put a song on repeat, are there streams that do? Could it be detected whether a stream supports this and make the option available accordingly?

Cheers, and congrats on your upcoming 1.4.2 release, it already appears to gonna be your best release ever ;)
Arend jr.
Comment 1 Martin Aumueller 2006-08-02 19:25:57 UTC
I see a use for these options: many users are connected via dial-up or other unreliable means. For them
- repeat would reconect to the stream when the connection is established again
- stop after would stop playing when the connection is interupted, ...
Comment 2 Seb Ruiz 2006-08-03 01:30:46 UTC
haha, dial up... now that is an answer i would never have thought of :)
Comment 3 Alexandre Oliveira 2006-12-31 15:10:21 UTC
SVN commit 618183 by aoliveira:

If the user is listening to a stream a queue a track on playlist, go to the queued track when current song of stream finishes.
BUG: 131736


 M  +1 -1      playlist.cpp  


--- trunk/extragear/multimedia/amarok/src/playlist.cpp #618182:618183
@@ -2005,7 +2005,7 @@
     {
         if ( m_currentTrack && !trackChanged ) {
             //if the track hasn't changed then this is a meta-data update
-            if( stopAfterMode() == StopAfterCurrent )
+            if( stopAfterMode() == StopAfterCurrent || !m_nextTracks.isEmpty() )
                 Playlist::instance()->playNextTrack( true );
             //this is a hack, I repeat a hack! FIXME FIXME
             //we do it because often the stream title is from the pls file and is informative
Comment 4 Alexandre Oliveira 2006-12-31 15:10:55 UTC
Stop After track was already done.