diff options
| author | mike bayer <mike_mp@zzzcomputing.com> | 2018-09-25 20:15:54 -0400 |
|---|---|---|
| committer | Gerrit Code Review <gerrit@ci.zzzcomputing.com> | 2018-09-25 20:15:54 -0400 |
| commit | f109e623f1660bc291c2d042b297eeed587efcd3 (patch) | |
| tree | ccd36bcaf80b9e3e07748c5edb16db6ec1a4911c /lib | |
| parent | 54768815c6471c4378c5ac57aa7090c68dc079dd (diff) | |
| parent | b7ba3f0d9395236cbf05f830d82f6494163d1dfb (diff) | |
| download | sqlalchemy-f109e623f1660bc291c2d042b297eeed587efcd3.tar.gz | |
Merge "Accept multiple expressions for aggregate_order_by order_by"
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/sqlalchemy/dialects/postgresql/ext.py | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/lib/sqlalchemy/dialects/postgresql/ext.py b/lib/sqlalchemy/dialects/postgresql/ext.py index 71fb3cc5b..a588eafdd 100644 --- a/lib/sqlalchemy/dialects/postgresql/ext.py +++ b/lib/sqlalchemy/dialects/postgresql/ext.py @@ -39,6 +39,8 @@ class aggregate_order_by(expression.ColumnElement): .. versionadded:: 1.1 + .. versionchanged:: 1.2.13 - the ORDER BY argument may be multiple terms + .. seealso:: :class:`.array_agg` @@ -47,9 +49,18 @@ class aggregate_order_by(expression.ColumnElement): __visit_name__ = 'aggregate_order_by' - def __init__(self, target, order_by): + def __init__(self, target, *order_by): self.target = elements._literal_as_binds(target) - self.order_by = elements._literal_as_binds(order_by) + + _lob = len(order_by) + if _lob == 0: + raise TypeError("at least one ORDER BY element is required") + elif _lob == 1: + self.order_by = elements._literal_as_binds(order_by[0]) + else: + self.order_by = elements.ClauseList( + *order_by, + _literal_as_text=elements._literal_as_binds) def self_group(self, against=None): return self |
