@Bertrand Thank you for clarifying that ALL urls in this discussion are mapped to the same page class.
@Igor The core of Wicket value proposition is that it manages the state of your pages correctly. So, verifying that the state that Wicket manages is consistent with user inputs (product id 123 vs 379) should not be necessary. I can add the checks that you recommended in onconfigure() but I would like to get your agreement that it's a workaround for a critical bug. Thanks, Alec On Fri, Apr 13, 2012 at 1:55 AM, Martin Grigorov <[email protected]> wrote: > https://issues.apache.org/jira/browse/WICKET-4441 > now I'm convinced that this additional check is needed there > > On Fri, Apr 13, 2012 at 12:00 AM, Igor Vaynberg <[email protected]> > wrote: >> in that case a bit of logic in the page that checks the product id in >> onconfigure() against one in the model, and if they are different >> redirects to the correct page... >> >> -igor >> >> On Thu, Apr 12, 2012 at 1:56 PM, Bertrand Guay-Paquet >> <[email protected]> wrote: >>>> you simply need to check what page class is mounted, and if the page >>>> retrieved by id is not of that class then dont render it but redirect >>>> to the bookmarkable url instead. >>>> >>>> -igor >>> >>> Both pages actually use the same MyPage.java class in this case. The only >>> difference is the page parameter encoded in the URL which presumably drives >>> a model. >>> >>> >>> --------------------------------------------------------------------- >>> To unsubscribe, e-mail: [email protected] >>> For additional commands, e-mail: [email protected] >>> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> For additional commands, e-mail: [email protected] >> > > > > -- > Martin Grigorov > jWeekend > Training, Consulting, Development > http://jWeekend.com > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
