Needs review
Project:
FFmpeg Wrapper
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
24 Aug 2010 at 08:26 UTC
Updated:
24 Aug 2010 at 12:35 UTC
In ffmpeg_wrapper.install on line 11, there's the following line:
require_once('./ffmpeg_wrapper.module');
This breaks in certain situations like when enabling the module via Drush because the working directory is not the ffmpeg_wrapper module directory. Instead, it should be:
require_once(drupal_get_filename('module', 'ffmpeg_wrapper'));
which will ensure ffmpeg_wrapper.module is loaded regardless of the current working directory. Attached is a patch.
| Comment | File | Size | Author |
|---|---|---|---|
| ffmpeg_wrapper-drupal_get_filename.patch | 770 bytes | mark trapp |
Comments
Comment #1
arthurf commentedFix is committed, thanks for the patch.