diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2022-01-18 17:19:24 -0500 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2022-01-18 17:19:24 -0500 |
| commit | e28ec27b599558b3e26ced106a972e8b4aa9e668 (patch) | |
| tree | c89c5447fd7ba8ace3c334918e0d43e34d44c851 /lib/sqlalchemy/sql/lambdas.py | |
| parent | e6ded82eef63235d7cbfe3ab3382a48f32913640 (diff) | |
| download | sqlalchemy-e28ec27b599558b3e26ced106a972e8b4aa9e668.tar.gz | |
reject methods as lambda SQL callables
Added an informative error message when a method object is passed to a SQL
construct. Previously, when such a callable were passed, as is a common
typographical error when dealing with method-chained SQL constructs, they
were interpreted as "lambda SQL" targets to be invoked at compilation time,
which would lead to silent failures. As this feature was not intended to be
used with methods, method objects are now rejected.
Fixes: #7032
Change-Id: If714715bd8c11557ab769ee3b1a24264b0b06acc
Diffstat (limited to 'lib/sqlalchemy/sql/lambdas.py')
| -rw-r--r-- | lib/sqlalchemy/sql/lambdas.py | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/lib/sqlalchemy/sql/lambdas.py b/lib/sqlalchemy/sql/lambdas.py index d71c85d60..ae7358870 100644 --- a/lib/sqlalchemy/sql/lambdas.py +++ b/lib/sqlalchemy/sql/lambdas.py @@ -6,6 +6,7 @@ # the MIT License: https://www.opensource.org/licenses/mit-license.php import collections.abc as collections_abc +import inspect import itertools import operator import types @@ -618,6 +619,10 @@ class AnalyzedCode: return analyzed def __init__(self, fn, lambda_element, opts): + if inspect.ismethod(fn): + raise exc.ArgumentError( + "Method %s may not be passed as a SQL expression" % fn + ) closure = fn.__closure__ self.track_bound_values = ( |