- π¬π§United Kingdom Hebl
I've added a patch which removes the password addition line for those who want to use the 8.x-1.3 version.
- πΊπΈUnited States agn507
Having trouble getting this patch to apply against 1.3. Any chance this can get rolled into 1.x without the patch?
Gathering patches for dependencies. This might take a minute.
- Installing drupal/openid_connect (1.3.0): Extracting archive
- Applying patches for drupal/openid_connect
https://www.drupal.org/files/issues/2023-10-23/openid-connect-remove-pas... β (Do not generate a password when registering a new user)
Could not apply patch! Skipping. The error was: Cannot apply patch https://www.drupal.org/files/issues/2023-10-23/openid-connect-remove-pas... β - Status changed to Needs work
8 months ago 4:36pm 24 October 2023 - π©πͺGermany sanduhrs πͺπΊ Heidelberg, Germany, Europe
This has been committed in #4, no need to patch.
-
sanduhrs β
committed 22a239d4 on 8.x-1.x
Issue #3214539 by Hebl, msno, jcnventura, sanduhrs: Do not generate a...
-
sanduhrs β
committed 22a239d4 on 8.x-1.x
- Status changed to Fixed
8 months ago 4:39pm 24 October 2023 - π©πͺGermany sanduhrs πͺπΊ Heidelberg, Germany, Europe
Has been committed to 8.x-1.x, will be in the next release.
- π©πͺGermany Harlor Berlin
Oh this issue causes a Fatal on Drupal 10 :S
user_password() does not exist in Drupal 10 anymore.
I'm really surprised, that upgrade status did not report this.
The reasons seems to be that the function was found in UserPasswordFixture.php
- π©πͺGermany sanduhrs πͺπΊ Heidelberg, Germany, Europe
Removing the usage of user_password() causes a fatal?
- π¬π§United Kingdom Hebl
Hey @sanduhrs,
Thanks a lot for adding the fix into 8.x-1.x, much appreciated.
Removing the user_password() does not cause a fatal (for me anyway). Having it left in on Drupal 10 however did cause the fatal error.
Thanks again!
Automatically closed - issue fixed for 2 weeks with no activity.