Great news Howard,

I'll try the fix right away. Thanks for your awesome work!

--
Olaf Tomczak

2012/4/13 Steve Eynon <steve.ey...@alienfactory.co.uk>

> Oh sweet. That is good news, cheers!
> --
> Steve Eynon
> -------------------------------
> "If at first you don't succeed,
>    so much for skydiving!"
>
>
>
>
> On 13 April 2012 06:47, Howard Lewis Ship <hls...@gmail.com> wrote:
> > Turns out its mutable fields that don't work.  I was fooled because my
> > tests mistakenly only attempted injected fields.
> >
> > In any case, it is fixed in 5.3.3-rc-3.
> >
> > On Wed, Apr 11, 2012 at 9:20 PM, Steve Eynon
> > <steve.ey...@alienfactory.co.uk> wrote:
> >> Hello,
> >>
> >> Directly accessing instrumented fields (@Parameter etc...) from
> >> subclasses does not work. This is fine, but I do wish T5 complained
> >> bitterly when it finds a instrumented field marked as non-public.
> >>
> >> Olaf found similar problems here (TAP5-1860):
> >>
> http://tapestry.1045711.n5.nabble.com/Protected-page-properties-confusion-td5607824.html
> >>
> >> We've come a cropper with this a couple of times when young'uns mark a
> >> @Parameter field as protected and accesses it from a subclass. The
> >> nasty gotcha being, that once the field is shadowed in DevMode, it
> >> works fine! But then fails spectacularly in ProdMode with an NPE as
> >> the actual field instance is never set!
> >>
> >> I know this is all related to TAP5-1801 and the T5.3.2 release but I'm
> >> not really sure what it was supposed to address. If, as I suspect, we
> >> shouldn't mark instrumented fields as protected could the
> >> documentation be updated to say so? (
> >>
> http://tapestry.apache.org/component-classes.html#ComponentClasses-InstanceVariables
> >> )
> >>
> >> Cheers,
> >>
> >> Steve.
> >> --
> >> Steve Eynon
> >> -------------------------------
> >> "If at first you don't succeed,
> >>    so much for skydiving!"
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
> >> For additional commands, e-mail: users-h...@tapestry.apache.org
> >>
> >
> >
> >
> > --
> > 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
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
> > For additional commands, e-mail: users-h...@tapestry.apache.org
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
> For additional commands, e-mail: users-h...@tapestry.apache.org
>
>

Reply via email to