Bug 78303 - Not all (remote) playlist files (.pls) are read (example: boombastic radio)
Summary: Not all (remote) playlist files (.pls) are read (example: boombastic radio)
Status: RESOLVED FIXED
Alias: None
Product: amarok
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Amarok Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-03-23 15:13 UTC by jsvrp.gw
Modified: 2006-06-11 12:32 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 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