Comments on: Musicplayer module for pluck http://jopsen.dk/blog/2009/10/musicplayer-module-for-pluck/ Thoughts mixed with random memory dumps... Fri, 13 Jan 2012 08:11:57 +0000 hourly 1 http://wordpress.org/?v=3.3.1 By: Jonas Finnemann Jensen http://jopsen.dk/blog/2009/10/musicplayer-module-for-pluck/comment-page-1/#comment-12422 Jonas Finnemann Jensen Sun, 09 Jan 2011 10:21:07 +0000 http://jopsen.dk/blog/?p=150#comment-12422 Hmm... Acoording to [1] it's possible to load the variables like this: xspf_jukebox.swf?autoplay=true&playlist_url=playlist.xml&... Or this: xspf_jukebox.swf?loadurl=var.txt&playlist_url=playlist.xml where var.txt contains the variables... What you're doing is obviously the first approach, and if that works better than the second then why not use it... You can mail me the patch my email is jopsen@gmail.com (If you know how to use git format-patch that's probably the easiest, otherwise just send me a list of changes). Or you can create a fork of the project on github, push you changes there and send a pull-request... Thanks for fixing this bug... [1] http://lacymorrow.com/projects/jukebox/xspfdoc.html Hmm… Acoording to [1] it’s possible to load the variables like this:
xspf_jukebox.swf?autoplay=true&playlist_url=playlist.xml&…
Or this:
xspf_jukebox.swf?loadurl=var.txt&playlist_url=playlist.xml
where var.txt contains the variables…

What you’re doing is obviously the first approach, and if that works better than the second then why not use it…

You can mail me the patch my email is jopsen@gmail.com
(If you know how to use git format-patch that’s probably the easiest, otherwise just send me a list of changes).
Or you can create a fork of the project on github, push you changes there and send a pull-request…

Thanks for fixing this bug…

[1] http://lacymorrow.com/projects/jukebox/xspfdoc.html

]]> By: giux http://jopsen.dk/blog/2009/10/musicplayer-module-for-pluck/comment-page-1/#comment-12420 giux Sun, 09 Jan 2011 09:04:53 +0000 http://jopsen.dk/blog/?p=150#comment-12420 perhaps i can send you code because in the textarea i cannot paste it correctly? perhaps i can send you code because in the textarea i cannot paste it correctly?

]]> By: giux http://jopsen.dk/blog/2009/10/musicplayer-module-for-pluck/comment-page-1/#comment-12419 giux Sun, 09 Jan 2011 09:02:53 +0000 http://jopsen.dk/blog/?p=150#comment-12419 hi, the problem was in pluckplayer/pages_site/player.php, when it build the url to pass loadurl parameter to player use vars.php. I don't know reason, but if i pass url in loadurl the player don't work fine, but if i print parameter the player work. You can see this on: http://pluck.ath.cx Excusme for my very bad english, i understand but i don't able to write fine:( .... //$variables = "data/modules/pluckplayer/player/vars.php"; $variables="&activeDownload=".m_v_bool_out($settings["activeDownload"])."&alphabetize=".m_v_bool_out($settings["alphabetize"]); $variables=$variables."&autoload=".m_v_bool_out($settings["autoload"])."&autoplay=".m_v_bool_out($settings["autoplay"])."&repeat=".m_v_bool_out($settings["repeat"]); $variables=$variables."&repeat_playlist=".m_v_bool_out($settings["repeat_playlist"])."&shuffle=".m_v_bool_out($settings["shuffle"])."&crossFade=".m_v_bool_out($settings["crossFade"]); $p_w = $settings["width"]; $p_h = $settings["height"]; ?> <object classid="clsid:d27cdb6e-ae6d-11cf-96b8-444553540000" width="" height="" id="flashObject" align="middle"> <param name="movie" value="?playlist_url=&skin_url=&loadurl=" /> <embed src="?playlist_url=&skin_url=&loadurl=" .... bie hi,
the problem was in pluckplayer/pages_site/player.php, when it build the url to pass loadurl parameter to player use vars.php.
I don’t know reason, but if i pass url in loadurl the player don’t work fine, but if i print parameter the player work.
You can see this on:
http://pluck.ath.cx

