[seam-issues] [JBoss JIRA] Issue Comment Edited: (JBSEAM-4774) Usage of propagation="nested" with <s:link> doesn't create nested conversation

[Alexey Ovchinnikov (JIRA)]

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

Alexey Ovchinnikov edited comment on JBSEAM-4774 at 2/3/11 10:16 AM:
---------------------------------------------------------------------

Marek, sorry, but I can't provide the test case at a moment. Just can point to bug placement :-)
Seems it is source ui/src/main/java/org/jboss/seam/ui/component/UISeamCommandBase.java, lines starting from 80: 

{code:title=UISeamCommandBase.java|borderStyle=solid}
      if ("default".equals(getPropagation()) || "join".equals(getPropagation())
               || "nest".equals(getPropagation()) || "end".equals(getPropagation()))
      {
         UIConversationId uiConversationId = UIConversationId.newInstance();
         uiConversationId.setViewId(viewId);
         url.addParameter(uiConversationId);
      }

      if ("join".equals(getPropagation()) || "nest".equals(getPropagation())
               || "begin".equals(getPropagation()) || "end".equals(getPropagation()))
      {
         UIConversationPropagation uiPropagation = UIConversationPropagation.newInstance();
         uiPropagation.setType(getPropagation());
         uiPropagation.setPageflow(getPageflow());
         url.addParameter(uiPropagation);
      }
{code} 

As you can see, "nest" word is checked, instead of "nested".


      was (Author: allen1005):
    Marek, sorry, but I can't provide the test case at a moment. Just can point to bug placement :-)
Seems it is source ui/src/main/java/org/jboss/seam/ui/component/UISeamCommandBase.java, lines starting from 80: 

      if ("default".equals(getPropagation()) || "join".equals(getPropagation())
               || "nest".equals(getPropagation()) || "end".equals(getPropagation()))
      {
         UIConversationId uiConversationId = UIConversationId.newInstance();
         uiConversationId.setViewId(viewId);
         url.addParameter(uiConversationId);
      }

      if ("join".equals(getPropagation()) || "nest".equals(getPropagation())
               || "begin".equals(getPropagation()) || "end".equals(getPropagation()))
      {
         UIConversationPropagation uiPropagation = UIConversationPropagation.newInstance();
         uiPropagation.setType(getPropagation());
         uiPropagation.setPageflow(getPageflow());
         url.addParameter(uiPropagation);
      }

As you can see, "nest" word is checked, instead of "nested".

  
> Usage of propagation="nested" with <s:link> doesn't create nested conversation
> ------------------------------------------------------------------------------
>
>                 Key: JBSEAM-4774
>                 URL: https://issues.jboss.org/browse/JBSEAM-4774
>             Project: Seam
>          Issue Type: Bug
>    Affects Versions: 2.2.1.Final
>            Reporter: Alexey Ovchinnikov
>            Assignee: Marek Novotny
>             Fix For: 2.3.0.ALPHA
>
>
> Have migrated my project to Seam 2.2.1.Final from Seam 2.2.0.GA and detected that JBSEAM-4755 still exists.
> <s:link propagation="nest"> throws Exception and <s:link propagation="nested"> does not create a nested conversation...

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira