Error on hook_install when invoked from drush site-install

For those in need of a quick workaround, I've created a MR that simply bypasses the code when there is no Role loaded.

This is clearly NOT the final solution, as it will result on divergent results (i.e: after a drush site-install --existing-config your permissions might be different as hook_install won't be executed!)

This is just a workaround for those that need to have a temporary solution. I keep the issue as "Needs work"

I got this error, after installing a minimal install and then running drush site:install --existing-config -y as shared by Philip Norton in Drupal 8: Install Site From Existing Configuration in Drupal 10.1.

[error]  Error: Call to a member function grantPermission() on null in matomo_install() (line 19 of /var/www/html/web/modules/contrib/matomo/matomo.install) #0 [internal function]: matomo_install(true)

As a temporary workaround I also commented out those lines during the install + import of config, after which it completed.

This breaks deployment, so upping Priority.

Hi,

After trying to update one of my website on Drupal 10.1 and reinstalling I also get this issue. And a similar issue for other contrib modules (Fontawesome).

Thanks for the MR. Attaching a patch file from it for Composer usage.

I agree that this is only a workaround and not a final solution. I currently do not have the time to search for a definitive solution.

Maybe switch to hook_modules_installed? adding a dependency on the user module (I tried that for Fontawesome, and it was not a success or I tested too quickly)?

I am in favor of merging and making a new release to unblock people.

@shelane, what do you think?

In a similar issue on a different module, @neclimdul proposed using a helper function, user_role_grant_permissions. See https://www.drupal.org/project/userprotect/issues/3349663#comment-15104566 🐛 Error: Call to a member function grantPermission() on null in userprotect_install() RTBC

In #3349663-7: Error: Call to a member function grantPermission() on null in userprotect_install() → neclimdul found out what is happening. In short, this is caused by the change in 📌 Remove special case of User module install Fixed . Modules are installed first and the default config for the modules is imported later. Since the authenticated user role is default config of the user module, the role doesn't exist yet in the install hook of other modules.

We can add the permissions using user_role_grant_permissions() which is also used in core by media_install() and node_install(). It basically uses the same checks, so we don't have to duplicate that code. We should probably also add a check for the user module existance (or add a dependency on the user module, but that seems a bit silly).

Since this only seems to happen when installing sites from existing config, it shouldn't really matter that the module install hook can't add the permissions to the authenticated user role.

Hi,

Thanks @seanB for the explanations and suggestions.

I have updated the MR.

If people are ok with it I will merge and create a new release when I will have time.

We have tried a patch with the latest changes in the MR and now installation works for us.

We are using a patch created from this MR (at the time of writing d74c9750)
This can be merged for what I can tell.

Thanks for the patch, it works well and fixes the bug, and can be rolled out.

Hi,

Merged manually on 8.x-1.x because of Gitlab CI workflow not ok. I guess due to dependency on PHP module.

Cherry-picked on 2.0.x.

Creating a new release for 8.x-1.x

Thank you @Grimreaper!

Thanks for the new release, it's always nice to have one less patch file and entry in composer.json to juggle with :)

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