Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi
if you have the image module installed and you install this module you get an error thrown by the create table since the table image_dimension already exists.
IMHO this shouldn't render in an errormessage to the user.
Comments
Comment #1
Devin Carlson CreditAttribution: Devin Carlson commentedI can't find any reference to the creation of an image_dimension table in either the core Image module or the Image module.
Which module is creating this table?
Comment #2
filijonka CreditAttribution: filijonka commentedhmm you're absolutly correct I must have misread greatly.
The only module that is creating this is file entity so I did some more checks and first time I install this it works fine but if making it unactive and then activating it again I get this error. Should it really call hook_install at all?
I did try this on a clean site to, D7.17.
Comment #3
jpwester CreditAttribution: jpwester commentedI'm having the same issue which I have been unable to resolve. It seems to be the same as #1856298: Table "image_dimensions" already exists. (file_entity_update_7200). I have upwards of 10 sites using the same codebase, but something is unique about two of the sites that is causing them to fail when I try to upgrade this module with the error above. I'm also upgrading a number of other modules, as well as Drupal core.
Comment #4
ParisLiakos CreditAttribution: ParisLiakos commentedWell maybe we should wrap the db_create_table in a table exists and be done with this weird error
Comment #5
jpwester CreditAttribution: jpwester commentedThat sounds reasonable enough to me. Who knows what is creating the table, but if that table exists, I can't upgrade this module. The only other alternative at this point is to manually delete the table, which isn't really feasible on our production environment. This is a better option than that!
Comment #6
ParisLiakos CreditAttribution: ParisLiakos commentedLets do this. See:
#1447790-31: image_load makes admin/content/media unusably slow
Comment #7
ParisLiakos CreditAttribution: ParisLiakos commentedAdded checks for other calls to db_create_table as well
http://drupalcode.org/project/file_entity.git/commit/7822f79