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
24228	Signals have unclear & inconsistent transaction handling	Ryan Gaffney	nobody	"Hello

The [https://docs.djangoproject.com/en/1.7/ref/signals/ signals documentation] says nothing about transactions.

Looking in deletion.py, [https://github.com/django/django/blob/fb48eb05816b1ac87d58696cdfe48be18c901f16/django/db/models/deletion.py#L282-L288 pre_delete and post_delete are within a transaction].

In base.py, however, [https://github.com/django/django/blob/378a0d27d630386c1b0a53d4a2d48be767453198/django/db/models/base.py#L717-L719 pre_save] and [https://github.com/django/django/blob/378a0d27d630386c1b0a53d4a2d48be767453198/django/db/models/base.py#L730-L732 post_save] are ''not'' within a transaction.

Whether this inconsistency is a bug is beyond my Django knowledge to determine, but regardless, '''it should be clear in the documentation what signals are within transactions'''."	Bug	new	Database layer (models, ORM)	dev	Normal		transaction, signals, documentation		Accepted	0	0	0	0	0	0