Sorry, but my update of the database fails:
/update.php

Modul forum_access
Aktualisierung #8101

Fehlgeschlagen: Drupal\Core\Database\DatabaseExceptionWrapper: SQLSTATE[HY000]: General error: 1709 Index column size too large. The maximum column size is 767 bytes.: ALTER TABLE {forum_access} ADD PRIMARY KEY (`tid`, `rid`); Array ( ) in forum_access_update_8101() (Zeile 77 in /pathtodrupal/modules/contrib/forum_access/forum_access.install).

Updating from Forum Access alpha0 to alpha1
ACL 8.x-1.x-dev
Drupal 8.7.7
PHP 7.2.19
MySQL 5.6.19

Comments

d4t3r created an issue. See original summary.

drupalbubb’s picture

Issue summary: View changes
drupalbubb’s picture

Issue summary: View changes
salvis’s picture

Thank you for your report, I'm looking into it.

salvis’s picture

Status: Active » Needs review
StatusFileSize
new1.95 KB

It turns out that the width of the tidrid field in the {forum_access} table is too big. It was 255, but according to Role IDs are now machine readable strings it should be 64.

This patch will fix the width and redo in update 8102 the work that update 8101 should have done.

Status: Needs review » Needs work

The last submitted patch, 5: forum_access-fix-tid-width-3084233-5.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

  • salvis committed b278671 on 8.x-1.x
    Issue #3084233 by salvis: Update 8101 (#3002918) can fail; fix rid width...
salvis’s picture

Status: Needs work » Fixed

I'm surprised that [3002918-7] came back with "Build successful". Unfortunately, we don't have any tests at this point, but I through the testbot would at least try installing the module, and then it should have produced your error IMO. The testbot uses MySQL 5.5, and I'm using 5.7.24, so your 5.6.19 is somewhere in between. I guess we'll never know for sure.

If you've already run Update 8101, there's no damage in your database. Just update to the -dev version (or the upcoming ALPHA2) and run Update 8102, which will fix the rid size and create the primary key and index.

drupalbubb’s picture

Update to alpha2 works. Thanks

salvis’s picture

Great, thanks!

Status: Fixed » Closed (fixed)

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