Closed (fixed)
Project:
Video
Version:
6.x-4.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Reporter:
Created:
11 Aug 2010 at 16:56 UTC
Updated:
29 Sep 2010 at 15:34 UTC
The attached patch makes two changes to code in includes/transcoder.inc.
The first at line 74 rectifies the call by reference error (I forget when this issue occurs but it needs fixing):
public function admin_settings_validate($form, &$form_state) {
return $this->transcoder->admin_settings_validate($form, &$form_state); // removes $ on &$form_state
}
The second fix brings the array_merge() into the loop when searching for video_{transcoder}.inc files.. without this fix, NO transcoder plugins will appear in the list, only those that are present in the /transcoder folder.
| Comment | File | Size | Author |
|---|---|---|---|
| includes_transcoder_inc_zencoder-options-fix.patch | 10.02 KB | robinmofo |
Comments
Comment #1
hypertext200Comment #2
hypertext200Adding to Dev
Comment #3
hypertext200Do you interest to work with the video module, then get an CVS access to the module? If your interest please comment on this and send this to Drupal admins, i would like to add you as a co-maintainer to the video module.
Comment #4
robinmofo commentedI would very much like to help. I'll get in touch with an admin straight away.
Comment #5
avpadernoFYI, I added frobinrobin as co-maintainer of this project.
Comment #6
hypertext200Well, Thanks a lot kiamlaluno
Comment #7
robinmofo commentedTo be closed.
Comment #8
robinmofo commentedComment #9
hypertext200