How to display other links/fields along with the actual mp3 link? [#981982]

Hello :)

I use the jPlayer module with CCK's Filefield and Views (jPlayer Style Plugin) to create a playlist. Everything works great so far. :)
Now I need to display the node's links "edit" and "delete" along with the actual mp3 field inside the playlist.
Currently this additionally fields are printed inside the mp3-link field.
Maybe I'm simply missing something? :)

If it's simpler I would also override the jplayer.tpl.php and put the needed links there but I couldn't call the $node variable.

Help is much appreciated.

Thank you! :)
Stefan

Comment	File	Size	Author
#2	jplayer_multiple_fields_981982.patch	5.64 KB	stevetweeddale
#1	jplayer_981982.patch	6.26 KB	stevetweeddale

Comments

Comment #1

stevetweeddale commented 15 February 2011 at 17:42

Status	File	Size
new	jplayer_981982.patch	6.26 KB

I wanted this functionality too - the ability to include arbitrary fields in the playlist views seemed to be a big potential functionality/flexibility boost. There's tonnes of use-cases: ecommerce links, taxonomy 'genre/album' fields, node edit/delete links for admins...it seems a shame not use utilize the flexibility that comes from being an HTML based player. Currently all fields get wrapped up together in the label link, so you're limited to just mucking about with the link text. So, here's a first attempt patch at removing this limitation.

In addition to specifying which field(s) you want to provide the audio file url, you can now specify which field to use for the track 'label' (if none is provided, the filename is used). This is the clickable field that selects the track in the player. Any other fields that are then added to the view get rendered as per-usual after the label within the playlist view.

In honesty, it appears the current release doesn't work with views, and needs patching (see #919700: Views Oddity) already. Since I was rewriting some of that code anyway, I've incorporated that fix into this patch.

I tested with both branches of views (in D6) and it seems to work fine - would love to see this make it into the module!

Comment #2

stevetweeddale commented 15 February 2011 at 22:18

Category:	support	» feature
Status:	Active	» Needs review

Status	File	Size
new	jplayer_multiple_fields_981982.patch	5.64 KB

Apologies, I accidentally left in some debugging code in there. Furthermore, I realised a minor change was needed to the js as well for links other than the label to be actually clickable. This one should work.

Comment #3

CandC540 commented 1 March 2011 at 12:55

#2 failed.

Getting white screen of death.

Comment #4

CandC540 commented 1 March 2011 at 12:56

Status:

Needs review

» Needs work

Comment #5

stevetweeddale commented 1 March 2011 at 14:00

Weird, I'm running that patch with drush make on a QA site right now...works fine. I'll check later which version of the module the patch was made against...

Comment #6

stevetweeddale commented 1 March 2011 at 21:01

Version:

6.x-1.0-beta2

» 6.x-1.0-beta3

Hmm, not sure what's going wrong for you, it's working fine here when applied to 1.x-beta3...any more details that might help me reproduce the problem?

I've changed the issue version to reflect the version the patch was against...

Comment #7

CandC540 commented 2 March 2011 at 05:43

Version:

6.x-1.0-beta3

» 6.x-1.0-beta2

I'm not sure why it's not working either. I'm running a site with 4 languages using the Sky theme. Could it be the views translation module? I also have a bunch of other translation modules enabled. The player works before the patch, but after the patch I get the white screen of death. Let me know what other info you might need to reproduce the issue.