[ 
https://issues.apache.org/jira/browse/WICKET-2924?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12887191#action_12887191
 ] 

Juergen Donnerstag commented on WICKET-2924:
--------------------------------------------

Since setDefaultFormProcessing() is defined in IFormSubmittingComponent, it 
probably makes sense to add 

        /**
         * @param <T>
         * @param defaultFormProcessing
         * @return this
         */
        <T extends Component> T setDefaultFormProcessing(boolean 
defaultFormProcessing);

to IFormSubmittingComponent as well. I've tested it locally and didn't require 
any other change. Obviously all Components implementing 
IFormSubmittingComponent are already implementing 
setDefaultFormProcessing(boolean).

Not sure we can make that change in 1.4 though, since existing user code 
implementing IFormSubmittingComponent  might not work anymore without a change.

> Duplicated method implementation setDefaultFormProcessing for 
> AbstarctSubmitLink and Button.
> --------------------------------------------------------------------------------------------
>
>                 Key: WICKET-2924
>                 URL: https://issues.apache.org/jira/browse/WICKET-2924
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket
>            Reporter: Michał Letyński
>             Fix For: 1.4.10
>
>
> My probllem here is that i have SubmitLinks and Buttons and i set 
> setDefaultFormProcessing for certain conditions. I must first check whant 
> instance is this, cast and set value. Its annoying.
> Please move it to one class or provide an interface for this. JavaDoc for 
> AbstractLink has this problem annotated already:  "TODO: This is a copy & 
> paste from Button"

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to