Bug 6533 - Live365 stations accessed from playlist are missing song information
: Live365 stations accessed from playlist are missing song information
Status: CLOSED WORKSFORME
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.0
: PC Other
: P2 normal (vote)
: ---
Assigned To: Andy Grundman
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-01-03 11:52 UTC by Greg Klanderman
Modified: 2008-12-18 11:12 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Greg Klanderman 2008-01-03 11:52:47 UTC
Back on October 8, on the beta forum I asked:

> Will I still be able to put Live365 stations in a playlist?
> Is there any chance I'll finally be able to get the song info when I
> do that?

Andy G responded:

> Absolutely, that was one of the motivations for the rewrite, the old
> plugin did not work right at all when dealing with old stored links
> because they contained the session ID. This should not be an issue
> any more since the URLs no longer have sessions in them.

However, that does not seem to be the case; I continue to only get the song info for Live365 stations when I access them via the Live365 menu, and not when I access them from a playlist.

After noticing that the current playlist seemed to contain a URL of the form "live365://...." when playing a Live365 station, I thought that maybe changing my playlist to refer to those stations in that way (rather than as "http://....") would solve the problem.

That resulted scan errors like this:

| [2008-01-03 14:28:23.3022] Slim::Formats::Playlists::Base::playlistEntryIsValid (122) Warning:
| file:///music/mp3/live365:/www.live365.com/play/wumb919st found in playlist:
|	file:///music/playlists/radio.pls doesn't exist on disk - skipping!

I tracked down the code to the call to Slim::Player::ProtocolHandlers->isValidHandler() from Slim::Music::Info::isRemoteURL().  It looks like Slim::Plugin::Live365::Plugin->initPlugin() is trying to add a Live365 handler:

|	Slim::Player::ProtocolHandlers->registerHandler(
|		live365 => 'Slim::Plugin::Live365::ProtocolHandler'
|	);

so I would have expected a URL of the form "live365://...." to work, but clearly it did not.

Next, I explicitly added the Live365 handler in Slim::Player::ProtocolHandlers.  Rescanning resulted in no errors, however those stations were silently dropped from the playlists containing them.

That's about as far as I want to pursue this right now.  I suspect Andy will know off the top of his head whether this should just work, or should be able to work with a small amount of effort, or is a larger undertaking.
Comment 1 Andy Grundman 2008-01-04 09:56:05 UTC
I recommend saving Live365 stations to your favorites instead of using a playlist.  Click the 'heart' icon or use the Favorite interface via the player UI.
Comment 2 Greg Klanderman 2008-01-07 23:39:15 UTC
Are you saying that it's not intended that you can put a live365 station in a playlist and get the song information?  Or that it's going to be a fair amount of work to fix?  Just wondering what the prognosis is.. thanks.
Comment 3 Greg Klanderman 2008-01-07 23:44:32 UTC
is the "new folder" button under favorites supposed to work?  it brings up a text entry box that I cannot type into.
Comment 4 Andy Grundman 2008-01-08 07:15:55 UTC
As long as the URL is live365://www.live365.com/play/<station> it should work.  Can you try it with the latest nightly?

There is a bug about radio stations in playlists, bug 4092, it may apply here.  That's why I said you should use Favorites instead of Playlists for radio.
Comment 5 Andy Grundman 2008-01-18 14:57:15 UTC
Works for me, closing.
Comment 6 Chris Owens 2008-03-07 09:03:26 UTC
This bug is being closed since it was resolved for a version which is now released!  Please download the new version of SqueezeCenter (formerly SlimServer) at http://www.slimdevices.com/su_downloads.html

If you are still seeing this bug, please re-open it and we will consider it for a future release.