diff options
author | Mike Bayer <mike_mp@zzzcomputing.com> | 2006-04-03 17:46:08 +0000 |
---|---|---|
committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2006-04-03 17:46:08 +0000 |
commit | 7e7407a03ec0b9ce888e3cc4db92fb2ba868a47b (patch) | |
tree | 29a93d3b81b18020c1f747e13ec9f4db8227c969 /lib/sqlalchemy/sql.py | |
parent | c62f10598331d9f5157f942721bf9eb59443fac9 (diff) | |
download | sqlalchemy-7e7407a03ec0b9ce888e3cc4db92fb2ba868a47b.tar.gz |
ack, fixes to the order by stuff from last night
Diffstat (limited to 'lib/sqlalchemy/sql.py')
-rw-r--r-- | lib/sqlalchemy/sql.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/sqlalchemy/sql.py b/lib/sqlalchemy/sql.py index 34f59fd9c..39888fe2a 100644 --- a/lib/sqlalchemy/sql.py +++ b/lib/sqlalchemy/sql.py @@ -1145,14 +1145,14 @@ class TableClause(FromClause): class SelectBaseMixin(object): """base class for Select and CompoundSelects""" def order_by(self, *clauses): - if clauses[0] is None: + if len(clauses) == 0 or clauses[0] is None: self.order_by_clause = ClauseList() elif getattr(self, 'order_by_clause', None): - self.order_by_clause = ClauseList(*(list(clauses)+list(self.order_by_clause.clauses))) + self.order_by_clause = ClauseList(*(list(self.order_by_clause.clauses) + list(clauses))) else: self.order_by_clause = ClauseList(*clauses) def group_by(self, *clauses): - if clauses[0] is None: + if len(clauses) == 0 or clauses[0] is None: self.group_by_clause = ClauseList() elif getattr(self, 'group_by_clause', None): self.group_by_clause = ClauseList(*(list(clauses)+list(self.group_by_clause.clauses))) |