Okay. For 2.0 we can have detach(Component component).. or maybe componentDetached(Component component), and we can call the cleanup() from there. I wouldn't mind.
But what to do for 1.2? One solution would be to call cleanup() from AbstractBehavior.detachModel(), the only thing that bothers me is that it wont be entirely backwards compatible (in case someone has overriden detachModel and doesn't call super.detachModel) WDYT? -Matej Igor Vaynberg wrote: > im not sure why we have detachModel() as opposed to detach() maybe we > should just add detach(), it seems logical to me > > -Igor > > > On 10/17/06, *Matej Knopp * <[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>> > wrote: > > Well, as far as I can see there's only detachModels in IBehavior. Is > this the place? > > -Matej > > Igor Vaynberg wrote: > > why not do the cleanup in detach() ? > > > > -Igor > > > > On 10/17/06, *Matej Knopp* <[EMAIL PROTECTED] > <mailto:[EMAIL PROTECTED]> <mailto:[EMAIL PROTECTED] > <mailto:[EMAIL PROTECTED]>>> wrote: > > > > Well, it seems like a bug, the question is how to fix it? > > The problem is that rendered() should be called after > component is > > rendered. But we do call it even after header is rendered, > because we > > need to do some cleanup there. > > > > So for 2.0 I think we need something like > > IBehavior.cleanup() - where the cleanup will be done > > IBehavior.componentRendered() - after the component has been > rendered - > > we print the autocomplete script here > > > > For 1.2 we need some kind of dirty hack perhaps? :) > > > > -Matej > > > > Igor Vaynberg wrote: > > > yes its a bug, please file it > > > > > > -igor > > > > > > > > > On 10/16/06, *Caleb Land* < [EMAIL PROTECTED] > <mailto:[EMAIL PROTECTED]> > > <mailto:[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>> > > > <mailto:[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]> > <mailto: [EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>>>> wrote: > > > > > > On 10/16/06, *Caleb Land* < [EMAIL PROTECTED] > <mailto:[EMAIL PROTECTED]> > > <mailto: [EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>> > > > <mailto:[EMAIL PROTECTED] > <mailto:[EMAIL PROTECTED]> <mailto:[EMAIL PROTECTED] > <mailto:[EMAIL PROTECTED]>>>> > > wrote: > > > > > > I'm using Wicket 2.0 > > > > > > The IBehavior#rendered() method is called twice when a > > component > > > that has that behavior is rendered via AJAX. When > > rendering the > > > component though the normal process (not ajax) it > is only > > called > > > once. > > > > > > Here are the two places it's getting called: > > > > > > The AjaxRequestTarget#respondComponent() method calls > > > component.renderComponent() which tells the behaviors > > that the > > > component was rendered (line 464 of > AjaxRequestTarget.java ). > > > > > > The AjaxRequestTarget#respondHeaderContribution() > method goes > > > calls each component's renderedBehaviors() method > which > > ends up > > > calling IBehavior#rendered() (lines 538 and 549 of > > > AjaxRequestTarget.java). > > > > > > > > > That should be "lines 513 and 524 of > AjaxRequestTarget.java" > > > > > > The result of this is that Firefox, Safari, and > Opera (I > > don't > > > know about IE) end up executing any javascript that's > > rendered > > > through a behavior twice. > > > > > > The example I tested was adding an > AutoCompleteTextField to a > > > page through ajax. > > > > > > If this is a bug I can add it to the JIRA. > > > > > > -- > > > Caleb Land > > > ( [EMAIL PROTECTED] > <mailto:[EMAIL PROTECTED]> <mailto:[EMAIL PROTECTED] > <mailto:[EMAIL PROTECTED]>> > > <mailto: [EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]> > <mailto: [EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>>>) > > > > > > > > > > > > > > > -- > > > Caleb Land > > > ( [EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]> > <mailto:[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>> <mailto: > > [EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]> <mailto: > [EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>>>) > > > > > > > > > ------------------------------------------------------------------------- > > > Using Tomcat but need to do more? Need to support web > services, > > > security? > > > Get stuff done quickly with pre-integrated technology > to make > > your > > > job easier > > > Download IBM WebSphere Application Server v.1.0.1 > based on Apache > > > Geronimo > > > > > > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642> > > > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642>> > > > > > > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642> > > < > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642>>> > > > > > > _______________________________________________ > > > Wicket-user mailing list > > > [email protected] > <mailto:[email protected]> > > <mailto:[email protected] > <mailto:[email protected]> > > > > <mailto: [email protected] > <mailto:[email protected]> > > <mailto:[email protected] > <mailto:[email protected]>>> > > > https://lists.sourceforge.net/lists/listinfo/wicket-user > > < https://lists.sourceforge.net/lists/listinfo/wicket-user> > > > > <https://lists.sourceforge.net/lists/listinfo/wicket-user> > > > > > > > > > > > > > > > > > > ------------------------------------------------------------------------ > > > > > > > > > > > ------------------------------------------------------------------------- > > > Using Tomcat but need to do more? Need to support web > services, > > security? > > > Get stuff done quickly with pre-integrated technology to make > > your job easier > > > Download IBM WebSphere Application Server v.1.0.1 based on > Apache > > Geronimo > > > > > > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642> > > > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642>> > > > > > > > > > > > > ------------------------------------------------------------------------ > > > > > > _______________________________________________ > > > Wicket-user mailing list > > > [email protected] > <mailto:[email protected]> > > <mailto: [email protected] > <mailto:[email protected]>> > > > https://lists.sourceforge.net/lists/listinfo/wicket-user > > > > > > > ------------------------------------------------------------------------- > > > > > Using Tomcat but need to do more? Need to support web services, > > security? > > Get stuff done quickly with pre-integrated technology to make > your > > job easier > > Download IBM WebSphere Application Server v.1.0.1 based on Apache > > Geronimo > > > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642> > > > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642>> > > _______________________________________________ > > Wicket-user mailing list > > [email protected] > <mailto:[email protected]> > > <mailto:[email protected] > <mailto:[email protected]>> > > https://lists.sourceforge.net/lists/listinfo/wicket-user > > < https://lists.sourceforge.net/lists/listinfo/wicket-user> > > > > > > > > > ------------------------------------------------------------------------ > > > > > ------------------------------------------------------------------------- > > > Using Tomcat but need to do more? Need to support web services, > security? > > Get stuff done quickly with pre-integrated technology to make > your job easier > > Download IBM WebSphere Application Server v.1.0.1 based on Apache > Geronimo > > > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642> > > > > > > > ------------------------------------------------------------------------ > > > > _______________________________________________ > > Wicket-user mailing list > > [email protected] > <mailto:[email protected]> > > https://lists.sourceforge.net/lists/listinfo/wicket-user > > > ------------------------------------------------------------------------- > > Using Tomcat but need to do more? Need to support web services, > security? > Get stuff done quickly with pre-integrated technology to make your > job easier > Download IBM WebSphere Application Server v.1.0.1 based on Apache > Geronimo > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > <http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642> > _______________________________________________ > Wicket-user mailing list > [email protected] > <mailto:[email protected]> > https://lists.sourceforge.net/lists/listinfo/wicket-user > <https://lists.sourceforge.net/lists/listinfo/wicket-user> > > > > ------------------------------------------------------------------------ > > ------------------------------------------------------------------------- > Using Tomcat but need to do more? Need to support web services, security? > Get stuff done quickly with pre-integrated technology to make your job easier > Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > > > ------------------------------------------------------------------------ > > _______________________________________________ > Wicket-user mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/wicket-user ------------------------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ Wicket-user mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/wicket-user
