summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authormike bayer <mike_mp@zzzcomputing.com>2018-09-25 20:15:54 -0400
committerGerrit Code Review <gerrit@ci.zzzcomputing.com>2018-09-25 20:15:54 -0400
commitf109e623f1660bc291c2d042b297eeed587efcd3 (patch)
treeccd36bcaf80b9e3e07748c5edb16db6ec1a4911c /lib
parent54768815c6471c4378c5ac57aa7090c68dc079dd (diff)
parentb7ba3f0d9395236cbf05f830d82f6494163d1dfb (diff)
downloadsqlalchemy-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.py15
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