The guys over at the SoundManager2 module are working to add support for the soundmanager2 player in D7. I took the liberty (here: #1146172: Merge with audiofield module) of suggesting to them that they merge their efforts with you here. Do you have any time or interest in supporting other players?
If none of you are interested in merging the two modules, then I might be tempted to look at it. I took a look at the API for audiofield, and it almost looks too easy! :-) It looks like there are just two functions to implement in a module. I can write php, but, unfortunately, I really don't know javascript. Are those two functions all I need?
Thanks again for your work!
Comment | File | Size | Author |
---|---|---|---|
#6 | add_soundmanager2-1146268-6.patch | 4.89 KB | gagarine |
Comments
Comment #1
tamerzg CreditAttribution: tamerzg commentedYes those 2 functions are all you need. Basically you just need to paste embedd code of Soundmanger in 2nd function and replace some arguments with php variables as in given example.
It would be best if the guys at Soundmanager add this to their module, implementing audiofield hooks. This way they could focus on Soundmanager UI, while the background work will be done by Audiofield.
Comment #2
BeaPower CreditAttribution: BeaPower commentedDoes it work now?
Comment #3
tamerzg CreditAttribution: tamerzg commentedDon't think they implemented it yet. You can ask Soundmanager module maintainers at http://drupal.org/node/1146172
Comment #4
infines CreditAttribution: infines commentedThe sound manager 2 module is pretty quiet... However I see in the repository viewer that SM2 functionality has been added to the 6.x version of audiofield, will it be coming to the d7 version too then?
Comment #5
tamerzg CreditAttribution: tamerzg commentedYes it will be ported to D7 in the coming days.
Comment #6
gagarine CreditAttribution: gagarine commentedVoilà... a patch.
It can be better.. using theme function and so. But it works and should be pretty easy to improve.
I also added what I said in #1468796: Send the field to the players so we can display the file's title so I can display the file title.
Comment #7
gagarine CreditAttribution: gagarine commentedI also add $settings360 but no time for a patch right now...
Comment #8
b3nji CreditAttribution: b3nji commentedI'm getting the following errors with the updated code posted in #7:
Line 211:
function audiofield_soundmanager2($player_path, $audio_file,$variables) {
Line 222:
$filename = $variables['file']->filename;
Comment #9
gagarine CreditAttribution: gagarine commentedYou have to use the patch #6 first and after replace the function.
Comment #10
ws.agency CreditAttribution: ws.agency at Websolutions Agency commentedNeeds work due to a latest bug fixes & patches applied.
Comment #11
Daniel.Moberly CreditAttribution: Daniel.Moberly commented7.x version allows for the use of Sound Manager 2. See issue #2615240: Soundmanager 2 Players for discussion on adding additional Sound Manager 2 players.