Changeset 7490

Timestamp:

04/27/08 21:27:19 (2 months ago)

Author:

mtredinnick

Message:

Fixed #7098 -- Old-style related-model order_by() syntax was being incorrectly
marked as erroneous. It's just more dangerous and risky, not forbidden. This
commit restores backwards compatibility there.

Files:

• django/trunk/django/db/models/sql/constants.py (modified) (1 diff)
• django/trunk/tests/regressiontests/queries/models.py (modified) (1 diff)

django/trunk/django/db/models/sql/constants.py

@@ -29 +29 @@
-SINGLE = 'single'
-
-\w
+[.\w]
-ORDER_DIR = {
-    'ASC': ('ASC', 'DESC'),

django/trunk/tests/regressiontests/queries/models.py

@@ -655 +655 @@
->>> s = qs.query.as_sql()   # test passes if this doesn't raise an exception.
-
+Bug #7098 -- Make sure semi-deprecated ordering by related models syntax still
+works.
+>>> Item.objects.values('note__note').order_by('queries_note.note', 'id')
+[{'note__note': u'n2'}, {'note__note': u'n3'}, {'note__note': u'n3'}, {'note__note': u'n3'}]
-"""}
-