"scott.stark(a)jboss.org" wrote :
| We are missing the setup of the Fields proxy. The fields creation in
AbstractManagedObjectFactory needs to be using the
org.jboss.managed.plugins.advice.WrapperAdvice.wrapFields utility to allow an advice to
update the metadata instance on the setValue call.
|
"scott.stark(a)jboss.org" wrote : Really, aop usage is not needed.
This means removing WrapperAdvice usage?
"scott.stark(a)jboss.org" wrote : Where we copy the client side value to the
server side property, the Fields impl can handle the writethrough, and the runtime
property dispatch be done in the ManagementViewImpl.
I saw we already have WritethroughManagedPropertyImpl, but I think it overrides the wrong
method - instead of overridding setField it should override setValue.
Fixing this is exactly what we need + adding runtime dispatch into MVI. And like you said,
with no AOP.
On the other hand, keeping the WrapperAdvice, does give you a lot of flexibility. I guess
its intention should be clarified.
View the original post :
http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4090892#...
Reply to the post :
http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&a...