oh yeah, a couple of internal methods changed too:
internalOnBeginRequest -> internalOnAttach
internalOnEndRequest -> internalOnDetach
internalBeginRequest -> internalAttach
internalEndRequest -> internalDetach
the methods that used to be internalBegin/EndRequest then look like this:
protected void internalAttach()
{
onBeginRequest();
onAttach();
internalOnAttach();
}
protected void internalDetach()
{
internalOnDetach();
onDetach();
onEndRequest();
}
Martijn
On 3/20/06,
Martijn Dashorst <[EMAIL PROTECTED]> wrote:
I've got a last minute fix for this bug that I'd like to include. Your thoughts and your vote are desired.
Best,
Jon
The change:
New methods onAttach(), onDetach(), onBeginRender() and onEndRender() have been introduced. onAttach() and onDetach() are called in internalOnBegin/EndRequest() so they should have the same semantics as the code that is currently doing attach/detach stuff in those places. In Wicket 1.3, we can tighten that up further.
In order to guarantee that calls to onBeginRender() and onEndRender() occur in all cases, i renamed Component.doRender() -> Component.renderComponent() and Page.doRender() -> renderPage so that i could make each one final. The renderComponent() method preserves the earlier doRender semantics exactly by checking for instanceof Page and calling Page.renderPage() in that case as if it were overridden.
The result is that we can deprecate Component.onBegin/EndRequest() now and provide alternatives for both former usages that will be easy to tighten up or modify in Wicket 1.3 or in the future. I also changed all internal uses of doRender() and onBegin/EndRequest() to the appropriate method.
Please vote for this issue as soon as possible.
Martijn
--
Cast your final vote for Wicket in the SourceForge.net 2006 Community Choice Awards!
http://www.wilsonresearch.com/2006/ostgawards06/ostgawards4.php
--
Cast your final vote for Wicket in the SourceForge.net 2006 Community Choice Awards!
http://www.wilsonresearch.com/2006/ostgawards06/ostgawards4.php
