Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I really do not see any logic in \Drupal\user\Form\UserPasswordForm having user_pass as a form id.
This should follow some naming convension principle and should be 'user_password_reset_form' or at least 'user_password_form'.
Comment | File | Size | Author |
---|---|---|---|
#10 | core-standardize-form-id-2477335-10.patch | 954 bytes | mmatsoo |
#1 | user-password-and-reset-form-ids.patch | 541 bytes | Anonymous (not verified) |
Comments
Comment #1
Anonymous (not verified) CreditAttribution: Anonymous commentedSorry, missed the other form.
Comment #3
markdorisonComment #9
joachim CreditAttribution: joachim as a volunteer commentedThis is a good idea.
But the backwards compatibility policy doesn't say anything about form IDs...
Comment #10
mmatsoo CreditAttribution: mmatsoo at Chromatic commentedPatch re-rolled against 8.7.x branch.
Comment #11
mmatsoo CreditAttribution: mmatsoo at Chromatic commentedComment #12
markdorisonThe patch in #10 looks good and seems to fulfill the mandate of the issue.
Comment #13
johnwebdev CreditAttribution: johnwebdev commentedRe #9 This would break any contrib. or custom module currently altering any of these forms. We should maintain BC and deprecate if we think we should pursue this.
Although the form classes are marked @internal, I think
- https://www.drupal.org/core/d8-bc-policy
is something we should do because "the benefits" of changing this value is not really worth all the breaking changes, even if the "fix" is simple.
This should probably also have a CR?
Comment #14
joachim CreditAttribution: joachim as a volunteer commented> Re #9 This would break any contrib. or custom module currently altering any of these forms. We should maintain BC and deprecate if we think we should pursue this.
Yes
But I don't think there's yet a way to deprecate a form ID?
Comment #15
cilefen CreditAttribution: cilefen commentedIf not, please open an issue for that if none exists and postpone this on it.