From 0ed06283ebf9de9a78dfb7d549418131f96159ff Mon Sep 17 00:00:00 2001 From: Mike Bayer Date: Sat, 24 Aug 2013 11:30:57 -0400 Subject: [PATCH] - this comment is wrong, the method here returns all mapper entities - use a simple isinstance() check instead of looking for presence of "primary_entity" flag, that's a little unclear --- lib/sqlalchemy/orm/query.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/lib/sqlalchemy/orm/query.py b/lib/sqlalchemy/orm/query.py index 2e849fba94..201f90ab99 100644 --- a/lib/sqlalchemy/orm/query.py +++ b/lib/sqlalchemy/orm/query.py @@ -276,11 +276,8 @@ class Query(object): @property def _mapper_entities(self): - # TODO: this is wrong, its hardcoded to "primary entity" when - # for the case of __all_equivs() it should not be - # the name of this accessor is wrong too for ent in self._entities: - if hasattr(ent, 'primary_entity'): + if isinstance(ent, _MapperEntity): yield ent def _joinpoint_zero(self): -- 2.47.2