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.

Comments

hypertext200’s picture

Status: Needs review » Patch (to be ported)
hypertext200’s picture

Adding to Dev

hypertext200’s picture

Do 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.

robinmofo’s picture

I would very much like to help. I'll get in touch with an admin straight away.

avpaderno’s picture

FYI, I added frobinrobin as co-maintainer of this project.

hypertext200’s picture

Well, Thanks a lot kiamlaluno

robinmofo’s picture

Status: Patch (to be ported) » Needs review

To be closed.

robinmofo’s picture

Assigned: Unassigned » hypertext200
hypertext200’s picture

Status: Needs review » Closed (fixed)