- Issue created by @Vishal Choudhary
- Status changed to Needs review
over 1 year ago 6:00am 23 June 2023 - First commit to issue fork.
- last update
over 1 year ago 45 pass - @solideogloria opened merge request.
I created a merge request. The only thing I changed from your patch was to add a space after
(int)
, for coding standards.I reviewed the code (looks good to me, as the user ID is cast to int at other places in the module), but I didn't test it, as I don't use that functionality.
- Status changed to RTBC
over 1 year ago 7:21pm 26 June 2023 - First commit to issue fork.
- 🇩🇪Germany gngn
I just noticed that the patches in 🐛 Unable to reset user password on non-ldap accounts Postponed: needs info also include this change (comments #11 and #12).
#3251947 also includes one other change.
-
bluegeek9 →
committed 84a6b3e2 on 8.x-4.x authored by
solideogloria →
Issue #3368890 by Vishal Choudhary: Unable to change admin user password
-
bluegeek9 →
committed 84a6b3e2 on 8.x-4.x authored by
solideogloria →
- Issue was unassigned.
- Status changed to Fixed
about 1 year ago 7:02pm 9 November 2023 Automatically closed - issue fixed for 2 weeks with no activity.