When I upgraded to the new version of the module, drush updb was stuck because the table eu_cookie_compliance_basic_consent already exists in the db. The update hook doesn't check that. This prevents all other updates to run.

CommentFileSizeAuthor
#2 eu_cookie_check_if_table_exists_2993312-2.patch738 bytesjoshahubbers

Comments

JoshaHubbers created an issue. See original summary.

joshahubbers’s picture

joshahubbers commented
StatusFileSize
new eu_cookie_check_if_table_exists_2993312-2.patch738 bytes

This patch adds the db_table_exists check to the update hook.

joshahubbers’s picture

joshahubbers commented
Status: Active » Needs review
poindexterous’s picture

poindexterous commented

Hi JoshuaHubbers,

I ran into the same problem yesterday when I was upgrading from version 7.x-1.23 to 7.x 1.25. Thank you for your patch, it worked and the following updates after eu_cookie_compliance_update_7006 ran.

svenryen’s picture

svenryen commented
Status: Needs review » Reviewed & tested by the community

  • svenryen committed 6e0c87b on 7.x-2.x authored by JoshaHubbers 
    Issue #2993312 by JoshaHubbers: Table exists in update hook 7006 breaks...
svenryen’s picture

svenryen commented
Status: Reviewed & tested by the community » Fixed

Committed to -dev. Thanks for the patch!

  • svenryen committed 6e0c87b on 7.x-1.x authored by JoshaHubbers 
    Issue #2993312 by JoshaHubbers: Table exists in update hook 7006 breaks...
  • svenryen committed ec666e2 on 7.x-1.x
    Merge branch '7.x-2.x' into 7.x-1.x

* 7.x-2.x:
Issue #3038237 by Spokje...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.