Thanks for all of the fixes in the 2.2 beta version! It made everything a lot easier as far as getting your module to work on my Apache/2.2.4 (Win32) PHP/5.2.3 localhost! I did run into one problem though:
I am not using the FFMPEG-PHP extension for PHP that support exists for in this module. It is not required for the module to function and convert files (I know this because everything works perfectly after I made the change mentioned below).
When I first started trying to convert my files using cron.php, I was getting no output (thumbnails or video). Several print statements later (within the flashvideo_cron function of flashvideo.module to debug), I discovered the problem was coming from where the modules tests that there is not a 'NOCONVERT' substring in the filename, that the max number of failures has not been reached, and that the ffmpeg extension is loaded:
if(!strpos($file->filename, 'NOCONVERT') && ($failures < $max_num_failures) && extension_loaded("ffmpeg")) { .... }
Since I don't have that particular extension installed (seems you guys get that automatically with citrix), this conditional was evaluating to false, and therefore, _perform_ffmpeg() wasn't even given the chance to run. Changing to:
if(!strpos($file->filename, 'NOCONVERT') && ($failures < $max_num_failures)) { .... }
worked & solved this problem for me. Just thought maybe this should be revised a bit for those of use who don't use citrix and therefore might not have ffmpeg-php installed.
Comments
Comment #1
travist commentedThanks for this... This will be fixed in the new version.
Comment #2
travist commentedComment #3
travist commentedVersion 2.2 fixes this problem.
Comment #4
(not verified) commented