Opened 17 years ago

Closed 17 years ago

#6745 closed (wontfix)

AddManipulator shouldn't clear related data

Reported by: curtis@… 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)

manipulators.py.patch (820 bytes ) - added by curtis@… 17 years ago.
Only remove related data on 'change'

Download all attachments as: .zip

Change History (2)

by curtis@…, 17 years ago

Attachment: manipulators.py.patch added

Only remove related data on 'change'

comment:1 by Malcolm Tredinnick, 17 years ago

Resolution: wontfix
Status: newclosed

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