Yes; I'd update the Javadoc, since what 5.3/5.4 does is actually better.
On Mon, Aug 5, 2013 at 8:33 AM, Massimo Lusetti <[email protected]> wrote: > On Mon, Aug 5, 2013 at 5:27 PM, Howard Lewis Ship <[email protected]> > wrote: > > I'm unaware of a problem; this is certainly a case of unexpected > > consequences. Is it a change in behavior? > > > > Please explain the mismatch, > > > > thanks! > > > > > Well the javadoc of PageActivationContext annotation states: > "You should not use this annotation within a class that already has an > onActivate() or onPassivate() method; doing so will result in a runtime > exception." > > But the reality is that the annotation worker simply check for double > presence of the annotation on the same page but not for the presence of a > activation handler method, so on 5.4 and 5.3.x you can have > @PageActivationContext with onActivate method on pages while the javadoc > says the opposite. > > Am I enough clear? > -- > Massimo Lusetti > -- Howard M. Lewis Ship Creator of Apache Tapestry The source for Tapestry training, mentoring and support. Contact me to learn how I can get you up and productive in Tapestry fast! (971) 678-5210 http://howardlewisship.com
