[
http://jira.jboss.com/jira/browse/JBSEAM-2609?page=all ]
Pete Muir closed JBSEAM-2609.
-----------------------------
Resolution: Done
Assignee: Pete Muir
Thanks Jacob, I applied your patch.
Locking issue for ConversationEntry when using
Conversation.instance().leave()
------------------------------------------------------------------------------
Key: JBSEAM-2609
URL:
http://jira.jboss.com/jira/browse/JBSEAM-2609
Project: JBoss Seam
Issue Type: Bug
Components: Core
Affects Versions: 2.0.1.GA
Reporter: Jacob Orshalick
Assigned To: Pete Muir
Priority: Minor
Fix For: 2.0.2.GA, 2.1.0.GA
Attachments: JBSEAM-2609.patch
When using Conversation.instance().leave(), it seems that the ConversationEntry lock is
not released. *Sometimes* it can be obtained when attempting to resume the conversation
but other times it cannot. This results in the user being redirected to the
no-conversation-view-id after the concurrent-request-timeout expires.
This can be observed by placing Conversation.instance.leave() in an action that redirects
to another page. Then simply use the back-button and resume the conversation by clicking
a link on the page. You may have to try this several times to get the result. I will see
if I can create a reliable test case and attach it.
Manager.switchConversation() performs an unlock of the ConversationEntry prior to
switching, which would also make sense in the case of leave (since it is really a switch
to a temp conversation).
The workaround is simply executing the following:
Code:
Conversation conversation = Conversation.instance();
ConversationEntry entry =
ConversationEntries.instance().getConversationEntry(conversation.getId());
conversation.leave();
entry.unlock();
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira