Would that not be too soon? The Button could not have been added to a form
yet?

My particular use case is that I have a chunk of code that returns a tree of
components that may not yet be attached to a page/form...


Matthijs Wensveen-2 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]
> 
> 
> 

-- 
View this message in context: 
http://www.nabble.com/AjaxFallbackButton-really-require-Form-in-constructor--tf4370843.html#a12462188
Sent from the Wicket - User mailing list archive at Nabble.com.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to