On Fri, Apr 6, 2012 at 18:42, Igor Vaynberg <[email protected]> wrote:
> i wrote a IDetachListener that automatically detaches any IModel > fields found on components. is this something we would be interested > in for core? its been running in production for a while without any > noticeable overhead and its nice not to have to implemenet onDetach() > all the time just to forward it to secondary models. the only downside > is that once we introduce this feature we can never remote it because > doing so will break code. > > thoughts? > > -igor > I'm neither for nor against, but people don't seem convinced (yet?), if the decision is "no", I think it would still be of great benefit to instead of detaching, log a warning and enable it in DEVELOPMENT mode only. I know at least 3 people that wrote something like this (myself included). Which also hints that model detaching in an actual problem and it might be a good idea to handle this more automatically.
