Postponed (maintainer needs more info)
Project:
FFmpeg Converter
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Apr 2010 at 12:27 UTC
Updated:
1 Aug 2011 at 16:17 UTC
The install function doesn't install the DB schema properly. There is an obvious error in hook_install() where an undefined $module variable is used. However, fixing that didn't help and I haven't had time to debug it properly.
Comments
Comment #1
3dloco commentedHello zoo33,
I'd like to test out your rules integration patch #531558: rules integration and was wondering what dev version should I use so that it installs properly.
I'm particularly interested in converting images to video and putting the converted video in a separate filefield, where users can also upload additional videos.
Appreciate your help,
KH
[EDIT]Would you be able to post your current working dev version...thanks[/EDIT]
Comment #2
zoo33 commentedHi. That patch is against the 2.x branch. I'll commit it shortly so you'll be able to download the 2.x-dev version and use it right away. Please post any further questions about the Rules integration in that thread.
Comment #3
3dloco commentedThanks a lot, this is working for me now.
Comment #4
zoo33 commentedSorry, but this issue is still about the install function failing, this is not resolved.
Comment #5
3dloco commentedOk thanks, after a fresh install I ran update.php and everything seems to work ok.
Comment #6
zoo33 commentedCommitted a fix.
Comment #8
ñull commentedStill not fixed. After install I get:
user warning: Table 'drupal6.ffmpeg_converter_preset' doesn't exist query: SELECT * FROM ffmpeg_converter_preset t__0 in /srv/www/htdocs/c2/drupal6/sites/all/modules/ctools/includes/export.inc on line 379.No database table is created.
Comment #9
zoo33 commentedSorry, I can't reproduce this problem with the latest 2.x-dev version. Could you please make sure you have the latest version, and make a clean install of the module? If you want to start fresh without reinstalling Drupal, you can simply delete the ffmpeg_converter row in the system table – assuming you're using a testing site.
Comment #10
ñull commentedReturning here to check out things, I see the same error when I use the present dev.
Comment #11
ñull commentedbump
I used the dev version that did not change since February and get the same error. Cannot save presets this way and cannot continue testing this promising module. Sad....
Just after install I get the message:
Work around is to /update.php and manually select update 6000 of this module, which will create the missing table.
Comment #12
zoo33 commentedStill can't reproduce this. Some details:
I installed Drupal and the related modules:
After that I can go to FFmpeg Converter's admin page and add new presets. The table "ffmpeg_converter_preset" is correctly installed.
These are the versions I used:
It would be nice to be able to close this and maybe make a new release. Please check your versions and see if you come up with any new leads... Thanks!
Comment #13
osopolar@ñull maybe you had some errors while installing or updating the 6.x-2.x-dev version of the module so the table was not created. Maybe just reinstall the module (disable + uninstall + install). Have you checked, if the table exists in your database.
I installed the module and the table ffmpeg_converter_preset was created fine. However I was not able to uninstall it correctly because of the missing uninstall hook (#1235812: Implementation of hook_uninstall). Without this a fresh install might fail.