Opened 16 years ago

Closed 16 years ago

#7350 closed (fixed)

Multi-table Models and Fixtures

Reported by: Greg Taylor Owned by: nobody
Component: Core (Other) Version: dev
Severity: Keywords: model, inheritance, subclass, fixture
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

It seems like multi-table sub-classed models (not abstract) don't
honor the "pk" being specified in fixtures. For example, I have an
Event class that is sub-classed by a Race model. The Event class is in
an app called "calendar" and the Race model is in an app called
"regattas". I dumpdata regattas, reset both apps, and loaddata the
regattas fixture. It loads properly but all of the PKs are now
sequential, meaning a lot of other models are now pointing at the
wrong regatta primary keys. There were regattas with PK IDs 14, 16, and 18, but
are now 1, 2, and 3. I have confirmed that the fixture contains the
old 14, 16, and 18 primary keys.

The problem is the keys seem to be compacted rather than being loaded
from the fixture. The keys are not always uninterrupted sequences of numbers due to deletes.
So the problem doesn't exhibit itself until you delete, dumpdata, then
try to re-load.

Change History (2)

comment:1 by Greg Taylor, 16 years ago

It looks as if this might be a problem in django.core.serializers. I tried to thumb through it but have absolutely no idea where to start.

comment:2 by Russell Keith-Magee, 16 years ago

Resolution: fixed
Status: newclosed

(In [7600]) Fixed #7350, #7202 -- Fixed serialization for multi-model inheritance, which had multiple problems:

  • Serializers were including all superclass fields in their output. Now only local fields are included.
  • Implicit OneToOne primary keys were not correctly added to the metamodel, so they were always marked to be serialized, even though they were primary
  • Model saving was too aggressive about creating new parent class instances during deserialization. Raw save on a model now skips saving of the parent class.
Note: See TracTickets for help on using tickets.
Back to Top