You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Sven Meier (Closed) (JIRA)" <ji...@apache.org> on 2012/03/19 22:19:38 UTC

[jira] [Closed] (WICKET-4462) Update AbstractPropertyModel and PropertyModel constructor to include parameter for IModel

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

Sven Meier closed WICKET-4462.
------------------------------

    Resolution: Invalid

Already working as suggested.
                
> Update AbstractPropertyModel and PropertyModel constructor to include parameter  for IModel<?>
> ----------------------------------------------------------------------------------------------
>
>                 Key: WICKET-4462
>                 URL: https://issues.apache.org/jira/browse/WICKET-4462
>             Project: Wicket
>          Issue Type: Improvement
>    Affects Versions: 1.5.5
>            Reporter: Don Ngo
>            Priority: Minor
>
> Please consider adding another constructor to AbstractPropertyModel and PropertyModel class to include a parameter for IModel<?>.  Since both of these classes implement the IChainingModel interface, I think it makes sense to include a parameter for IModel<?> in the constructor for these classes, as in the case of CompoundPropertyModel class.
> For what's it worth, below is my custom class to enhance the PropertyModel class:
> public class CustomPropertyModel<T> extends PropertyModel<T> {
> 	private static final long serialVersionUID = 1L;
> 	public CustomPropertyModel(IModel<T> model, String expression) {
> 		super(model.getObject(), expression);
> 		setChainedModel(model);
> 	}
> }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira