This only requires two changes to support https://drupal.org/project/media_recorder

hook_audioconverter_allowed_fields needs to return 'media_recorder' so that it shows up on the fields widget page.
The other is dropping the requirement for audiofield; though not a direct requirement it's not nessecary to use this module if you have media_recorder

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

btopro’s picture

Status: Active » Needs review
FileSize
726 bytes

patch

Anonymous’s picture

Wow, thanks btopro. I will try out the patch and report back asap. This is a current feature request for the Media Recorder:

https://drupal.org/node/2164211

btopro’s picture

Glad to hear, hope we can get this more visibility and pushed through

From planet recently -- this is working via patches: https://drupal.psu.edu/content/html5-audio-recording-drupal

Anonymous’s picture

Thanks again btpro, installed the patch and it is working great for new content. For some reason I couldn't get the audioconverter cron update option to run, but that is a minor point for me.

Also, why did you switch to avconv? Do you have any instructions on the install with FFMPEG?

Once again, thank you. This was the number one issue holding me back from widespread usage of the media recorder module!

Anonymous’s picture

May have jumped the gun. The conversion is happening to mp3, but the media recorder field display is still linked to the wav file. Thoughts?

btopro’s picture

Do you have the settings such that it will remove the original file and link to the new one. I have a feature in the rubric module that has settings that work OOTB as long as the patches are applied.

http://drupalcode.org/project/rubric.git/tree/refs/heads/7.x-1.x:/featur...

Anonymous’s picture

Found the problem. I had not setup the correct path for the Audio Converter module settings when using ffmpeg. I set the path to the directory that contained ffmpeg but did not include the program executable. In my case:

I had this:

usr/bin/ffmpeg

and I needed this:
usr/bin/ffmpeg/ffmpeg

Seems to be working perfectly now. This is a huge feature improvement for the Media Recorder module, btopro. Thank you once again for contributing to Drupal!

Anonymous’s picture

One other issue may have cropped up. I set Audio converter to run on cron to convert my older saved wav files but it seems like the Media Recorder field does not change to the new mp3 file after cron runs. Alternatively, I may have just messed up the cron run for audioconverter, not sure.

btopro’s picture

gotcha, haven't tested that; I just do it on the fly.

Anonymous’s picture

I will do more testing to see what is happening, could be user error or an issue with Media Recorder.

tamerzg has agreed to consider a commit for the patch. Great work, this is an extremely useful feature improvement.

sylvaticus’s picture

I can confirm that the two-lines patch in #1 is working, even for cron jobs. Any chance this will be committed to audioconverter ?

ws.agency’s picture

Status: Needs review » Reviewed & tested by the community

Patch in #1 works.

tamerzg’s picture

Pushed to git.

tamerzg’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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