Bug 78303

Summary:	Not all (remote) playlist files (.pls) are read (example: boombastic radio)
Product:	[Applications] amarok	Reporter:	jsvrp.gw
Component:	general	Assignee:	Amarok Developers <amarok-bugs-dist>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
Latest Commit:		Version Fixed In:

Description jsvrp.gw 2004-03-23 15:13:12 UTC

Version:            (using KDE KDE 3.2.1)
Installed from:    SuSE RPMs

When I try to open (add) http://www.boombasticradio.com/shoutcast/64.pls nothing happens.

Noatun plays it allright.

Cheers,

Jeroen

Comment 1 Mark Kretschmann 2004-03-23 15:25:25 UTC

Here's the file's content, for convenience:

[playlist]
numberofentries=2
File1=http://stream01.prolocation.net:11000
Title1=WWW.BOOMBASTICRADIO.COM - DUAL ISDN/BROADBAND#1 [64kbps]
Length1=-1
File2=http://stream02.prolocation.net:11000
Title2=WWW.BOOMBASTICRADIO.COM - DUAL ISDN/BROADBAND#2 [64kbps]
Length2=-1
Version=2

Comment 2 jsvrp.gw 2004-04-19 22:33:32 UTC

Still not fixed in AmaroK 1.0 beta1.

Cheers,

Jeroen

Comment 3 Max Howell 2004-04-20 00:16:11 UTC

You can insert the streams themselves into the list. It also works fine if you download the playlist and open it locally. I'm confused.

I'll check the failure code.

Comment 4 Max Howell 2004-04-20 05:47:42 UTC

CVS commit by mhowell: 

Seems to fix this bug where a remote playlist wouldn't load. What gets me is I have no idea how any streams were loading before.

CCMAIL: 78303-done@bugs.kde.org


  M +1 -2      playlistloader.cpp   1.66


--- kdeextragear-1/amarok/amarok/playlistloader.cpp  #1.65:1.66
@@ -472,6 +472,5 @@ PlaylistLoader::DownloadPlaylistEvent::m
     //but we need to add .pls/.m3u extension or the Loader will fail
     QString path = m_url.filename();
-    //FIXME KTempfile doesn't allow no prefix AND requires you to add the '.'! whattapileofcrap!
-    KTempFile tmpfile( QString::null, path.right( path.findRev( '.' ) ) ); //use default suffix
+    KTempFile tmpfile( QString::null, path.mid( path.findRev( '.' ) ) ); //use default prefix
     path = tmpfile.name();

Comment 5 jsvrp.gw 2004-04-23 10:57:48 UTC

FYI: This bug only occurs, when trying to play the remote PLS-file (boombastic radio) from within AmaroK, using the Add Media button. When AmaroK is triggered from outside (e.g. from commandline or a konqueror bookmark) it works fine:

#amarok http://www.shoutcast.com/sbin/shoutcast-playlist.pls?rn=8286&file=filename.pls
#[Loader::tryConnect()] connecting to/tmp/ksocket-jeroen/amarok.loader_socket
[amaroK loader] amaroK is already running. Transmitting command line arguments..
[amaroK loader] Loader exiting.
[1]+  Done                    amarok http://www.shoutcast.com/sbin/shoutcast-playlist.pls?rn=8286

...and the stream keeps playing.

I just found this out, sorry for the late notice. Hopefully your fix will fix this.

Cheers and keept up the great work,

Jeroen