video_7.module

Thank you for your work.

Would you please submit your modifications as patch ?
http://drupal.org/diffandpatch

This will make reviewing and testing really easier for us.

Fabio

What module dependencies (or other dependencies on the system, like ffmpeg?) are required for this module_7 to work?

Thanks for the cool additions mayborhome!!

Changes to video.module required to get this to work in Drupal 4.7.

Comment out these functions:
theme_video_metadata() video/plugins/video_optmetadata/video_optmetadata.module on line 110
theme_video_image_teaser() video/plugins/video_image/video_image.module on line 117
theme_video_customfields() video/plugins/video_customfields/video_customfields.module on line 178
theme_video_image_body() video/plugins/video_image/video_image.module on line 139

Also you need to the Player.swf:

>wget http://www.maybor.com/Player.swf

Not sure what's required to make that work yet, still digging.

In video_7.module there's a function called:

661 if ($file = _file_check_upload_perl($field_name)) {

but I can't find _file_check_upload_perl();... commenting out.

Finally gave up when line 1355:

function _video_get_toolkit() {
    $ffmpeg_ext = "ffmpeg";
    if (!extension_loaded($ffmpeg_ext)) {
      return false;
    }

...meant I needed ffmped to be an extention... which I could not accomplish. SO CLOSE!! :-(

please don't close active bugs without discussing.

Closing