diff options
| author | mike bayer <mike_mp@zzzcomputing.com> | 2019-04-03 14:37:09 +0000 |
|---|---|---|
| committer | Gerrit Code Review <gerrit@bbpush.zzzcomputing.com> | 2019-04-03 14:37:09 +0000 |
| commit | e67b4e768bcb80e8fc3dcd486ed917f3931cb461 (patch) | |
| tree | b501769534a382650f0fa2919318c8acafd107af /lib/sqlalchemy | |
| parent | df62c5462990ca235ba26faddb02c4ec4268bb76 (diff) | |
| parent | 25a42e93f4ef5ce1a9f9c23fbcdea3e21a7b3f1a (diff) | |
| download | sqlalchemy-e67b4e768bcb80e8fc3dcd486ed917f3931cb461.tar.gz | |
Merge "Fold entities into existing joins when resolving FROM ambiguity"
Diffstat (limited to 'lib/sqlalchemy')
| -rw-r--r-- | lib/sqlalchemy/sql/util.py | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/lib/sqlalchemy/sql/util.py b/lib/sqlalchemy/sql/util.py index 3077840c6..8719b26ec 100644 --- a/lib/sqlalchemy/sql/util.py +++ b/lib/sqlalchemy/sql/util.py @@ -20,6 +20,7 @@ from .annotation import _shallow_annotate # noqa from .base import _from_objects from .base import ColumnSet from .ddl import sort_tables # noqa +from .elements import _expand_cloned from .elements import _find_columns # noqa from .elements import _label_reference from .elements import _textual_label_reference @@ -149,6 +150,16 @@ def find_left_clause_to_join_from(clauses, join_to, onclause): idx.append(i) break + if len(idx) > 1: + # this is the same "hide froms" logic from + # Selectable._get_display_froms + toremove = set( + chain(*[_expand_cloned(f._hide_froms) for f in clauses]) + ) + idx = [ + i for i in idx if clauses[i] not in toremove + ] + # onclause was given and none of them resolved, so assume # all indexes can match if not idx and onclause is not None: |
