Re: #4001 - newforms commit=false and m2m data loss

2007-07-21 Thread David Danier
> However, if commit=False, the m2m function is dynamically added to the > form instance. This means that the user can choose to invoke save_m2m > whenever they need. No m2m data loss! Why not add a handler for the post_save-signal of the Model? This way the user doesn't even have to know, that t

#4001 - newforms commit=false and m2m data loss

2007-07-20 Thread Russell Keith-Magee
Hi all, I've been looking at ticket #4001, and the problem with form_for_model and form_for_instance losing m2m data if you use commit=False. The comment in the code has a FIXME, with a comment that we should raise an exception in this case. The ticket has a patch, contributed by SmileyChris, tha