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
15011	An improvement of GenericForeignKey which behaves badly in some a bit uncommon but possible situations	pinkeen@…	nobody	"The fk_field associated with GenericForeignKey field gets updated upon assigning an object to GFK field and initialization of the parent model (via kwargs). In my opinion it should also get updated when the parent model is saved.

Suppose we have a model (A) with GFK field. We create an instance (b) of some model (B) and assign it to the GFK field of an instance of model ''A'' (a). If ''a'' wasn't saved prior to the assigning then the fk_field will be None when saving this instance of ''A''. If we don't have NOT NULL (null=True) constraints on the fk_field then the ''a'' will be saved with fk_field=NULL thus failing silently (I don't consider this to be a proper nor logical behaviour.

My proposition is to add a hook to pre_save signal of the parent model in which we update the fk_field if not set. If the the GFK field has an object assigned already but it isn't saved yet then we throw an exception.

I am attaching my improved version of the GFK class (via subclassing) - it should be trivial to merge if you deem it worthy.

Regards,
Filip Sobalski"		closed	Contrib apps	1.2		invalid			Unreviewed	0	0	0	0	0	0