Changes between Initial Version and Version 3 of Ticket #5450


Ignore:
Timestamp:
Feb 28, 2008, 10:58:24 AM (17 years ago)
Author:
Jacob
Comment:

Agreed with Russell -- save_m2m() is very specific to that instance; better behavior would be to raise an error if the object doesn't have a pk. I'm coopting this ticket to handle that issue instead.

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #5450

    • Property Triage Stage Unreviewed → Accepted
    • Property Summary Save_m2m in forms created by form for model should take a instance argument. → save_m2m should raise an exception for unsaved models

  • Ticket #5450 – Description

    initial v3  
    1 {{
    2  
     1{{{
    32 object = form.save(commit=False)
    43 object.foofield = 'something'
    … …  
    76
    87 form.save_m2m(object) # would be better
     8}}}
    99
    10 }}}
     10(See below for more discussion -JKM)
Back to Top