Audio Port to 6.x

rayarub - January 18, 2008 - 15:18
Project:Audio
Version:6.x-1.x-dev
Component:audio_attach
Category:task
Priority:critical
Assigned:Unassigned
Status:closed
Description

Is anyone working on a Drupal 6 port yet?

#1

Compactman - January 20, 2008 - 04:13

I'd like this one to be ported as well.

#2

drewish - January 28, 2008 - 06:12

#3

kpaul - February 15, 2008 - 07:56

Any new news? Thanks.

#4

gravedancer - February 22, 2008 - 06:44

Guys, i really need this module ported, is there somebody working on it?

#5

rayarub - March 10, 2008 - 12:17

Would it be possible for a developer to give us some kind of update on this? Thanks.

#6

drewish - March 21, 2008 - 21:06

marked http://drupal.org/node/235588 as a duplicate

#7

smoltron - March 22, 2008 - 06:35

I would appreciate very much if someone is working on this. Any info or comments, please.

#8

djdanska - March 25, 2008 - 13:32

Any sign of a 6.x version of this module?

#9

zilla - March 26, 2008 - 02:03

hmmmm, over 2 months and no response - anybody with skills considering a port as a patch? i have no idea how to do it though would love to see it happen...too bad. there are a LOT of d6 sites that can not go up without audio module because they can't migrate their podcasts and other related items...

#10

smoltron - March 26, 2008 - 11:51

Audio module was the reason why I selected Drupal in the beginning. Now that reason seems to fade away ...

#11

kidzior - March 27, 2008 - 18:48

I'm too looking forward to get this module working. Does anyone knows anything about future of this module?

#12

HorsePunchKid - March 30, 2008 - 05:19
Status:active» needs work

This needs lots and lots of work, but it's a start if anyone wants to continue. Here's a summary of progress:

  • Almost completely passes tests in coder.module.
  • All of the modules should install and uninstall cleanly, including upgrading.
  • The settings forms mostly work.

node/add/audio doesn't work, which is a pretty big impediment to further progress. The node form renders fine, but if you submit it, it just takes you back to the add page as if nothing happened. I've thrown a couple of FIXMEs in where I know something isn't right yet.

Patch is against CVS head.

AttachmentSize
audio-6.x-1.x_port_12.patch 119.28 KB
audio-6.x-1.x_port_12.tgz 194.07 KB

#13

Zephurus - April 9, 2008 - 16:01

Ahhh...any progress on this?

#14

Arancaytar - April 15, 2008 - 08:01

I see the problem with your node submission. hook_submit() has been removed. You need to add it as an extra submit handler to the form now. I'm currently testing a fix; will post a new patch here if it works.

#15

Arancaytar - April 15, 2008 - 08:35

I have just created an audio node using my fix.

The "click to play" link is broken without clean URLs (it produces a link like ?q={site_base}?q=audio/play/1), and the file transfer doesn't actually work (audio_file_transfer() tries to close a non-existing output buffer, and audio/play/n returns 404), but I've been able to save the node, at least.

I'm sorry, but I only have jEdit here and it won't let me print a unified diff. This isn't a "real" patch anyway, but a patch on the new files uploaded above - a "patch-patch".

