Bugzilla – Bug 1784
radio streams play command could use rework from server side.
Last modified: 2009-09-08 09:25:03 UTC
i am unable to select radio streams saved in several playlists. the right pane remained empty eventhough i click on the play button. below is a sample stream that did not work. * http://rigel.th2.frontier.fr:8100 i installed the latest nightly. i used a safari browser on a mac running 10.4.
actually, I can't seem to play any item within a playlist. playing the whole playlist is fine.
nevermind. I bothced the code while trying to extract info. songs work fine. streams within playlists seem to require the playlist add command in order to work. This worked fine: http://server:9000/index.html?player=<mac>&p0=playlist&p1=add&p2=http://205.188.234.38:8070/
Created attachment 629 [details] example of how to make this work we can use the itemobj.isRemoteURL tt object to choose the url appropriate for playing remote url's or local tracks. This can be done rather quickly, in time for 6.1. Later, this could be kept or rework Pages.pm to serve up more of the url so that the skins don't have to code so much of it. This example is for fishbone and default skins.
Created attachment 631 [details] updated to include EN and HandHeld Have tested with all 4, and individual urls within playlist will play/add
kdf - can you check this in, if you haven't already? Thanks.
will do.
committed at change 3752. punting post 6.1 for consideration of server rework to make this cleaner
kdf: what more needs to be done here?
I'm not sure really. I thought originally that the server could feed the needed command (addtracks/add or playtracks/play) but that seem to just be more of the problem by being too specific. If the current method used in the skins is ok, we can close this off. They do work.
Dean, If you are fine with the skins handling this (as in Default, EN, HandHeld and Fishbone), I can simply make sure the rest of the included skins are done this way.
Moser and Touch skin done at change 4124. that seems to be all the skins that need it now.