id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
7217	ModelForm save method should take an optional dictionary.	Peter of the Norse <RahmCoff@…>	nobody	"The only time we use {{{save(commit=False)}}} is when there are fields in the model that aren’t in the form. There are a lot of places where it looks like 
{{{
form = SomeForm(request.POST)
instance = form.save(commit=False)
instance.other_field = ""some value""
instance.another_field = calulated_value()
instance.save()
}}}

There are literally dozens of places where I have that, but no other places that use {{{commit=False}}}. It would be better if there was a way to pass a dictionary of values like {{{form.save(extra = {""other_field"": ""some value"", ""another_field"": calculated_value()})}}}


"		closed	Uncategorized	dev		wontfix			Unreviewed	0	0	0	0	0	0