Opened 16 years ago
Closed 16 years ago
#6745 closed (wontfix)
AddManipulator shouldn't clear related data
| Reported by: | Owned by: | nobody | |
|---|---|---|---|
| Component: | Core (Other) | Version: | dev |
| Severity: | Keywords: | ||
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
I have a model that pre-populates an m2m fields on save. However, the m2m field would not get populated correctly when the model was added through the 'admin' interface. I tracked down the problem to the AutomaticManipulator that clears (the just saved) related data. Clearing data for an 'add' doesn't seem to make sense.
Attachments (1)
Change History (2)
by , 16 years ago
| Attachment: | manipulators.py.patch added |
|---|
comment:1 by , 16 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
Since we're in the process of actively deprecating the whole manipulator framework, replacing it with newforms, this isn't worth changing. It would introduce a backwards incompatibility into an area of code that is only going to exist for providing full backwards compatibility.
If the same problem still exists in newforms-admin once that is completed, then it's worth starting a discussion on django-developers.
Only remove related data on 'change'