Bug 16219 - Not playing entire Classical.com playlist
: Not playing entire Classical.com playlist
Status: RESOLVED FIXED
Product: MySqueezebox.com
Classification: Unclassified
Component: Classical.Com
: Prod
: PC Windows XP
: -- normal (vote)
: ---
Assigned To: Michael Herger
: partner_important
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-05-14 09:14 UTC by Mickey Gee
Modified: 2010-10-12 14:34 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
proposed patch (548 bytes, patch)
2010-08-24 05:12 UTC, Alan Young
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mickey Gee 2010-05-14 09:14:19 UTC
The Squeezebox plays the Classical.com playlists, but at the end of each track either stops or plays it again.

The system does not go on to play the next track in the playlist.

It is important to have the tracks play from whichever track the user starts to play, so the music keeps playing through the list.
Comment 1 Mickey Gee 2010-05-14 09:14:59 UTC
Reported by Classical.com
Comment 2 Chris Owens 2010-06-28 09:23:54 UTC
QA to repro
Comment 3 Chris Owens 2010-06-28 09:24:29 UTC
Alan suggests that some recent changes may have fixed this.
Comment 4 Alan Young 2010-06-29 01:47:57 UTC
I have no problem with play advancing from track to track but I only get 60s of each track. More information needed to be able to reproduce this properly.
Comment 5 Mickey Gee 2010-07-02 12:09:39 UTC
Our Classical.com accounts have been fixed and should no longer play just 60 seconds. I've verified that doesn't happen anymore.

Now that I can play more than 60 seconds, I've been able to play successive tracks. Classical.com says they think that the default repeat mode is repeat track, but in my testing I see that the repeat mode is repeat playlist.
Comment 6 Mickey Gee 2010-07-09 14:00:49 UTC
I think Classical.com might have issues with the action of how playlists are invoked on specific products or in specific situations.

SB Radio:

- Press Play button on Playlist name: Sequences through entire playlist
- Press Play button on track in playlist: Repeats same track

Touch (No Play button on UI)

- Touch playlist name: Same as right arrow; goes to first track on playlist but doesn't play track
- Touch first track on playlist: Plays first track repeatedly. Playlist has only one track on it.

Have asked Classical.com for clarification.
Comment 7 Mickey Gee 2010-08-09 15:01:50 UTC
Here's the reply from classical.com:

Playlists: best setting would be continuous play through to the next tracks, wherever the user starts. Users seldom want to play one track only.

Also on my device, the Touch, it is not possible to click the playlist name. 

I believe this means classical.com wants Touch to adopt the Radio's characteristics for initiating playlists.
Comment 8 Chris Owens 2010-08-23 09:33:27 UTC
Michael notes he believes the behavior is different between touch and radio, and mysb.com and local server.  Alan notes that if 'play all tracks in an album' is set, which is hte default (player specific options in basic settings), then this should work.

Michael notes this appears to have been implemented, but then commented out for no reason.  He will investigate.
Comment 9 Alan Young 2010-08-24 05:10:09 UTC
Yes, the problem is that the Classical OPML entries are missing the "playall" attribute:

             {
               bitrate   => 128,
               duration  => 977,
               image     => "http://www.classical.com/img/coverart/2684362743.jpg",
               items     => [],
               name      => "Th\xE8me et variations in C sharp minor, Op. 73 by Gabriel Faur\xE9",
               on_select => "play",
               play      => "classical://3221232583.mp3",
               type      => "link",
               url       => "http://www.mysqueezebox.com/api/classical/v1/opml/trackInfo?trackId=3221232583",
               value     => "http://www.mysqueezebox.com/api/classical/v1/opml/trackInfo?trackId=3221232583",
             },

whereas they are present for Napster:

             {
               duration  => 226,
               items     => [],
               name      => "Dog Like Vultures by Haste the Day from Attack Of The Wolf King (Deluxe Edition)",
               on_select => "play",
               play      => "napster://track:33423101.wma",
               playall   => 1,
               type      => "link",
               url       => "http://www.mysqueezebox.com/api/napster/v1/opml/trackInfo?track=track%3A33423101&inLibrary=0",
               value     => "http://www.mysqueezebox.com/api/napster/v1/opml/trackInfo?track=track%3A33423101&inLibrary=0",
             },
Comment 10 Alan Young 2010-08-24 05:12:39 UTC
Created attachment 6941 [details]
proposed patch
Comment 11 SVN Bot 2010-08-25 03:39:52 UTC
 == Auto-comment from SVN commit #8715 to the network repo by mherger ==
 == http://svn.slimdevices.com/network?view=revision&revision=8715 ==

Fixed Bug: 16296
Fixed Bug: 16219
Description: fix playall behaviour on albums/playlists for various services (Classical, Deezer, Napster, WiMP). Play all items in list, unless it's a search result.
Comment 12 SVN Bot 2010-10-12 14:34:25 UTC
 == Auto-comment from SVN commit #9229 to the network repo by agrundman ==
 == http://svn.slimdevices.com/network?view=revision&revision=9229 ==

Bug 16219, backport fix from qa to trunk