delete() on instances of models without any dependencies doesn't clear PKs.

[Artur]

Deleting any model with no dependencies not updates the PK on the model. It should be set to None after .delete() call.

See Django.db.models.deletion:276-281. Should update the model line 280.

[Mariusz Felisiak]

Reproduced at 1ffddfc233e2d5139cc6ec31a4ec6ef70b10f87f.

Regression in bc7dd8490b882b2cefdc7faf431dc64c532b79c9.

Thanks for the report.

[Mariusz Felisiak]

Regression test.

[Florian Apolloner]

I have attached a simple fix which mimics what ​https://github.com/django/django/blob/master/django/db/models/deletion.py#L324-L326 does for multiple objects. I am not sure if we need ​https://github.com/django/django/blob/master/django/db/models/deletion.py#L320-L323 (the block above) because I think field_updates is only ever filled if the objects are not fast-deletable -- ie ​https://github.com/django/django/blob/master/django/db/models/deletion.py#L224 is not called due to the can_fast_delete check at the beginning of the collect function.

That said, if we want to be extra "safe" we can just move lines 320 - 326 into an extra function and call that from the old and new location (though I do not think it is needed).

[Florian Apolloner]

PR in ​https://github.com/django/django/pull/11179

[Mariusz Felisiak <felisiak.mariusz@…>]

In afc708c:

Fixed #30330 -- Fixed setting of primary key to None during fast-delete.

Regression in bc7dd8490b882b2cefdc7faf431dc64c532b79c9.

[Mariusz Felisiak <felisiak.mariusz@…>]

In b88ffef7:

[2.2.x] Fixed #30330 -- Fixed setting of primary key to None during fast-delete.

Regression in bc7dd8490b882b2cefdc7faf431dc64c532b79c9.

Backport of afc708cf6d047b35db57bd7c55baeffef459d279 from master