You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Mikael Björertz (JIRA)" <ji...@apache.org> on 2014/08/26 16:07:57 UTC

[jira] [Updated] (WICKET-5683) PreviousButton isEnabled() should bo logical conjunction of getWizardModel().isPreviousAvailable() and super.isEnabled()

     [ https://issues.apache.org/jira/browse/WICKET-5683?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mikael Björertz updated WICKET-5683:
------------------------------------

    Summary: PreviousButton isEnabled() should bo logical conjunction of getWizardModel().isPreviousAvailable() and super.isEnabled()  (was: PreviousButton isEnabled() should bo logical conjunction of getWizardModel().isNextAvailable() and super.isEnabled())

> PreviousButton isEnabled() should bo logical conjunction of getWizardModel().isPreviousAvailable() and super.isEnabled()
> ------------------------------------------------------------------------------------------------------------------------
>
>                 Key: WICKET-5683
>                 URL: https://issues.apache.org/jira/browse/WICKET-5683
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket-extensions
>            Reporter: Mikael Björertz
>              Labels: navigation, wizard
>   Original Estimate: 5m
>  Remaining Estimate: 5m
>
> The implementation of isEnabled in NextButton breaks the implicit contract of setEnabled(false) -> isEnabled() returning false.
> Implementation should be 
> return super.isEnabled() && getWizardModel().isNextAvailable();



--
This message was sent by Atlassian JIRA
(v6.2#6252)