#27351 closed New feature (duplicate)
Add support for composite fields
Reported by: | Victor Porton | Owned by: | nobody |
---|---|---|---|
Component: | Database layer (models, ORM) | Version: | 1.10 |
Severity: | Normal | Keywords: | |
Cc: | porton@… | Triage Stage: | Unreviewed |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
I want a "composite" Django fields which corresponds to two (rather than one) database columns.
In fact the composite field should consist of two ForeignKey fields.
Otherwise it should behave like a normal field. For example it can be used in a form to input data.
It seems that Django 1.10.1 does not support this. I may try to write a patch. Any advice?
Change History (6)
comment:1 by , 8 years ago
Cc: | added |
---|
comment:2 by , 8 years ago
Resolution: | → duplicate |
---|---|
Status: | new → closed |
comment:3 by , 8 years ago
Resolution: | duplicate |
---|---|
Status: | closed → new |
comment:4 by , 8 years ago
#373 isn't about multiple primary keys. It's about a single primary key that is split across several fields in the database, also called a composite primary key. A well-designed, framework-level solution to the problem you described would solve #373 and vice versa. So I agree with Tim when he says it's a duplicate.
Outside of primary keys (and perhaps other relational fields, you didn't give enough details about your use case to tell on which side it falls) you can easily manage this at the application level: implement a read-write property to proxy access to the underlying fields and you're done. So the problem that needs solving in Django is composite primary keys. The solution is to introduce a composite fields, which is what you described, as far as I understand.
Since you asked for advice:
- When the person who's paid to keep tabs on all development efforts tells you where to start, pay attention instead of brushing off their advice in less than 5 minutes, or else don't ask for advice in the first place.
- The order of magnitude of the work required to fix this is likely hundred hours for an experienced contributor. If it was significantly easier, I think it would have been done. The two GSoC devoted to this problem didn't suffice. No offense meant, but you're talking about a well understood problem and you don't describe it in a way that acknowledges previous efforts in this area.
- To tackle this, in addition to #373, everything Michal Petrucha wrote about his GSoCs, and everything Thomas Stephenson wrote when he formalized DEP 191, you should review related discussions on the django-developers mailing list and figure out which parts of the design are solved and which parts aren't, what code was written and why it couldn't be merged. The meta refactor may help; if memory serves the interactions with composite fields came up in the mailing-list discussions.
- The next step would be to revise DEP 191 and post the resulting proposal to django-developers.
- Your description mixes up form fields and model fields. If the difference is unclear to you, you need to familiarize yourself with the Django code base before tackling this. If the difference is clear to you, then you should pay attention to using accurate terms.
comment:5 by , 8 years ago
I have implemented it:
https://github.com/vporton/django-composite-fields
Please help to improve it and then add it to Django core.
comment:6 by , 8 years ago
Resolution: | → duplicate |
---|---|
Status: | new → closed |
Summary: | A Django field consisting of several fields → Add support for composite fields |
I'll happily reopen the ticket if someone makes a serious proposal that doesn't also solve #373.
It's not easy. See #373 for discussion and past attempts.