- Issue created by @RoSk0
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 9:08pm 1 May 2023 - last update
over 1 year ago 101 pass - last update
over 1 year ago 99 pass, 2 fail The last submitted patch, 2: openid_connect-3357538-2-test-only.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- 🇳🇿New Zealand ericgsmith
Initial thought - would the same bug would be present on the user form when the current user is an admin and they are editing another user without a linked account? It feels unrelated to the specific form id.
It seems like an incorrect assumption in the module that the form is for the current user.
The comment in the existing code "// Whether the current user is allowed to change its password." is not checking that at all - its checking the currently logged in user.
Could you do something like this?
// Whether the user's password is changeable its password. $account = $form_state->getFormObject()->getEntity(); if (\Drupal::service('openid_connect.openid_connect')->hasSetPasswordAccess($account)) { return; }
- Assigned to RoSk0
- Status changed to Active
over 1 year ago 10:33pm 1 May 2023 - Issue was unassigned.
- Status changed to Needs review
over 1 year ago 10:55pm 1 May 2023 - last update
over 1 year ago 99 pass, 2 fail - last update
over 1 year ago 101 pass - 🇳🇿New Zealand RoSk0 Wellington
Improved patch version.
Thanks for the pointer Eric!
The last submitted patch, 7: openid_connect-3357538-7-test-only.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- Status changed to RTBC
over 1 year ago 11:15pm 1 May 2023 - 🇳🇿New Zealand ericgsmith
Change and test looks good to me and resolves the issue. Changing to RTBC