Excusme for my very bad english, i understand but i don’t able to write fine:(

….
//$variables = “data/modules/pluckplayer/player/vars.php”;
$variables=”&activeDownload=”.m_v_bool_out($settings["activeDownload"]).”&alphabetize=”.m_v_bool_out($settings["alphabetize"]);
$variables=$variables.”&autoload=”.m_v_bool_out($settings["autoload"]).”&autoplay=”.m_v_bool_out($settings["autoplay"]).”&repeat=”.m_v_bool_out($settings["repeat"]);
$variables=$variables.”&repeat_playlist=”.m_v_bool_out($settings["repeat_playlist"]).”&shuffle=”.m_v_bool_out($settings["shuffle"]).”&crossFade=”.m_v_bool_out($settings["crossFade"]);

$p_w = $settings["width"];
$p_h = $settings["height"];
?>

<object classid="clsid:d27cdb6e-ae6d-11cf-96b8-444553540000" width="” height=”" id=”flashObject” align=”middle”>
<param name="movie"
value="?playlist_url=&skin_url=&loadurl=” />

<embed
src="?playlist_url=&skin_url=&loadurl=”
….
bie

]]> By: giux http://jopsen.dk/blog/2009/10/musicplayer-module-for-pluck/comment-page-1/#comment-12396 giux Sun, 09 Jan 2011 01:09:25 +0000 http://jopsen.dk/blog/?p=150#comment-12396 hi, i've correct the bug if you want i can send you a patch. Giuseppe hi, i’ve correct the bug
if you want i can send you a patch.
Giuseppe

]]> By: Jonas Finnemann Jensen http://jopsen.dk/blog/2009/10/musicplayer-module-for-pluck/comment-page-1/#comment-12383 Jonas Finnemann Jensen Sat, 08 Jan 2011 20:32:26 +0000 http://jopsen.dk/blog/?p=150#comment-12383 Have you tried to change the settings? If that doesn't work try out another theme, some of them can be broken... Else you can take a look into the source, it's not that complicated. You can set extra variables for XSPF Jukebox in player/vars.php Take a look at: http://lacymorrow.com/projects/jukebox/xspfdoc.html and https://github.com/jopsen/pluckplayer/blob/master/player/vars.php Have you tried to change the settings?
If that doesn’t work try out another theme, some of them can be broken… Else you can take a look into the source, it’s not that complicated. You can set extra variables for XSPF Jukebox in player/vars.php
Take a look at:
http://lacymorrow.com/projects/jukebox/xspfdoc.html
and
https://github.com/jopsen/pluckplayer/blob/master/player/vars.php

]]> By: giux http://jopsen.dk/blog/2009/10/musicplayer-module-for-pluck/comment-page-1/#comment-12379 giux Sat, 08 Jan 2011 19:52:35 +0000 http://jopsen.dk/blog/?p=150#comment-12379 hi, i want test your modules on this demo webpage: http://pluck.ath.cx but i've some problem with it. The player work fine if i want listen one song, but don't start next song. For me is a bug. You can help me? Excusme for my bad english Thank you in advance. Giuseppe hi,
i want test your modules on this demo webpage:

http://pluck.ath.cx

but i’ve some problem with it. The player work fine if i want listen one song, but don’t start next song. For me is a bug. You can help me?
Excusme for my bad english
Thank you in advance.
Giuseppe

]]> By: admin http://jopsen.dk/blog/2009/10/musicplayer-module-for-pluck/comment-page-1/#comment-8069 admin Sun, 07 Feb 2010 20:28:09 +0000 http://jopsen.dk/blog/?p=150#comment-8069 Okay, I've fixed the bugs and released the source on http://github.com/jopsen/pluckplayer Enjoy... If you find any bugs or similar feel free to contact me or create an issue at github... Okay, I’ve fixed the bugs and released the source on http://github.com/jopsen/pluckplayer

Enjoy… If you find any bugs or similar feel free to contact me or create an issue at github…

]]> By: admin http://jopsen.dk/blog/2009/10/musicplayer-module-for-pluck/comment-page-1/#comment-7616 admin Sun, 03 Jan 2010 19:37:06 +0000 http://jopsen.dk/blog/?p=150#comment-7616 I think I have the security thing in order... at least on the pages_admin/playlist.php file... Don't know if it should be anywhere else... :) And yeah, I probably need to make the mkdir call recursive... Thanks... will fix it as soon as a get a test install up and running... :) I think I have the security thing in order… at least on the pages_admin/playlist.php file… Don’t know if it should be anywhere else… :)

And yeah, I probably need to make the mkdir call recursive… Thanks… will fix it as soon as a get a test install up and running… :)

]]> By: A_Bach http://jopsen.dk/blog/2009/10/musicplayer-module-for-pluck/comment-page-1/#comment-7614 A_Bach Sun, 03 Jan 2010 19:08:03 +0000 http://jopsen.dk/blog/?p=150#comment-7614 You forget about two things: http://www.pluck-cms.org/docs/dev:modules:security And second - you cant mkdir with a dir and subdir with one operation. Thats why Musicplayer module cant be install in pluck. You forget about two things:

http://www.pluck-cms.org/docs/dev:modules:security

And second – you cant mkdir with a dir and subdir with one operation. Thats why Musicplayer module cant be install in pluck.

]]> By: admin http://jopsen.dk/blog/2009/10/musicplayer-module-for-pluck/comment-page-1/#comment-7556 admin Tue, 29 Dec 2009 19:09:15 +0000 http://jopsen.dk/blog/?p=150#comment-7556 Music Player module uses spyc (an YAML reader) for it's configuration file... The spyc.php files included module requires php5... So my guess is that you're running php4, from the error message it looks like a problem parsing a php5 class... If you're on php4, you may try to replace spyc.php with a the version for php4, it's in the same zip, available from: http://code.google.com/p/spyc/ Note: From the spyc changelog, it appears the php4 version have been generated using the 5-to-4 tool. So I have no idea if it'll work... Music Player module uses spyc (an YAML reader) for it’s configuration file… The spyc.php files included module requires php5…

So my guess is that you’re running php4, from the error message it looks like a problem parsing a php5 class…

If you’re on php4, you may try to replace spyc.php with a the version for php4, it’s in the same zip, available from:
http://code.google.com/p/spyc/
Note: From the spyc changelog, it appears the php4 version have been generated using the 5-to-4 tool. So I have no idea if it’ll work…

]]>