Ah, ok... I didn't look in the actual patch, but just read your description... I thought you added the assert to the actual AjaxRequestTarget and hence *all* AjaxRequestTargets would require to have a returned component.
Looking at the patch, this seems ok. Martijn On 6/24/06, Frank Bille Jensen <[EMAIL PROTECTED]> wrote: > Hi > > I'm not sure I understand. The idea with this assert is to check that a > component has been added to an AjaxRequestTarget and not to check > whether a specific Ajax event is fired. > > We use this assert to check that the code we have written actually does > what we expect. > > So if we have: > > ... > new AjaxLink("onclick") { > public void onClick(AjaxRequestTarget target) { > MyComponent my = new MyComponent("someid"); > ... > target.addComponent(my); > } > } > ... > > > we use the assert to check that we have actually called > target.addComponent(). > > > Regards > Frank Bille > > > On Sat, 2006-06-24 at 11:12 +0200, Martijn Dashorst wrote: > > I don't think this is a good idea, you can have requests that don't > > update something in the page. Or nothing has been updated. > > > > I could envision an ajax 'keep session alive' auto update component, > > that doesn't update the webpage, but does a request each 10 minutes or > > so. > > > > Just like a page: a page doesn't have to have components attached to it. > > > > Martijn > > > > On 6/24/06, SourceForge.net <[EMAIL PROTECTED]> wrote: > > > Patches item #1511607, was opened at 2006-06-24 00:32 > > > Message generated for change (Tracker Item Submitted) made by Item > > > Submitter > > > You can respond by visiting: > > > https://sourceforge.net/tracker/?func=detail&atid=684977&aid=1511607&group_id=119783 > > > > > > Please note that this message will contain a full copy of the comment > > > thread, > > > including the initial issue submission, for this request, > > > not just the latest update. > > > Category: core > > > Group: None > > > Status: Open > > > Resolution: None > > > Priority: 5 > > > Submitted By: Frank Bille Jensen (billen) > > > Assigned to: Nobody/Anonymous (nobody) > > > Summary: Assert that a component has been added to AjaxRequestTarget > > > > > > Initial Comment: > > > Added a method to WicketTester. Use this to test that a > > > component has actually been added to an > > > AjaxRequestTarget. It does it by checking the response > > > if it's an AJAX response and if the component's markup > > > id is present in the response. > > > > > > Attached are patches for 1.2 and 2.0. I believe this > > > could be used in 1.2 since it doesn't introduce any API > > > break, but just extends it. > > > > > > ---------------------------------------------------------------------- > > > > > > You can respond by visiting: > > > https://sourceforge.net/tracker/?func=detail&atid=684977&aid=1511607&group_id=119783 > > > > > > 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-develop mailing list > > > Wicket-develop@lists.sourceforge.net > > > https://lists.sourceforge.net/lists/listinfo/wicket-develop > > > > > > > > > > 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-develop mailing list > Wicket-develop@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/wicket-develop > -- Download Wicket 1.2 now! Write Ajax applications without touching JavaScript! -- http://wicketframework.org 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-develop mailing list Wicket-develop@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-develop