When I try to save programatically group_permissions entity, I get this error.
[error] Drupal\Core\Database\DatabaseExceptionWrapper: SQLSTATE[42S22]: Column not found: 1054 Unknown column 'revision_log' in 'field list': UPDATE {group_permission_revision} SET id=:db_update_placeholder_0, revision_created=:db_update_placeholder_1, revision_user=:db_update_placeholder_2, revision_log_message=:db_update_placeholder_3, status=:db_update_placeholder_4, permissions=:db_update_placeholder_5, revision_default=:db_update_placeholder_6, revision_log=:db_update_placeholder_7...
Seems also it have implications in database logs. We can't store entity revision log massage.
The problem is in entity file (GroupPermission.php) that try to get the revision_log column that is called revision_log_message.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | 3210227-8-revision_log_problem.patch | 5.55 KB | ludo.r |
| #4 | 3210227-4-revision_log_problem.patch | 5.7 KB | lobsterr |
| #4 | interdiff-3-4.txt | 4.82 KB | lobsterr |
| #2 | 3210227-2-revision_log_problem.patch | 1.2 KB | ion.macaria |
Comments
Comment #2
ion.macaria commentedFix patch:
Comment #3
ion.macaria commentedComment #4
lobsterr commentedYou missed other cases. Also I found some issues with routes.
Can you please check my patch and if it is ok I will release it.
Comment #6
lobsterr commentedComment #8
ludo.rUpdating #4 ptach as it contains some test code that breaks the form page.
I created this patch from
1.0.0-alpha4version (we are using this version), so I didn't test with other versions.Comment #9
lobsterr commentedHm, it was already removed here:
https://git.drupalcode.org/project/group_permissions/-/commit/a9e0e5504f35c21926b0d8a27239fb4bceccb0cb
Comment #10
ludo.rYes indeed, but I need to use this patch for now, we are unfortunately stuck with 1.0.0-alpha4 version because of this: #3231592: Group permissions module stuck on version 1.0.0-alpha4
Comment #11
lobsterr commentedI just wonder what is the reason for alpha4? the next versions bring only bug fixes