[WICKET-1097] AjaxEditableLabel: Model change events not propagated from Editor

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.3.0-beta4
• Fix Version/s: 1.3.0-rc2
• Component/s: wicket-extensions
• Environment: any
• Assignee: Unassigned
• Reporter: Jan Kriesten
• Created: 24/Oct/07 14:08
• Updated: 16/Sep/08 20:32
• Resolved: 04/Nov/07 22:26

Description

Model changes are not propagated from the Editor to the AjaxEditableLabel, so overriding onModelChanging/onModelChanged doesn't work as one might expect. The implementation should be changed to something like this (code sample by Gerolf):

—
protected FormComponent newEditor(MarkupContainer parent, String componentId, IModel model)
{
TextField editor = new TextField(componentId, model)
{
private static final long serialVersionUID = 1L;

protected void onModelChanging()

{ super.onModelChanging(); AjaxEditableLabel.this.onModelChanging(); }

protected void onModelChanged()

{ super.onModelChanged(); AjaxEditableLabel.this.onModelChanged(); }

};
editor.setOutputMarkupId(true);
editor.setVisible(false);
editor.add(new EditorAjaxBehavior());
return editor;
}
—

The same issue might apply to the other inline-editors.