[jbossseam-issues] [JBoss JIRA] Created: (JBSEAM-1901) ConversationEntry.isDisplayable should also check that its not isRemoveAfterRedirect

[Chris Rudd (JIRA)]

ConversationEntry.isDisplayable should also check that its not isRemoveAfterRedirect
------------------------------------------------------------------------------------

                 Key: JBSEAM-1901
                 URL: http://jira.jboss.com/jira/browse/JBSEAM-1901
             Project: JBoss Seam
          Issue Type: Bug
          Components: Core
    Affects Versions: 2.0.0.BETA1
            Reporter: Chris Rudd


I have a situation where temporary conversations that are promoted to long running (due to a redirect) for views with a valid description show up in the conversationList.

If ConversationEntry.isDisplayable adds the contraint !isRemoveAfterRedirect() , then the issue does not appear.

public boolean isDisplayable() 
{
   return !isEnded() && !isRemoveAfterRedirect() && getDescription()!=null;
}


NOTE: The particular instance this occurs in for me is due to a current limitation in seam. Currently there is no way to end the current conversation and start a new one (under a new conversation context) in the same request. For instance :

Im in conversation id 5, I hit the "done" button, which I want to end the current conversation (and destroy that context), then redirect me toto the same view starting a new conversation.

There use to be a "new" propagation mode, but that seems to be missing now. If you simply start a new conversation after the redirect, the current conversation (the one promoted due to the redirect, the same one you wanted to end) is just propagated along and contains all same state as before. This is not the desired affect, I want a NEW conversation that contains only new items

So my "workaround" is to have the "done" out come end the conversation, redirect me to another action, that then redirects me to my final view with a propagation none.  The "problem" is that the first conversation, is still out there waiting to be "continued" so that it can convert it back into a temporary (as its marked removeAfterRedirect). Since is a long running conv, it shows up in the conv list (which it should not as its not really a long running conversation). The conv is eventually pruned (due to conv timeout).

<rule if-outcome="done">
   <end-conversation/>
   <redirect> <!-- redirect to same page and fire the edit action -->
             <param name="actionOutcome" value="#{'edit'}"/>
             <param name="conversationPropagation" value="none"/>
   </redirect>
</rule>

<rule if-outcome="edit">
   <begin-conversation join="true"/>
</rule>

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