Friends,

I am using this module for some projects and I need to update to drupal version 9.

Could you contribute to this compatibility?

Thank you

CommentFileSizeAuthor
#7 3174212-7.patch3.92 KBsuresh prabhu parkala
#5 keycloak-drupal9_updates-3174212-5.patch1.88 KBcgoffin
#4 keycloak-drupal9_updates-3174212-4.patch1.88 KBcgoffin

Issue fork keycloak-3174212

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

guilhermevm created an issue. See original summary.

joe.murray’s picture

joe.murray commented

@guilhermevm There is an auto-created patch created to assist in making this module D9 compatible - see https://www.drupal.org/project/keycloak/issues/3140888

cgoffin made their first commit to this issue’s fork.

cgoffin’s picture

cgoffin commented
StatusFileSize
new keycloak-drupal9_updates-3174212-4.patch1.88 KB

Here you can find a patch for the 1.3 release.

cgoffin’s picture

cgoffin commented
StatusFileSize
new keycloak-drupal9_updates-3174212-5.patch1.88 KB

There was a bug in the patch. Here is the one that does the trick.

lamp5’s picture

lamp5
Primary language Polish
Location Rzeszow
commented
Version: 8.x-1.3 » 8.x-1.x-dev

You should use a dependency injection or MessengerTrait instead of global drupal service class.

suresh prabhu parkala’s picture

suresh prabhu parkala commented
Status: Needs work » Needs review
StatusFileSize
new 3174212-7.patch3.92 KB

Updated patch please review.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Related issues: +#3174212: Update for drupal 9
bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Status: Needs review » Reviewed & tested by the community
bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Related issues: -#3174212: Update for drupal 9 +#3140888: keycloak on Drupal 9

Referenced the wrong issue.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Priority: Normal » Critical
Status: Reviewed & tested by the community » Closed (duplicate)

There is overlap with #3174212: Update for drupal 9 Closing this one in favour of the other issue since the patch there is more complete.

cgoffin opened merge request !1