This would be wrong, as during constructor the component is not yet added to
any parent.
-Matej
On 9/3/07, Matthijs Wensveen <[EMAIL PROTECTED]> wrote:
>
> Or maybe:
>
> public AjaxButton(String id)
> {
> this(id, getForm());
> }
>
>
> This could work the same for AjaxFallbackButton.
>
> Matthijs
>
> Johan Compagner wrote:
> > It is needed i guess if we say that the method
> > protected abstract void onSubmit(AjaxRequestTarget ajaxrequesttarget,
> Form
> > form1);
> >
> > needs a form parameter that can't be null
> >
> > Also i see something strange in the AjaxButton because if you dont give
> the
> > form then the form parameter
> > can be null:
> >
> > public AjaxButton(String id)
> > {
> > this(id, null);
> > }
> >
> > public Form getForm()
> > {
> > if(form != null)
> > return form;
> > else
> > return super.getForm();
> > }
> >
> > public AjaxButton(String id, final Form form)
> > {
> > super(id);
> > this.form = form;
> > add(new AjaxFormSubmitBehavior("onclick", form) {
> >
> > protected void onSubmit(AjaxRequestTarget target)
> > {
> > AjaxButton.this.onSubmit(target, form);
> > }
> >
> > protected void onError(AjaxRequestTarget target)
> > {
> > AjaxButton.this.onError(target, form);
> > }
> >
> > i think this is wrong, but maybe it was on purpose?
> > i think it should be:
> >
> > public AjaxButton(String id)
> > {
> > this(id, null);
> > }
> >
> > public Form getForm()
> > {
> > if(form != null)
> > return form;
> > else
> > return super.getForm();
> > }
> >
> > public AjaxButton(String id, final Form form)
> > {
> > super(id);
> > this.form = form;
> > add(new AjaxFormSubmitBehavior("onclick", form) {
> >
> > protected void onSubmit(AjaxRequestTarget target)
> > {
> > AjaxButton.this.onSubmit(target, getForm());
> > }
> >
> > protected void onError(AjaxRequestTarget target)
> > {
> > AjaxButton.this.onError(target, getForm());
> > }
> >
> > So that we return the given form or try to find one.
> >
> > johan
> >
> >
> >
> > On 9/3/07, Sam Hough <[EMAIL PROTECTED]> wrote:
> >
> >> Since neither AjaxButton or Button require Form in the constructor why
> >> does
> >> AjaxFallbackButton? Seems a shame to make it required if not really
> >> needed...
> >>
> >> I'm not quite clear from the source or comments what the difference is
> in
> >> AjaxButton between the behaviour from the two different constructors...
> >> --
> >> View this message in context:
> >>
> http://www.nabble.com/AjaxFallbackButton-really-require-Form-in-constructor--tf4370843.html#a12457890
> >> Sent from the Wicket - User mailing list archive at Nabble.com.
> >>
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: [EMAIL PROTECTED]
> >> For additional commands, e-mail: [EMAIL PROTECTED]
> >>
> >>
> >>
> >
> >
>
>
> --
> Matthijs Wensveen
> Func. Internet Integration
> W http://www.func.nl
> T +31 20 4230000
> F +31 20 4223500
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