836a837,839
>  
>   // Register the submit hander.
>   $form['#submit'][] = 'audio_node_submit';
908c911
<  * Implementation of hook_submit().
---
>  * Node submit handler
910,912c913,915
< function audio_submit(&$node) {
<   // If a new file was stored in the session move it into the $node.
<   $nid = ($node->nid) ? $node->nid : 'new_node';
---
> function audio_node_submit($form, &$form_state) {
>   // If a new file was stored in the session move it into the $form_state.
>   $nid = ($form_state['values']['nid']) ? $form_state['values']['nid'] : 'new_node';
914,915c917,918
<     $node->audio_file['file_path'] = $_SESSION['audio_new_file'][$nid];
<     $node->audio_file['newfile'] = TRUE;
---
>     $form_state['values']['audio_file']['file_path'] = $_SESSION['audio_new_file'][$nid];
>     $form_state['values']['audio_file']['newfile'] = TRUE;

#16

HorsePunchKid - April 15, 2008 - 19:59

Excellent, thank you. I will incorporate that fix in the next couple days. Can you repost that diff in the usual patch format (diff -up)?

#17

Arancaytar - April 16, 2008 - 06:01

Unfortunately I don't have a diff utility on the computer I'm currently using, and jEdit will only print the above format.

#18

Shai - April 17, 2008 - 03:37

I'll test this once there is a patch made in regular format.

Thanks so much to HorsePunchKid and Arancaytar!

Shai

#19

scarer - April 18, 2008 - 01:57

Are there any fixes available for 6.2?

#20

Arancaytar - April 18, 2008 - 06:41

HorsePunchKid, are you on this? I don't want to bother diffing all the files without making more substantial changes, but I could try to solve the other problems (the clean URL redirect and the download links, for example).

#21

mohammed76 - April 22, 2008 - 07:38

subscribing.

#22

davebv - April 22, 2008 - 15:36

. I am expecting this to be in drupal6! thank you all!

#23

Designer Dude - April 25, 2008 - 13:42

great work guys. I'm available for testing a beta module when it's ready.

#24

Arancaytar - April 29, 2008 - 14:51

I can start working on this around June, if nobody else takes it up.

#25

steve@hopeworks.org - April 30, 2008 - 17:19

looking forward to a Drupal 6 version, I can't upgrade until this great module is updated. Thanks to those in the community who have the ability and time to make this happen!

#26

artis - May 7, 2008 - 01:40

I made the changes in #15 against the tarball in #12 and re packaged it. I made the changes by hand so it would be good if Arancaytar could look it over and make sure they are right. I'll try installing this in my dev site and see how it works and report back. I'm not the greatest with php code but I'll see if I can work on this a little in my "spare time".

AttachmentSize
audio-6.x-1.x_port_13.tar_.gz 191.83 KB

#27

cinemasaville - May 9, 2008 - 06:25

Tried using the package from #26 & got this message:
Fatal error: Call to undefined function file_check_upload() in /home/cinemasa/public_html/modules/audio/audio_images.module on line 146.

So more work to be done, but thanks for the effort. We all want/need this for 6.

#28

davebv - May 10, 2008 - 10:25

I am getting the same error with the package from #26

#29

sami_k - May 10, 2008 - 21:06

Here's what someone did to the image module to fix this problem, get your hints from there.
http://drupal.org/files/issues/image-d6upgrade-181809-32.patch

#30

davebv - May 15, 2008 - 16:44

I modified the audio_image.module by editing the line which contanied the file_check() function and the problem seemed to dissapear.

- if ($upload = file_check_upload('image')) {
+ $validators = array(
+ 'file_validate_is_image' => array(),
+ );
+ if ($upload = file_save_upload('image', $validators)) {

I am not an expert programmer so it is likely I'm doing something really wrong, and the page still spit some errors about getid3 about not getting the tags for file "". And the audio node is created but nothing can played.

So... any news on this?

#31

wretched sinner... - May 16, 2008 - 01:22

subscribe

#32

davecormier - May 17, 2008 - 17:42

subscribing. have installed #15... not yet working.

#33

pancapangrawit - May 19, 2008 - 07:57

subscribe

#34

stefkeB - May 22, 2008 - 10:00

Are there alternatives?

#35

redaccion - May 23, 2008 - 03:10

does I can help ???

I've 2 years experience in C#... think that I can do something...

#36

cinemasaville - May 23, 2008 - 03:17

That would be great! We're all looking forward to this.

#37

redaccion - May 23, 2008 - 04:22

Sorry, but although I want help, I'm asking permission to drewish...

please, 'u need understand... he's the creator...

#38

cinemasaville - May 23, 2008 - 06:00

I'm sure he'd be fine with it. It is an open source community after all. The point is to make is to make it work. Thanks for your help, I'm really eager to start using this module.. I have a lot of great stuff waiting to go.

#39

thomas23@drupal.org - May 24, 2008 - 09:16

Even though I'm fairly new to drupal and php as I'm developing/ building some band's site I would offer any help I can to port this module to D6. And may it be testing. This is the only module I'm planing in using that is not yet available for D6. Thanks for everyone's effort on this port so far!

#40

drewish - May 25, 2008 - 17:22
Status:needs work» active

i created a DRUPAL-5--2 branch so that the D6 work can happen in HEAD now. I've gone ahead and committed the work that HorsePunchKid and Arancaytar did. it should make it easier to see what subsequent patches change. there's still some big bits to get working so i'll leave this issue open.

#41

drewish - May 25, 2008 - 17:22
Version:5.x-2.x-dev» 6.x-1.x-dev

#42

drewish - May 25, 2008 - 23:13
Category:feature request» task
Priority:critical» normal

i had some time this afternoon and banged around on the update a bit. uploads seem to be working player theme functions are now registered. it actually looks like the module will be a lot smaller when this all done... i'd had a lot of cruft code to get the previews working and it seems like the forms api updates have made a lot of that code unnecessary.

#43

bomarmonk - May 28, 2008 - 02:51

I am testing this out for Drupal 6 and things are looking good. I have imported files and played them successfully. The player works well!

#44

Dignan - May 29, 2008 - 17:45

I'm testing this on Drupal 6.2 and can import audio files and create nodes. However, I can't seem to get any of the players working. I recall in the 5.x version that under Audio Settings was a tab to choose which player to use but I don't see that option now. Thoughts?

#45

thomas23@drupal.org - May 29, 2008 - 22:43

same here (D6.2 also and no play back, player selection or any of that kind)

#46

drewish - May 29, 2008 - 22:45

there are definitely some problems with the menus. not sure about the players, for testing you might try disabling the audio_getid3 module and see if they work then, i haven't gotten into that code yet.

#47

thomas23@drupal.org - May 29, 2008 - 23:26

thanks for your fast reply. I'll report what I find out when I get to including adio into the side I need it for (which might take a couple of days).

#48

drewish - May 29, 2008 - 23:36

thomas23@drupal.org, be very careful with it, it's pretty broken.

#49

bomarmonk - May 30, 2008 - 03:47

My initial problem with the player was that I imported audio into Drupal without first installing the getid3 stuff, so the file was messed up. Either that or the file was incompatible with the player. Once I took care of both of these, the audio played well with a nice player interface.

#50

cinemasaville - May 30, 2008 - 19:05

I'm using it, and I see the player, but cannot get the playlist or images working. Any idea when this functionality might be coming online? Thanks again.

#51

thomas23@drupal.org - May 30, 2008 - 22:15

Thanks for pointing out, drewish, but I'm aware of that. After all, it's for some small site :)

#52

thomas23@drupal.org - June 1, 2008 - 14:43

I don't know if this is only new to me but I'll share it anyway:

There's currently a bug in Firefox 2.0 and people are reporting similar problems in Opera. In IE7 this isn't a problem.

The bug:

If you set the width and height parameters to percentage values, i.e. width="50%" height="50%", the SWF will not render in the bowser window. You can only use absolute pixel values, i.e. width="320" height="240". This happens with standard (Flash 6) embed code and with SWFObject 2.0.

The workaround is to use SWFObject 1.5. For some reason this renders percentage values without any problems in every browser I've tested (I haven't tried it with Opera though). SWFObject 1.5 is at: http://blog.deconcept.com/swfobject/

and from swfobject:

SWFObject detects Flash player versions in these browsers from version 3 and up, and will allow users to interact with your Flash content without ‘activating’ it first.

This I became aware of while testing audio 6.x since the player didn't even show from sf.net in Opera. So I guess this is no miricle any more :)

As I don't know much about audio so far bare with me. But I think I read somewhere around here someone asking why the player needs to be clicked twice before it starts. So if it is not allready this maybe could be integrated into audio. It's MIT licenced.

No harm ment,
CU

#53

thomas23@drupal.org - June 1, 2008 - 14:43

Oh, I nearly forgot, here is something else that might come in handy for audio. Here is a small wrapp-up:

A Flash-based MP3 music player with an autoresume feature! I've added this feature to the popular, free XSPF Music Player software.
This player stores the player's current track and playback position in a Flash cookie. When the user moves between pages, the player goes away, of course. But the player on the new page will see the cookie and pick up where the first player left off.
(...)
My autoresume code respects the stop button.
(...)
2006-12-29: shuffle and autoplay are fully compatible with autoresume. Volume control settings are remembered by autoresume. General bug fixes.
(...)
Lacy Morrow created an enhanced version with shuffle capabilities. My own version is based on Lacy Morrow's and adds the autoresume option.

Cheers

#54

bomarmonk - June 2, 2008 - 06:04

Interestingly enough, I can see the player fine in Firefox and Safari, but in IE 7 it's just a white box.

#55

thomas23@drupal.org - June 2, 2008 - 10:40

Update: I know have successfully imported two albums on another site (not local, no .htaccess allowed, i.e. no clean urls, upload_max_filesize and post_max_size set to 50MB if that matters at all) after I first ran into the situation that the importer messed up tags/albums. After I deleted those and uploaded new files with no blank spaces in their file names I worked fine (I don't know if that's the problem, though). I also see und can use 1pixelOut but there is know options to select players, either. The only thing not working at all I noticed was the playlist module. It does install allright but after deploying the link add to playlist the following error occurs:

warning: array_fill() [function.array-fill]: Number of elements must be positive in /path/to/drupal/includes/database.inc on line 235.
warning: implode() [function.implode]: Bad arguments. in /path/to/drupal/includes/database.inc on line 235.
warning: array_keys() [function.array-keys]: The first argument should be an array in /path/to/drupal/modules/user/user.module on line 500.
user warning: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')' at line 1 query: SELECT p.perm FROM drupal_nptest_role r INNER JOIN drupal_nptest_permission p ON p.rid = r.rid WHERE r.rid IN () in /path/to/drupal/modules/user/user.module on line 500.

Edit: I noticed two more things:

  1. Only in Opera (9.50) on my standart installation on the same server with garland I have to click 1PixelOut twice to start playback. On my "self-made" subtheme of zen it starts right away on the first click (tested with FF 3 and IE 7 also).
  2. Autopath generates funny aliases with a posfix %00 on most(!) titles and album names resulting in 404 error when calling that path. Tags on the files where generated using Windows foobar2000 (a high-quality player if your not familiar) so a render it more likely to have something to do with Windows line-breaks? Just guessing. When editing nodes tag fields on such a node I get this:
    The following errors were encountered, preventing the ID3 tags from being saved:
    - ID3v2 APIC data is not properly structured
    - FormatDataForID3v2() failed

/Edit

I'm considering installing another test installation in D5 to see what should be possible (in the long run). Or is there a more detailed description I haven't gotten my hands on, yet, besides the handbook part? I'm interested in a more gallery like (image module) display of by/album/albumname to show the albums image(s) only once and list the tracks rather li-style, i.e. so it's more like one node rather than a list of nodes. I'm I on the right track to assume this needs views? Any hints appreciated!

Cheers.

#56

shah_waliuallah - June 3, 2008 - 13:21

Hi

When will the new drupal 6.2 audio module be available?

#57

thomas23@drupal.org - June 3, 2008 - 15:38

re: When will the new drupal 6.2 audio module be available?

see #48 from May 30, 2008

be very careful with it, it's pretty broken.

which I can confirm, so I assume it will take a few weeks, still. Correct me, if I'm underestimating, drewish.

#58

shah_waliuallah - June 4, 2008 - 08:31

Thanks thomas

#59

thomas23@drupal.org - June 6, 2008 - 21:02

Small update:

In the meantime I could get my hands on audio.module in more detail and just did a fresh install (meaning deleting the audio folder and replacing it with a fresh version downloaded here plus version 1.7.8b2 of getid3 going through your nice README.txt in detail).

The issue I experienced before (see #55 above) when poof. When I now edit some audio node and save it that before pathauto send to /album/day-in-june%00/lazy%00 now is homed correctly at and links are fine.

The only error persisting is the one about APIC data not properly structured (bug reported as #265493: Getid3 error.

Cheers.

#60

thomas23@drupal.org - June 6, 2008 - 21:47

Small update:

In the meantime I could get my hands on audio.module in more detail and just did a fresh directory install meaning deleting the audio folder and replacing it with a fresh 6.x-1.x-dev plus version 1.7.8b2 of getid3 going through that nice README.txt included in detail.

The issue I experienced before (see #55 above) went poof. When I now edit some audio node that pathauto send to /album/day-in-june%00/lazy%00 before now is homed correctly at /album/day-in-june/lazy after submitting the node edit form. A pitty there is no shortcut in admin/content/node as the image.module's on to "rebuild image thumbnails", something like "reread all audio meta tags". But for me that's fine, and to be honest: When does one really need that? Lucky me the band I'm building the site for hasn't been around for decades ;) I just deleted all audio nodes and reimported them which led to no errors and links are fine. Hurray! Just be warned if someone followes my descriptions that audio files are deleted, too. No errors, though, if you move them manually before actually deleting the node in drupal.

The only error persisting is the one about APIC data not properly structured (bug reported as #265493: Getid3 error).

Cheers and thanks once more for your effort!

#61

liquid crystal - June 8, 2008 - 12:54

subscribe

#62

drewish - June 10, 2008 - 18:27

did a little more work last night. the audio module now depends on the getid3 module.

#63

Netmonger - June 16, 2008 - 00:12

So does anyone actually have this module working with 6.2? For me, on a fresh install with no other additional modules except audio, token, and views, when I try to create an audio node, and I browse to an mp3 and click submit or preview, the file doesnt get attached and error messages show up complaining about NULL filenames.. Any else see this?

#64

thomas23@drupal.org - June 16, 2008 - 09:01

On my site I used autdio's importer which worked fine for me (see above). I have two albums worth of audio nodes and playback works fine with them. Although I haven't done any individual stuff yet like creating "album views" so it's just plain out-of-the-box. Except one block where I tried to include a button player which does show but not play. But maybe it's just not configured right; I haven't worked on it lately.

Oh and theming the player does not work either as described for template.php in the dokumentation. But again, it might also be me.

Cheers.

Btw. Is there any more documentation that I might have missed that the one pointed out from the module's main page?

#65

cinemasaville - June 29, 2008 - 23:43

Drewish,

Any news on the status of this, and when it could be production site ready? I know you're probably pretty busy, but I'm waiting on this thing to launch my site.

Thanks.

#66

drewish - July 3, 2008 - 19:28

sorry i don't, i was hoping that some of the people who did the initial porting would have some time work on it.

#67

dubitable - July 9, 2008 - 01:10

Hi folks, I couldn't find this mentioned anywhere in the issues or in this thread, so I figured I'd mention it. If you'd prefer I start another issue let me know.

This is the 6.x dev version of the module, running on Drupal 6.2, that is currently up on the module page (to be precise 6.x-1.x-dev dated from June 19th).

I get this PHP error when I enable "Attach audio files" in the workflow settings for the Story content type (I'm going to assume this would be the same with any content type):

PHP Fatal error: Call to undefined function audio_prepare() in /var/www/htdocs/sites/all/modules/audio/contrib/attach/audio_attach.module on line 153, referer: http://somesite.com/node/add/blog

Function got renamed?

Anyways, hope this helps...if I have time to dig further and offer a patch I will do so...otherwise thanks for all the work done on this!

Best,
Dave

#68

daniel-san - July 14, 2008 - 04:42

I am also interested in the porting of this module to drupal 6. I'm no programmer, but in the interest of using the module on an upcoming site, I would be happy to assist in some way so that we can get a stable release for the module.

#69

bomarmonk - July 14, 2008 - 15:52

This module works beautifully in Drupal 5 with the first version of views (I downgraded my site from Drupal 6 to 5 for the time being). It's definitely worth it to have the views integration working before anyone uses this with Drupal 6.

#70

dtmatthes - July 16, 2008 - 15:43

Subscribing

#71

curtisfranklin - July 17, 2008 - 23:00

subscribe

#72

onepartscissors - July 23, 2008 - 09:12

subscribe

#73

andypost - July 26, 2008 - 19:30

Fix menus for 6.2 and more: just add marked lines... no time to make patch
Next is something wrong with playlist...

<?php
  $items
['admin/settings/audio/metadata'] = array(
   
'title' => 'Metadata tags',
   
'page callback' => 'drupal_get_form',
   
'page arguments' => array('audio_admin_settings_metadata'),
   
'access arguments' => array('administer audio'), //add this
   
'file' => 'audio.admin.inc', //add this
   
'type' => MENU_LOCAL_TASK,
  );
 
$items['admin/settings/audio/players'] = array(
   
'title' => 'Players',
   
'page callback' => 'drupal_get_form',
   
'page arguments' => array('audio_admin_settings_players'),
   
'access arguments' => array('administer audio'), //add this
   
'file' => 'audio.admin.inc', //add this
   
'type' => MENU_LOCAL_TASK,
  );
?>

#74

drewish - July 26, 2008 - 23:44

i'm guessing that the playlists are totally screwed up... but yeah roll a patch for that and i'll commit it asap

#75

onepartscissors - July 31, 2008 - 13:50

Does anyone have this module working on 6.3 yet? i'm battling, and client would really love this module, they are wanting to podcast audio, so this module's going to be a huge part of that.

Which one should i be using - the one on this page, or the 19 June 08 one on the Module's home page?

thanks,

#76

thepiano - August 18, 2008 - 17:58

subscribe

#77

alpinejag - August 20, 2008 - 15:26

tracking

#78

acochran - August 21, 2008 - 01:21

This is so exciting! It looks like this module isn't stuck in open source limbo any more!

#79

enhornenator - August 23, 2008 - 14:35

Subscribe

#80

thomas23@drupal.org - August 24, 2008 - 19:26
Title:Port to 6.x» Audio Port to 6.x

#81

acochran - August 26, 2008 - 04:03
Title:Audio Port to 6.x» I guess I was wrong

I wasn't looking closely at the dates. I guess this module is stuck in open source limbo. Why isn't this module a priority?

#82

mohammed76 - August 26, 2008 - 08:54
Title:I guess I was wrong» Audio Port to 6.x

@acochran, regardless of what you had to say, this issue is about porting the audio.module to drupal 6.x, so have updated the title again to reflect this fact.

thanks.

#83

smoltron - September 3, 2008 - 07:24

We have been waiting for more than six months for this module to be ready for D6. We are now giving up hope, sorry. Does anyone know if it is possible to downgrade from D6.4 to D5?

#84

mohammed76 - September 3, 2008 - 07:49

yes, unfortunately, this module is taking too long to be ported. But since I am not a coder, I can't be demanding. I know.

however, and to answer smoltron's question, it all depends on your specific case and what you want to achieve with the module. but, Views + CCK can very well have a valid alternative.

#85

aaron - September 6, 2008 - 18:36

here's a patch for the menu items

AttachmentSize
audio-admin-menu-items.patch 1.38 KB

#86

drewish - September 6, 2008 - 19:41

thanks aaron, committed that to HEAD.

#87

KaityUK - September 11, 2008 - 15:23

subscribe

But anyone know of a really basic Audio player to play about a dozen tracks for a bands site running Drual 6.x?

#88

ChristophWeber - September 11, 2008 - 16:54

Second KaityUK.
I'm going to try out the dev release regardless of any issues, but am eagerly awaiting a beta.

#89

pbarnett - September 12, 2008 - 01:41

@KaityUK:

I have the Audio module latest release of 6.x-1.x-dev (2008-Sep-07) working on 6.x at http://amplifires.co.uk/

Haven't tried audio attach, but so far so good...

Pete

#90

drewish - September 30, 2008 - 00:11
Status:active» fixed

closing this since there's plenty of bug specific issues.

--project followup subject--

Anonymous (not verified) - October 14, 2008 - 00:36

Automatically closed -- issue fixed for two weeks with no activity.

#91

Anonymous (not verified) - October 14, 2008 - 00:42
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

#92

4vinoth - November 16, 2008 - 00:09
Component:Code» audio_attach
Priority:normal» critical
Status:closed» active

same Error here.

There are many duplicate nodes related this...

http://drupal.org/node/311284
http://drupal.org/node/272926
http://drupal.org/node/299037
http://drupal.org/node/294317

Call to undefined function audio_prepare() in /sites/all/modules/audio/contrib/attach/audio_attach.module on line 153

any Luck?

#93

drewish - November 25, 2008 - 15:59
Status:active» closed

4vinoth, this was closed for a reason. use the related issues.

 
 

Drupal is a registered trademark of Dries Buytaert.