"adrian(a)jboss.org" wrote : I note you've still not fixed some of the public methods you introduced
| on the AbstractKernelController that I complained about before.
|
| They need fixing otherwise somebody can break the entire state machine with injudicious
| (or malign) invocations on those methods.
|
| This is basic OO principles. All public methods should keep the invariant state
| of the underlying Object, there should be no backdoors. :-)
I'll have a look.
Closing all backdoors on the way out. ;-)
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4071217#4071217
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4071217
"adrian(a)jboss.org" wrote :
| I don't understand the alias + "_Alias" reference?
When you add a new alias, I install a new AliasControllerContext into the Controller, for the alias to be a full dependency item.
This new AliasControllerContext gets the name after alias --> alias + "_Alias".
So when you are interested if alias already exists or is pending, you need to know this impl detail.
But I wouldn't want this detail to be exposed outside AbstractController.
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4071210#4071210
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4071210
I note you've still not fixed some of the public methods you introduced
on the AbstractKernelController that I complained about before.
They need fixing otherwise somebody can break the entire state machine with injudicious
(or malign) invocations on those methods.
This is basic OO principles. All public methods should keep the invariant state
of the underlying Object, there should be no backdoors. :-)
View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4071197#4071197
Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4071197