#10109 closed (fixed)
Move raw sql from models.fields.related to models.sql.query
Reported by: | David Larlet | Owned by: | Alex Gaynor |
---|---|---|---|
Component: | Database layer (models, ORM) | Version: | 1.0 |
Severity: | Keywords: | ||
Cc: | artagnon, Jannis Leidel, Sergey Belov | Triage Stage: | Accepted |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Otherwise backends which do not rely on sql can't use many-to-many relations as discussed here:
http://groups.google.com/group/django-developers/browse_thread/thread/c3cd2eb4f63aed5d/6e8de9e05424d18f
(do not care with messages from people that doesn't know how to use reply-to on mailing-lists)
Now let's discuss around the proposed patch.
Attachments (5)
Change History (18)
Changed 15 years ago by
Attachment: | patch_django_10109.20090123.diff added |
---|
comment:1 Changed 15 years ago by
Needs documentation: | set |
---|---|
Needs tests: | set |
comment:2 Changed 15 years ago by
Triage Stage: | Unreviewed → Accepted |
---|
This needs to happen as part of a bigger refactor of related fields.
comment:3 Changed 15 years ago by
Cc: | artagnon added |
---|
comment:4 Changed 15 years ago by
Noting here that this(or some variant thereof) is going to be a blocker on having many to many fields work correctly with Multiple Database support(GSOC).
Changed 15 years ago by
Attachment: | patch_django_10109.20090422.diff added |
---|
Update patch against r10618
comment:5 Changed 15 years ago by
Owner: | changed from David Larlet to Alex Gaynor |
---|
For the record:
[15:43] <Alex_Gaynor> david`bgk: bitchin! There's probably no time for that to go in before 1.1. But if you remind me I'll bug the shit out of jacobkm and russ to get it in fterwords
I'll ping you anyway.
comment:6 Changed 14 years ago by
See #11795 for a related request that this ticket may make possible.
comment:7 Changed 14 years ago by
Cc: | Jannis Leidel added |
---|
comment:8 Changed 14 years ago by
Cc: | Sergey Belov added |
---|
Changed 14 years ago by
Attachment: | m2m-refactor.r11683.diff added |
---|
RC1 of the m2m refactor stemming from Alex's GSoC 2009 project
comment:9 Changed 14 years ago by
The patch I have just uploaded is ready for trunk. The code is also available on Alex Gaynor's github branch
My intention is to commit this mid next week. Feedback is welcome.
comment:10 Changed 14 years ago by
milestone: | → 1.2 |
---|---|
Needs documentation: | unset |
Needs tests: | unset |
Changed 14 years ago by
Attachment: | m2m-refactor.r11683.2.diff added |
---|
RC2 of m2m-refactor patch. Incorporates fix for abstract base classes.
comment:11 Changed 14 years ago by
Another small issue: Intermediary models should have 'unique_together': (from_, to),
to match the old behaviour.
Changed 14 years ago by
Attachment: | m2m-refactor.r11705.diff added |
---|
RC3 for m2m-refactor patch. Incorporates unique_together fix.
comment:12 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Initial patch, without any documentation nor tests, design discussion for now