- π¬π§United Kingdom alexpott πͺπΊπ
FWIW Drupal 10 will throw an exception when a permission does not exist and you save a role. I don't think this change could be made in Drupal 7 because it could break things.
Whenever a user call user_role_grant_permissions if the permission is not valid (not being implemented by any module) Drupal shouldn't try to execute the db_merge since it will incur in a database error.
The function already load all permissions by using $modules = user_permission_get_modules();
We should check if $modules[$name] is valid before trying to execute db_merge.
Patch provided, needs review.
None
None
Closed: outdated
7.0 β°οΈ
Last updated
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
FWIW Drupal 10 will throw an exception when a permission does not exist and you save a role. I don't think this change could be made in Drupal 7 because it could break things.