Hi Kalle, I am currently working on this JIRA, i have seen that the getActivatePage() is only called to pre allocate names in Form component so it should have. So i guess the page you are returning contains a Form.
Despite it sound right that PageResponseRenderer should set the active page, don't you think that in case of a security exception it would be easier to send an HTTP error and let the application developer use standard web.xml request dispatching to configure the page to display ? Cheers 2010/7/20 Kalle Korhonen <kalle.o.korho...@gmail.com> > Pretty please, any committer? It's a one-liner to fix > https://issues.apache.org/jira/browse/TAP5-1201 and the patch is > attached. > > Kalle > > > On Mon, Jul 19, 2010 at 2:23 PM, Pierce Wetter <pie...@paceap.com> wrote: > > > > On Jul 9, 2010, at 1:40 PM, Kalle Korhonen wrote: > > > >> Sorry to be a pest, but how could I find a champion to apply the patch > >> for TAP5-1201? I understand Howard's busy and his time is probably > >> better spent on bigger issues than this, but I hope that some other > >> committer would be able to pick this up and apply the one-line change > >> that would restore functionality that existed in T5.1. It's my > >> understanding its perfectly safe to do as suggested so we can't go > >> that far off even if somebody wants to refactor it later. > > > > Ditto, since I'm using tapestry-security with 5.2-SNAPSHOT. > > > > > > --------------------------------------------------------------------- > > 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 > > -- Regards, Christophe Cordenier. Committer on Apache Tapestry 5 Co-creator of wooki @wookicentral.com
