- adjusted the definition of "self-referential" to be

any two mappers with a common parent (this affects
whether or not aliased=True is required when joining
with Query).

diff --git a/CHANGES b/CHANGES
index d555062abd3d630d443b0835baa10ea2302465e0..002f0fa5d30a4ec7e7f5d57d11edbbc97bd59631 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -20,6 +20,11 @@ CHANGES
 
     - Session.execute can now find binds from metadata
 
+    - adjusted the definition of "self-referential" to be
+      any two mappers with a common parent (this affects
+      whether or not aliased=True is required when joining
+      with Query).
+      
     - assorted "cascade deletes" fixes:
         - Fixed "cascade delete" operation of dynamic
           relations, which had only been implemented for

diff --git a/lib/sqlalchemy/orm/properties.py b/lib/sqlalchemy/orm/properties.py
index e34c725ffe9b866c20dceae20f178f0241da82ad..13ed83bc94a66e65e0de8a91fb4658f36f2a91d9 100644 (file)
--- a/lib/sqlalchemy/orm/properties.py
+++ b/lib/sqlalchemy/orm/properties.py
@@ -759,7 +759,7 @@ class PropertyLoader(StrategizedProperty):
         return self.parent.mapped_table is self.target or self.parent.select_table is self.target
     
     def _is_self_referential(self):
-        return self.mapper.isa(self.parent)
+        return self.mapper.common_parent(self.parent)
         
     def primary_join_against(self, mapper, selectable=None, toselectable=None):
         return self.__cached_join_against(mapper, selectable, toselectable, True, False)