[jbossseam-issues] [JBoss JIRA] Closed: (JBSEAM-2345) pages-2.0.xsd handling of <Navigation> doesn't allow standard rules and a null-accepting implicit rule

[Pete Muir (JIRA)]

     [ http://jira.jboss.com/jira/browse/JBSEAM-2345?page=all ]

Pete Muir closed JBSEAM-2345.
-----------------------------

    Resolution: Done
      Assignee: Pete Muir  (was: Norman Richards)

I applied this fix

### Eclipse Workspace Patch 1.0
#P jboss-seam
Index: src/main/org/jboss/seam/pages-2.0.xsd
===================================================================
RCS file: /cvsroot/jboss/jboss-seam/src/main/org/jboss/seam/pages-2.0.xsd,v
retrieving revision 1.2
diff -u -r1.2 pages-2.0.xsd
--- src/main/org/jboss/seam/pages-2.0.xsd	20 Nov 2007 13:20:07 -0000	1.2
+++ src/main/org/jboss/seam/pages-2.0.xsd	5 Jan 2008 17:48:50 -0000
@@ -124,7 +124,7 @@
     <xs:element name="restrict" type="xs:string"/>
     <xs:element name="navigation">
         <xs:complexType>
-            <xs:choice>
+            <xs:choice maxOccurs="2">
                 <xs:sequence>
                     <xs:choice minOccurs="0">
                         <xs:element ref="pages:begin-conversation"/>

Which makes your test case pass and doesn't alter the ordering requirements of elements

> pages-2.0.xsd handling of <Navigation> doesn't allow standard rules and a null-accepting implicit rule
> ------------------------------------------------------------------------------------------------------
>
>                 Key: JBSEAM-2345
>                 URL: http://jira.jboss.com/jira/browse/JBSEAM-2345
>             Project: JBoss Seam
>          Issue Type: Bug
>    Affects Versions: 2.0.0.GA
>            Reporter: Matt Drees
>         Assigned To: Pete Muir
>            Priority: Minor
>             Fix For: 2.0.1.GA
>
>         Attachments: pages-2.0.xsd-patch.txt
>
>
> pages-2.0.xsd doesn't allow something like this:
> <page view-id="/admin/manage/registrants/confirmDeleteRegistrant.xhtml">
>    <navigation from-action="#{registrantController.deleteSelectedRegistrant}">
>       <rule if="#{not empty registration.existingRegistrants}">
>          <redirect view-id="/admin/manage/registrants/showRegistration.xhtml"/>
>       </rule>
> 	   		
>       <redirect view-id="/admin/registrantSearching.xhtml"/>
>    </navigation>
> </page>
> The XSD file doesn't allow a set of <rule>s *and* an implicit rule, even though Pages.java handles it correctly.

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