Use CallableResolver for \Drupal\user\PermissionHandler::buildPermissionsYaml() instead of the controller resolver

@vbouchet thank you for the recent work on the drupal.org issues. If you could start using gitlab vs uploading patches though. Eventually everything will be on gitlab and patches are no longer auto ran, manually had to trigger those tests. Also gitlab tests take around 10 minutes while drupalCi could be an 1-2 hours. Did nothing wrong but patches are slowly being phased out. Let me know if you have any questions!

But reviewing your patch and deprecation seems correct. Message seems good and all tests updated.

LGTM

Thanks for the headsup @smustgrave. Will try to setup my local for easier work with issue fork. 😉

The Needs Review Queue Bot → tested this issue.

While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)

Created MR from the patch.

Reroll to MR seems fine.

Committed 15ac0b4 and pushed to 11.x. Thanks!

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

The trigger_error here is for deprecating in 10.2.0 and should be 10.3.0. I have made a followup to fix that. 🐛 Correct version in deprecation for #3363424 Active