11 Comments »

1. 

   I tried to use your Music Player module. I installed the module. But I do not understand what to do now.
   Look at the website under the button: Muziek. What did I do wrong.

   Greets,
   Ellie Rouwenhorst

   Comment by Ellie Rouwenhorst — December 29, 2009 @ 09:37

2. 

   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…

   Comment by admin — December 29, 2009 @ 11:09

3. 

   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.

   Comment by A_Bach — January 3, 2010 @ 11:08

4. 

   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…

   Comment by admin — January 3, 2010 @ 11:37

5. 

   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…

   Comment by admin — February 7, 2010 @ 12:28

6. 

   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

   Comment by giux — January 8, 2011 @ 11:52

7. 

   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

   Comment by Jonas Finnemann Jensen — January 8, 2011 @ 12:32

8. 

   hi, i’ve correct the bug
   if you want i can send you a patch.
   Giuseppe

   Comment by giux — January 8, 2011 @ 17:09

9. 

   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

   Comment by giux — January 9, 2011 @ 01:02

10. 

    perhaps i can send you code because in the textarea i cannot paste it correctly?

    Comment by giux — January 9, 2011 @ 01:04

11. 

    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

    Comment by Jonas Finnemann Jensen — January 9, 2011 @ 02:21

Leave a comment