[seam-issues] [JBoss JIRA] Updated: (JBSEAM-4284) Wrong pageflow name in case of nested conversation

[Cody Lerum (JIRA)]

     [ https://jira.jboss.org/browse/JBSEAM-4284?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Cody Lerum updated JBSEAM-4284:
-------------------------------

    Attachment: JBSEAM-4284


Also seeing this. I created a patch against the trunk 

> Wrong pageflow name in case of nested conversation
> --------------------------------------------------
>
>                 Key: JBSEAM-4284
>                 URL: https://jira.jboss.org/browse/JBSEAM-4284
>             Project: Seam
>          Issue Type: Bug
>    Affects Versions: 2.1.2.GA
>            Reporter: Vesa Hiltunen
>         Attachments: JBSEAM-4284
>
>
> I tried to start nested conversation with new pageflow as follows: 
>    <s:link value="DoIt" action="/pages/myPage.xhtml">
>       <s:conversationPropagation type="nested" pageflow="myPageflowName" />
>    </s:link>
> Seam did not find pageflow because it is looking for pageflow named as: d.myPageflowName
> Request parameter was: conversationPropagation=nested.myPageflowName
> It looks like method getPropagationFromRequestParameter(Map parameters) in org.jboss.seam.core.ConversationPropagation contains a bug. Method assumes that string "nested." was "nest."
> Current code: 
>    else if (value.startsWith("nest"))
>    {
>       propagationType = PropagationType.NEST;
>       if ( value.length()>5 )
>       {
>          pageflow = value.substring(5);
>       }         
>    }
> One way to fix the code: 
>    else if (value.startsWith("nest"))
>    {
>       propagationType = PropagationType.NEST;
>       if ( value.length()>7 )
>       {
>          pageflow = value.substring(7);
>       }         
>    }

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira