myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Leonardo Uribe (JIRA)" <...@myfaces.apache.org>
Subject [jira] [Resolved] (MYFACES-3916) Wrong navigation handling for <if> clause on null outcome
Date Tue, 02 Sep 2014 21:25:21 GMT

     [ https://issues.apache.org/jira/browse/MYFACES-3916?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Leonardo Uribe resolved MYFACES-3916.
-------------------------------------
    Resolution: Invalid
      Assignee: Leonardo Uribe

I have tested it with a junit example and the logic is correct. I don't see any problem with
the logic. If from-outcome is not null and outcome is null the case is not mached. This part
works as JSF 2.2 section 7.4.2 says. I'll close this issue as invalid.

> Wrong navigation handling for <if> clause on null outcome
> ---------------------------------------------------------
>
>                 Key: MYFACES-3916
>                 URL: https://issues.apache.org/jira/browse/MYFACES-3916
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: JSR-344
>    Affects Versions: 2.2.4
>            Reporter: Dominik Appl
>            Assignee: Leonardo Uribe
>            Priority: Minor
>
> Outcome is null, but the following navigation case matches (even though "from-outcome"
is present).
> {noformat}
> <navigation-case>
>       <from-outcome>go_mainMenu</from-outcome>
>       <if>#{!mainLayoutBean.isXS}</if>
>       <to-view-id>/mainMenu.xhtml</to-view-id>
> </navigation-case>
> {noformat}
> Specification says:
> To match an outcome value of null, *the <from-outcome> must be absent* and the
<if> element present. 
> For evaluation look at  org.apache.myfaces.application.NavigationHandlerImpl#calcMatchingNavigationCase
> perhaps especially at
> {noformat}
> if(outcome == null && (cazeOutcome != null || cazeIf == null) && actionRef
== null)  {
>   continue;
> }
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message