move ddlevents dispatch to SchemaItem; "before_parent_attach", "after_parent_attach" to all SchemaItem classes; replace column._on_table_attach with event
Issue #2037
resolved
No description provided.
Comments (6)
-
reporter -
reporter --- a/lib/sqlalchemy/schema.py Sun Jan 30 15:08:41 2011 -0500 +++ b/lib/sqlalchemy/schema.py Sun Jan 30 18:24:35 2011 -0500 @@ -66,6 +66,11 @@ raise NotImplementedError() + def _set_parent_with_dispatch(self, parent): + self.dispatch.before_parent_attach(self.parent) + self._set_parent(parent) + self.dispatch.after_parent_attach(self.parent) + def get_children(self, **kwargs): """used to allow SchemaVisitor access""" return []
-
reporter those should be commas, i.e. "self, parent"
-
reporter this is in 12073e281eebdece0fe4e24c6704d57eafdc9247. Still need to update the wiki page.
-
reporter - changed title to move ddlevents dispatch to SchemaItem; "before_parent_attach", "after_parent_attach" to all SchemaItem classes; replace column._on_table_attach with event
- changed status to resolved
ok the wiki's done, don't care about the 'init' event for now.
-
reporter - removed milestone
Removing milestone: 0.7.0 (automated comment)
- Log in to comment
then update the recipe at http://www.sqlalchemy.org/trac/wiki/UsageRecipes/NamingConventions to use the events. Remove the usage of Column subclasses: