Opened 5 years ago
Closed 5 years ago
#31718 closed Uncategorized (wontfix)
changepassword command should pass update_fields to user.save()
| Reported by: | Marcel Tschopp | Owned by: | Marcel Tschopp |
|---|---|---|---|
| Component: | Core (Management commands) | Version: | 3.0 |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
The changepassword command should pass update_fields=("password",) when saving the user.
Change History (4)
comment:1 by , 5 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
follow-up: 3 comment:2 by , 5 years ago
comment:3 by , 5 years ago
Now that you say it, I think you are right. I also don't see any decent solution to make this configurable or extendable.
Should I close my PR?
Note:
See TracTickets
for help on using tickets.
Not sure this is desirable as it breaks any custom
Usermodel with anupdated_at = models.DateTimeField(auto_now=True)field.If
updated_fieldsis used this potential field won't be updated on a password change through the management command anymore.I'm inclined to 'wontfix' this ticket since
update_fieldswould bring little benefit with a slightly backward incompatible behaviour in this case.