diff options
author | Mike Bayer <mike_mp@zzzcomputing.com> | 2006-04-03 17:49:41 +0000 |
---|---|---|
committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2006-04-03 17:49:41 +0000 |
commit | 82104a5d0dd3a1b428298dde98354821115c7c0f (patch) | |
tree | 70782caa4184086e02c54212a73e915fe0f81d13 /lib/sqlalchemy/sql.py | |
parent | 7e7407a03ec0b9ce888e3cc4db92fb2ba868a47b (diff) | |
download | sqlalchemy-82104a5d0dd3a1b428298dde98354821115c7c0f.tar.gz |
further order_by: order_by() with no arguments should not affect the current order_by clause (same with group_by)
Diffstat (limited to 'lib/sqlalchemy/sql.py')
-rw-r--r-- | lib/sqlalchemy/sql.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/sqlalchemy/sql.py b/lib/sqlalchemy/sql.py index 39888fe2a..816fac378 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 len(clauses) == 0 or clauses[0] is None: + if len(clauses) == 1 and clauses[0] is None: self.order_by_clause = ClauseList() elif getattr(self, 'order_by_clause', None): 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 len(clauses) == 0 or clauses[0] is None: + if len(clauses) == 1 and 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))) |