[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-4432) Query uses closed EntityManager if scoped to PAGE context

[Bernard Labno (JIRA)]

    [ https://jira.jboss.org/jira/browse/JBSEAM-4432?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12487875#action_12487875 ] 

Bernard Labno commented on JBSEAM-4432:
---------------------------------------

So you are afraid I'm making false assumption, but there are already assumptions made. If cached persistenceContext is null then it is assumed it's name has been set and persistenceContext is loaded from context. I guess we can assume that if someone is setting persistenceContext directly, then he knows what he is doing. Proper javadoc on setPersistenceContext mentioning PAGE scope issue would do in my opinion.

> Query uses closed EntityManager if scoped to PAGE context
> ---------------------------------------------------------
>
>                 Key: JBSEAM-4432
>                 URL: https://jira.jboss.org/jira/browse/JBSEAM-4432
>             Project: Seam
>          Issue Type: Bug
>          Components: Framework
>    Affects Versions: 2.1.1.GA
>         Environment: JBoss AS 4.2.2.GA, linux
>            Reporter: Bernard Labno
>            Assignee: Pete Muir
>         Attachments: PersistenceController.java.patch
>
>   Original Estimate: 30 minutes
>  Remaining Estimate: 30 minutes
>
> Home objects cache entity manager which is fine for most scopes except for PAGE scope. I think, that PersistenceController.getPersistenceContext() method should check if cached entityManager is closed and if so than fetch one from context.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira