From 0b0ff18b83c5beb9ef8ffa5c838c9eb16d879b02 Mon Sep 17 00:00:00 2001 From: Mike Bayer Date: Tue, 31 May 2011 18:06:57 -0400 Subject: [PATCH] - dont actually need separate init(), but be conservative for now. --- lib/sqlalchemy/orm/interfaces.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/sqlalchemy/orm/interfaces.py b/lib/sqlalchemy/orm/interfaces.py index 07a6a782c2..19c302ec9e 100644 --- a/lib/sqlalchemy/orm/interfaces.py +++ b/lib/sqlalchemy/orm/interfaces.py @@ -313,7 +313,6 @@ class StrategizedProperty(MapperProperty): def __init_strategy(self, cls): self._strategies[cls] = strategy = cls(self) - strategy.init() return strategy def setup(self, context, entity, path, reduced_path, adapter, **kwargs): @@ -638,6 +637,11 @@ class LoaderStrategy(object): self.is_class_level = False self.parent = self.parent_property.parent self.key = self.parent_property.key + # TODO: there's no particular reason we need + # the separate .init() method at this point. + # It's possible someone has written their + # own LS object. + self.init() def init(self): raise NotImplementedError("LoaderStrategy") -- 2.39.5