[
https://jira.jboss.org/browse/JBSEAM-4284?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Marek Novotny resolved JBSEAM-4284.
-----------------------------------
Assignee: Marek Novotny
Fix Version/s: 2.2.1.CR3
Resolution: Done
I applied patch plus I fixed also ambiguity of type attribute in
s:conversationPropagation tag.
Now implementation and Seam tag lib library describes type of
conversationPropagation the same as enum org.jboss.seam.core.PropagationType.
> 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
> Assignee: Marek Novotny
> Fix For: 2.2.1.CR3
>
> Attachments: JBSEAM-4284, JBSEAM-4284.stacktrace.txt
>
>
> 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
_______________________________________________
seam-issues mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/seam-issues
