I'm taking a look at BEEHIVE-1127 and wanted to share some thoughts about ensuring the annotation processing of validatable bean properties declared in an external form bean class during the check() phase.
The current implementation for processing of the Controller annotations does not process external form bean during the check() phase of the TwoPhaseCoreAnnotationProcessor. During the Controller annotation processing, FlowControllerChecker.onCheck() creates the FormBeanChecker and then during the onCheckInternal() uses it to check ValidatableProperty annotations on inner classes only. I'd like to modify FlowControllerChecker.checkMethod() (for action grammar checking) to also look at the parameter of an action and check for validation rules. The change would only use the FormBeanChecker on the parameter class if it is a ClassDeclaration without a declaring type (so we don't re-check an inner class). Does this sound right? Anyone have some additional thoughts? Rich, do you have some input on this AP change? Thanks, Carlin
