diff options
author | Mike Bayer <mike_mp@zzzcomputing.com> | 2010-08-21 19:38:28 -0400 |
---|---|---|
committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2010-08-21 19:38:28 -0400 |
commit | ac9eb5c9c3bc33c38eff5407fa4724c9277ba342 (patch) | |
tree | bdadc6eb2196ca789df900de566e9b108dbf0736 /lib/sqlalchemy/sql/expression.py | |
parent | 61562e58435efddda19d15c61afd6e5326745a31 (diff) | |
download | sqlalchemy-ac9eb5c9c3bc33c38eff5407fa4724c9277ba342.tar.gz |
- Similarly, for relationship(), foreign_keys,
remote_side, order_by - all column-based
expressions are enforced - lists of strings
are explicitly disallowed since this is a
very common error
Diffstat (limited to 'lib/sqlalchemy/sql/expression.py')
-rw-r--r-- | lib/sqlalchemy/sql/expression.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/sqlalchemy/sql/expression.py b/lib/sqlalchemy/sql/expression.py index a7f5d396a..6f593ab48 100644 --- a/lib/sqlalchemy/sql/expression.py +++ b/lib/sqlalchemy/sql/expression.py @@ -1037,12 +1037,12 @@ def _no_literals(element): else: return element -def _only_column_elements(element): +def _only_column_elements(element, name): if hasattr(element, '__clause_element__'): element = element.__clause_element__() if not isinstance(element, ColumnElement): - raise exc.ArgumentError("Column-based expression object expected; " - "got: %r" % element) + raise exc.ArgumentError("Column-based expression object expected for argument '%s'; " + "got: '%s', type %s" % (name, element, type(element))) return element def _corresponding_column_or_error(fromclause, column, |