diff options
author | Robert Leftwich <rtl@pobox.com> | 2006-01-22 12:08:10 +0000 |
---|---|---|
committer | Robert Leftwich <rtl@pobox.com> | 2006-01-22 12:08:10 +0000 |
commit | 2d31b8fd60a26d64c72510613cb493131bc4b406 (patch) | |
tree | bd3f1752e312406ca21274aef52c3af127d20bcb /lib/sqlalchemy/sql.py | |
parent | 82883285bc8aeb9e0cbea44c62d9a41085758b42 (diff) | |
download | sqlalchemy-2d31b8fd60a26d64c72510613cb493131bc4b406.tar.gz |
order_by and group_by being output in wrong order. Added tests for same.
Diffstat (limited to 'lib/sqlalchemy/sql.py')
-rw-r--r-- | lib/sqlalchemy/sql.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/lib/sqlalchemy/sql.py b/lib/sqlalchemy/sql.py index 49040ec93..c37a8d620 100644 --- a/lib/sqlalchemy/sql.py +++ b/lib/sqlalchemy/sql.py @@ -1119,15 +1119,15 @@ class CompoundSelect(SelectBaseMixin, FromClause): self.use_labels = kwargs.pop('use_labels', False) self.oid_column = selects[0].oid_column for s in self.selects: - s.order_by(None) s.group_by(None) + s.order_by(None) self.clauses = [] - order_by = kwargs.get('order_by', None) - if order_by: - self.order_by(*order_by) group_by = kwargs.get('group_by', None) if group_by: self.group_by(*group_by) + order_by = kwargs.get('order_by', None) + if order_by: + self.order_by(*order_by) def hash_key(self): return "CompoundSelect(%s)" % string.join( [util.hash_key(s) for s in self.selects] + @@ -1196,10 +1196,10 @@ class Select(SelectBaseMixin, FromClause): for f in from_obj: self.append_from(f) - if order_by: - self.order_by(*order_by) if group_by: self.group_by(*group_by) + if order_by: + self.order_by(*order_by) class CorrelatedVisitor(ClauseVisitor): """visits a clause, locates any Select clauses, and tells them that they should |