[WICKET-5586] NextButton isEnabled() should bo logical conjunction of getWizardModel().isNextAvailable() and super.isEnabled()

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 6.12.0, 7.0.0-M1
• Fix Version/s: 6.16.0, 7.0.0-M2
• Component/s: wicket-extensions
• Labels: navigation wizard
• Assignee: Sven Meier
• Reporter: Mikael Björertz
• Created: 08/May/14 13:27
• Updated: 12/May/14 08:03
• Resolved: 12/May/14 08:03

Description

The implementation of isEnabled in NextButton breaks the implicit contract of setEnabled(false) -> isEnabled() returning false. There is no apparent reason and no explanation to why this is done.

Implementation should be
return super.isEnabled() && getWizardModel().isNextAvailable();