[WICKET-2843] Palette is incompatible with ListMultipleChoice in its use of the model

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.4.7
• Fix Version/s: 1.4.9, 1.5-M1
• Component/s: wicket-extensions
• Assignee: Igor Vaynberg
• Reporter: Frank van Lankvelt
• Created: 21/Apr/10 09:17
• Updated: 26/Apr/10 17:39
• Resolved: 26/Apr/10 17:39

Description

Palette, like the listmultiplechoice component, modifies the object (a list) in-place. Both components have the same behavior in their updateModel documentation, but the implementation differs. Whereas ListMultipleChoice invokes getModel().setObject(), Palette uses setDefaultModelObject(). This has the consequence that it never invokes the setObject method as that is guarded by a model comparator.

Suggested fix is to use the same logic in both updateModel implementations.