From: Mike Bayer <mike_mp@zzzcomputing.com>
Date: Tue, 4 Jun 2013 01:01:20 +0000 (-0400)
Subject: - pulling out more aliases, sort of
X-Git-Tag: rel_0_9_0b1~294^2~14
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=87909841c50d00b40153b1a1181a4acfea070fa8;p=thirdparty%2Fsqlalchemy%2Fsqlalchemy.git

- pulling out more aliases, sort of
---

diff --git a/lib/sqlalchemy/orm/query.py b/lib/sqlalchemy/orm/query.py
index 80fee4a098..69ae5155d2 100644
--- a/lib/sqlalchemy/orm/query.py
+++ b/lib/sqlalchemy/orm/query.py
@@ -1879,16 +1879,21 @@ class Query(object):
                     (right_selectable.description,
                     right_mapper.mapped_table.description))
 
-            if not isinstance(right_selectable, expression.Join):
-                if not isinstance(right_selectable, expression.Alias):
-                    right_selectable = right_selectable.alias()
-
+            if isinstance(right_selectable, expression.SelectBase):
+                # TODO: this isn't even covered now!
+                right_selectable = right_selectable.alias()
                 need_adapter = True
+
             right = aliased(right_mapper, right_selectable)
 
         aliased_entity = right_mapper and \
                             not right_is_aliased and \
                             (
+                                # TODO: this produces queries that fail the
+                                # compiler transformation in test_polymorphic_rel
+                                # isinstance(right_mapper._with_polymorphic_selectable, expression.Alias)
+
+                                # current
                                 right_mapper.with_polymorphic
                                 or
                                 overlap # test for overlap: