You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Johan Compagner (JIRA)" <ji...@apache.org> on 2007/10/29 22:46:50 UTC
[jira] Closed: (WICKET-336) Inheritable model cannot be a wrap
model
[ https://issues.apache.org/jira/browse/WICKET-336?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Johan Compagner closed WICKET-336.
----------------------------------
Resolution: Fixed
test first if it is an inheritable model so then we don't unwrap
> Inheritable model cannot be a wrap model
> ----------------------------------------
>
> Key: WICKET-336
> URL: https://issues.apache.org/jira/browse/WICKET-336
> Project: Wicket
> Issue Type: Bug
> Components: wicket
> Affects Versions: 1.3.0-beta5
> Environment: n.a.
> Reporter: Jan Vermeulen
> Assignee: Johan Compagner
> Fix For: 1.3.0-beta5
>
>
> In the code of Component:initModel(), when it comes accross an IWrapModel, it always gets its nested model, not considering if this model itself might be an IInheritableModel.
> for (Component current = getParent(); current != null; current = current.getParent())
> {
> // Get model
> IModel model = current.getModel();
> if (model instanceof IWrapModel)
> {
> model = ((IWrapModel)model).getNestedModel();
> }
> if (model instanceof IInheritableModel)
> {
> ....
> }
> }
> Would it be correct do make the following correction ?
> if (model instanceof IWrapModel && !(model instanceof IInheritableModel))
> {
> model = ((IWrapModel)model).getNestedModel();
> }
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.