[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-959) NullPointerException in ConversationEntry if stack is null

[Keith Naas (JIRA)]

    [ http://jira.jboss.com/jira/browse/JBSEAM-959?page=comments#action_12354618 ] 
            
Keith Naas commented on JBSEAM-959:
-----------------------------------

Also needed to change isNested.

public boolean isNested()
   {
      return conversationIdStack != null && conversationIdStack.size()>1;
   }

So then changed the constructor to be


if ( isNested() )
      {
         // get the root conversation entry lock (we want to share the same lock  
         // among all nested conversations in the same conversation stack)
         lock = entries.getConversationEntry( conversationIdStack.get( conversationIdStack.size()-1 ) ).lock;
      }
      else
      {
         lock = new ReentrantLock(true);
      }


> NullPointerException in ConversationEntry if stack is null
> ----------------------------------------------------------
>
>                 Key: JBSEAM-959
>                 URL: http://jira.jboss.com/jira/browse/JBSEAM-959
>             Project: JBoss Seam
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 1.2.0.GA
>            Reporter: Keith Naas
>
> When the conversation stack is null, the ConversationEntry constructor throws a NullPointerException.
> There is a conditional check earlier in the constructor that causes the conversationIdStack to be set to null.  Then later it checks to see if the stack size is greater than 1.  Kablooey!
> changed to
>  if ( conversationIdStack != null && conversationIdStack.size()>1 )

-- 
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