FormComponent.setType should be called in setModel
--------------------------------------------------

                 Key: WICKET-1117
                 URL: https://issues.apache.org/jira/browse/WICKET-1117
             Project: Wicket
          Issue Type: Improvement
          Components: wicket
    Affects Versions: 1.3.0-beta4
         Environment: Linux
            Reporter: Bruno Borges
             Fix For: 1.3.0-beta5


The method FormComponent.setType() is called in onBeforeRender() and this will 
happen everytime, without having changes in the Model.
The only way I see to change the Model's Component,  is calling setModel().
So, my suggestion is to override Component.setModel in FormComponent calling 
super.setModel, and after that update the type.

public Component setModel(final IModel model)
{
    Component component = super.setModel(model);
    updateType();
    return component;
}

protected void updateType()
{
        if (getType() == null)
        {
                // Set the type, but only if it's not a String (see WICKET-606).
                // Otherwise, getConvertEmptyInputStringToNull() won't work.
                Class type = getModelType(getModel());
                if (!String.class.equals(type))
                {
                        setType(type);
                }
        }
}

-- 
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