Opened 8 years ago

Closed 8 years ago

#6745 closed (wontfix)

AddManipulator shouldn't clear related data

Reported by: curtis@… Owned by: nobody
Component: Core (Other) Version: master
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:


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) (820 bytes) - added by curtis@… 8 years ago.
Only remove related data on 'change'

Download all attachments as: .zip

Change History (2)

Changed 8 years ago by curtis@…

Only remove related data on 'change'

comment:1 Changed 8 years ago by mtredinnick

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Resolution set to wontfix
  • Status changed from new to 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.

Note: See TracTickets for help on using tickets.
Back to Top