[ 
https://issues.apache.org/jira/browse/ISIS-1226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14985710#comment-14985710
 ] 

ASF subversion and git services commented on ISIS-1226:
-------------------------------------------------------

Commit a7519835dcede00bc7c4ddd84882c9900a8e3a37 in isis's branch 
refs/heads/master from [~danhaywood]
[ https://git-wip-us.apache.org/repos/asf?p=isis.git;h=a751983 ]

ISIS-1226: detach EntityModels in BreadcrumbModel


> Drop-down breadcrumbs broken.
> -----------------------------
>
>                 Key: ISIS-1226
>                 URL: https://issues.apache.org/jira/browse/ISIS-1226
>             Project: Isis
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 1.9.0
>            Reporter: Dan Haywood
>            Assignee: Dan Haywood
>            Priority: Minor
>             Fix For: 1.10.0
>
>
> The BreadcrumbModel is held by the session (specifically, the 
> AuthenticatedWebSessionForIsis, subclass of a Wicket session class).
> This holds a list of EntityModels.  
> The EntityModels are LoadableDetachable, which caches the underlying 
> ObjectAdapter in a transient field.
> It seems that the model isn't being serialized/deserialized, and there is no 
> explicit detach, thus on the next request the EntityModel still holds the 
> ObjectAdapter (for the previous PersistenceSession) and which in turn holds 
> the pojo.  From JDO/Datanucleus' perspective, this pojo is not persistent.
> As per ISIS-1194 simplificaitons, we now delegate to the pojo to determine 
> whether it is persistent or not; and so JDO/DN reports that this pojo is NOT 
> persistent.
> In turn this gives us a "" empty string for the oid str, and things then 
> break.
> ~~~
> The fix is to explicitly detach all the EntityModels of the BreadcrumbModel.  
> This can be done by hooking into the AuthenticatedWebSessionForIsis#detach() 
> hook.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to