Opened 5 years ago

Last modified 5 years ago

#31297 closed Bug

force_insert not respected by parent class with default primary key — at Initial Version

Reported by: Abhijeet Viswa Owned by: Abhijeet Viswa
Component: Database layer (models, ORM) Version: 3.0
Severity: Normal Keywords: force_insert mti
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Consider the following modesl:

class A(models.Model):
    uuid = models.UUIDField(primary_key=True, default=uuid.uuid4)
    test = models.IntegerField()


class B(A):
    test2 = models.IntegerField()
    pass
b = B()
b.test = 10
b.test2 = 20
b.save(force_insert=True)

will generate the following update/insert statements for PostgreSQL:

UPDATE "save_force_insert_a"
SET "test" = 10
WHERE "save_force_insert_a"."uuid" = '8275d9e0-c0fd-4489-b987-44b9dbe379b4'::UUID;


INSERT INTO "save_force_insert_a" ("uuid",
                                   "test")
VALUES ('8275d9e0-c0fd-4489-b987-44b9dbe379b4'::UUID, 10);


INSERT INTO "save_force_insert_b" ("a_ptr_id",
                                   "test2")
VALUES ('8275d9e0-c0fd-4489-b987-44b9dbe379b4'::UUID, 20);

It's the same even when saving a new instance of a model and the user provides the UUID.

However, in the Model Instance Reference, it says the following:
`
Changed in Django 3.0:
Model.save() no longer attempts to find a row when saving a new Model instance and a default value for the primary key is provided, and always executes an INSERT.
`

It is expected that using a force_insert (and a value is provided explicitly or as a default), force_insert should be respected by the parents as well.
This isn't fully related to #18305. I believe that ticket also talks about AutoField PKs.

Change History (0)

Note: See TracTickets for help on using tickets.
Back to Top