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

Reply via email to