Allow password reset form to 'pull' user from crowd if they have an account there but not in Drupal

Wrong patch file

The last submitted patch, 3: 3369899.patch, failed testing. View results →

Cursory review

1. +++ b/src/Hooks/PasswordReset.php
   @@ -0,0 +1,119 @@
   +final class PasswordReset implements ContainerInjectionInterface {
   ...
   +    if (($role_id = $this->configFactory->get(SettingsForm::CROWD_SETTINGS)->get('verified_role')) &&
   

   Line is a bit long

2. +++ b/src/Hooks/PasswordReset.php
   @@ -0,0 +1,119 @@
   +      $user->addRole($role_id);
   

   Should we bother if the user exists and is blocked?

3. +++ b/tests/src/Functional/PasswordResetTest.php
   @@ -0,0 +1,88 @@
   +    $account->sessionId = $this->getSession()
   ...
   +        ->getOptions(\Drupal::request())['name']);
   

   Whack indentation

Fix for D9 HEAD and fixes #5

Excellent call re the user being blocked

Committed and cut an alpha2

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