Problem/Motivation

grantSingleAuthorization changed from public function grantSingleAuthorization(UserInterface $user, $consumer_mapping): void to public function grantSingleAuthorization(UserInterface $user, $mapping, string $profile_id): void

CommentFileSizeAuthor
#7 remove_php_warning_for_PR_18_#3485136.patch1.09 KBinput

Issue fork authorization_group-3485136

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

karlshea created an issue. See original summary.

karlshea’s picture

karlshea
he/him
Primary language English
Location Minneapolis 🇺🇸
commented
Title: Fix grantSingleAuthorization signature » Fix grantSingleAuthorization/revokeGrants

karlshea opened merge request !18

karlshea’s picture

karlshea
he/him
Primary language English
Location Minneapolis 🇺🇸
commented
Status: Active » Needs review

Also fixed a bunch of stuff from phpcs/phpstan. Will have to live with the deprecated interface warning, using the static methods didn't work when the tests ran (was throwing an error about a missing chained service).

willabby’s picture

willabby commented

After updating site to Authorization 1.5 we experienced same error. Can confirm this patch fixed issue, thank you!

input’s picture

input commented

Currently testing with PR from #3 with Authorization 1.5 and LDAP.
For the moment it looks good.

input’s picture

input commented
StatusFileSize
new remove_php_warning_for_PR_18_#3485136.patch1.09 KB

If you set "Group and Role" to None (basic membership) there will be a warning created when using PR from #3.
Think i fixed this.

karlshea’s picture

karlshea
he/him
Primary language English
Location Minneapolis 🇺🇸
commented

Thanks, @input!

  • karlshea committed 2c8c651f on 2.0.x 
    Issue #3485136 by karlshea, input: Fix grantSingleAuthorization/...
karlshea’s picture

karlshea
he/him
Primary language English
Location Minneapolis 🇺🇸
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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