[jbossseam-issues] [JBoss JIRA] Closed: (JBSEAM-2881) Conversation Switcher doesn't work with natural conversations

Shane Bryzak (JIRA) jira-events at lists.jboss.org
Thu Nov 6 02:14:20 EST 2008

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

Shane Bryzak closed JBSEAM-2881.

    Resolution: Done

That seems reasonable, I've updated it in SVN.  Could you please test it?

> Conversation Switcher doesn't work with natural conversations
> -------------------------------------------------------------
>                 Key: JBSEAM-2881
>                 URL: https://jira.jboss.org/jira/browse/JBSEAM-2881
>             Project: Seam
>          Issue Type: Bug
>          Components: JSF Controls, JSF Integration
>    Affects Versions: 2.0.2.CR1
>         Environment: Seam 2.0.2.CR1, Mac OS X 10.5, Java 1.5.0_13
>            Reporter: Caleb Land
>            Assignee: Shane Bryzak
>             Fix For: 2.1.1.CR1
> The Switcher component's (org.jboss.seam.faces.Switcher) switch() method does not work correctly when using natural conversations.
> The issue is related to line 102 (in trunk and in 2.0.2.CR1) where seam tests whether the selected item is an outcome or a conversation id:
> boolean isOutcome = conversationIdOrOutcome==null || !Character.isDigit( conversationIdOrOutcome.charAt(0) );
> The problem is that with natural conversations, the value of conversationIdOrOutcome is something like "accountEdit:4" so the switcher thinks it's a view.
> I copied the code from the Switcher and created a new component and replaced that line with:
> boolean isOutcome = conversationIdOrOutcome==null ||
>               (!Character.isDigit(conversationIdOrOutcome.charAt(0)) && conversationIdOrOutcome.indexOf(':') < 0);
> and it works correctly. Of course this approach is naive, but it seems to work for me.

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


More information about the seam-issues mailing list