Uploaded image for project: 'Seam 2'
  1. Seam 2
  2. JBSEAM-1901

ConversationEntry.isDisplayable should also check that its not isRemoveAfterRedirect

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Done
    • Affects Version/s: 2.0.0.BETA1
    • Fix Version/s: 2.0.1.CR1
    • Component/s: Core
    • Labels:
      None

      Description

      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>

        Gliffy Diagrams

          Attachments

            Activity

              People

              • Assignee:
                pmuir Pete Muir
                Reporter:
                cmrudd Chris Rudd
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: