You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Martin Makundi (JIRA)" <ji...@apache.org> on 2013/03/21 05:45:15 UTC

[jira] [Updated] (WICKET-5108) AbstractPropertyModel does not detach possible model nested inside property path (maybe latest version also?)

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

Martin Makundi updated WICKET-5108:
-----------------------------------

    Attachment: patch.txt

Attached proposal for the fix.
                
> AbstractPropertyModel does not detach possible model nested inside property path (maybe latest version also?)
> -------------------------------------------------------------------------------------------------------------
>
>                 Key: WICKET-5108
>                 URL: https://issues.apache.org/jira/browse/WICKET-5108
>             Project: Wicket
>          Issue Type: Bug
>    Affects Versions: 1.4.21
>            Reporter: Martin Makundi
>         Attachments: patch.txt
>
>   Original Estimate: 2h
>  Remaining Estimate: 2h
>
> AbstractPropertyModel does not detach possible model nested inside property path if target object is not IDetachable itself.
> For example:
> new PropertyModel(pojoMultipleModelBean, "detachableParticularProperty")
> Now AbstractPropertyModel will investigate target pojoModelBean which is not IDetachable because it holds on to multiple models and would not make sense to detach all at same time.
> Fix proposal is that AbstractPropertyModel detach investigates in addition the whole path to the property model whether there are detachable models, and detaches them.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira