Patch to support bean aliases and exception handling in destroy() methods

[Martin Ellis]

Wilfred will have seen this, but it's probably best to mention on the
list too...

I've submitted a pull request that adds support for bean aliases and
exception handling in destroy() methods:
https://github.com/wspringer/spring-me/pull/2

Regards,
Martin

[Wilfred Springer]

Hi Martin, 

Looking good! I checked it in a minute ago. It did make me wonder if we perhaps should alter the meta model a little. But as long as the current model floats your boat, I have no problem continuing on the same lines. 

Wilfred

--
You received this message because you are subscribed to the Google Groups "Spring ME" group.
To post to this group, send email to spri...@googlegroups.com.
To unsubscribe from this group, send email to spring-me+...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/spring-me?hl=en.

[Martin Ellis]

Hi Wilfred,

Thanks for merging.

I did have some thoughts about the meta-model, but nothing conclusive.
I find it difficult to keep track of the pre- and post-conditions for
each part of the analysis (i.e. which classes are responsible for
setting which fields in the model). But I don't have any ideas how to
simplify that right now.

I also wondered about 'resolving' (somehow) MutableInstanceReferences
to the MutableInstance to which they refer. This might have helped
with the FactoryBean support, but in the end I implemented it by just
copying the type information from the latter to the former.

I'll need to shuffle and tidy a few commits, and then I'll send you a
pointer to the FactoryBean stuff.

Martin