When updating to 8.61 the update breaks because the table eu_cookie_compliance_basic_consent already exists.

CommentFileSizeAuthor
#10 table_exists_in_update_hook_8107-3001110-6.patch728 bytesIvanner Mora
#8 table_exists_in_update_hook_8107-3001110-6.patch786 bytesIvanner Mora
#5 table_exists_in_update_hook_8107-3001110-5.patch772 bytesvdenis
#5 table_exists_in_update_hook_8107-3001110-5.patch772 bytesvdenis
#2 table_exists_in_update_hook_8107-3001110-1-8.x.patch1.02 KBtipit

Comments

TipiT created an issue. See original summary.

tipit’s picture

tipit commented
StatusFileSize
new table_exists_in_update_hook_8107-3001110-1-8.x.patch1.02 KB
tipit’s picture

tipit commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: table_exists_in_update_hook_8107-3001110-1-8.x.patch, failed testing. View results

vdenis’s picture

vdenis commented
StatusFileSize
new table_exists_in_update_hook_8107-3001110-5.patch772 bytes
new table_exists_in_update_hook_8107-3001110-5.patch772 bytes

Changed method db_table_exists with Database::getConnection()->schema()->tableExists() because db_table_exists() is deprecated.

vdenis’s picture

vdenis commented
Status: Needs work » Needs review

The last submitted patch, 5: table_exists_in_update_hook_8107-3001110-5.patch, failed testing. View results

Ivanner Mora’s picture

Ivanner Mora commented
StatusFileSize
new table_exists_in_update_hook_8107-3001110-6.patch786 bytes

Updated lines 249 to 251

Status: Needs review » Needs work

The last submitted patch, 8: table_exists_in_update_hook_8107-3001110-6.patch, failed testing. View results

Ivanner Mora’s picture

Ivanner Mora commented
StatusFileSize
new table_exists_in_update_hook_8107-3001110-6.patch728 bytes

Fixed line number

svenryen’s picture

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

  • svenryen committed 7f47d9b on 8.x-1.x authored by Ivanner Mora 
    Issue #3001110 by Ivanner Mora, vdenis, TipiT: Table exists in update...
svenryen’s picture

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

Committed to -dev. Thanks for the help fixing this issue!

Status: Fixed » Closed (fixed)

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