Opened 19 years ago

Closed 19 years ago

Last modified 18 years ago

#1020 closed defect (fixed)

[patch]mutually referential models

Reported by: rjwittams Owned by: Adrian Holovaty
Component: contrib.admin Version:
Severity: normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Subsumes #746.

Allows mutually referential models. The string name is used as an argument to the ForeignKey.
Most field special casing in meta/init.py is moved to individual fields.

I think it should go in the "messing with models" branch.

Attachments (1)

mutually-referential.diff (67.6 KB ) - added by rjwittams 19 years ago.

Download all attachments as: .zip

Change History (7)

by rjwittams, 19 years ago

Attachment: mutually-referential.diff added

comment:1 by ian@…, 19 years ago

this patch looks pretty self contained, and is backward compatible isn't it?

I mean..joe coder will not have to change 1 thing in his code if this patch went into the main branch.
the messing with models branch is about a much larger, incompatible change.

I vote stick it in the main branch, and fix the bugs IF they come.
I've been using #746 for months now with no problems.

comment:2 by rjwittams, 19 years ago

Yes, it is backwards compatible AFAIK. So it could go in trunk.

comment:3 by rjwittams, 19 years ago

Summary: mutually referential models[patch]mutually referential models

comment:4 by Adrian Holovaty, 19 years ago

Status: newassigned

Thanks for this, rjwittams. My plan is to explore the patch and eventually apply it to trunk. But the removing-magic stuff will definitely get its own branch.

comment:5 by Adrian Holovaty, 19 years ago

(In [1591]) Negligible refactoring in django.core.meta.fields and django.contrib.admin.views.main. Taken from #1020. Refs #1020

comment:6 by Adrian Holovaty, 19 years ago

Resolution: fixed
Status: assignedclosed

This is working in magic-removal.

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