[Orchestra] TODO in UrlParameterNavigationHandler

[David Brainard]

Hi,

i was looking into the UrlParameterNavigationHandler class i saw a TODO 
with the following comments

// TODO: this only supports one EL expression at the moment; it would be 
nice
   // to support multiple.
protected String interceptRedirect(FacesContext context, String url)

But when i tried to use multiple el expressions in faces-config it seems 
that it is wrking fine....

my faces-config entry and the URL

<navigation-rule>
       <display-name>Main</display-name>
       <from-view-id>/Main.jsp</from-view-id>
       <navigation-case>
           <from-outcome>redirect</from-outcome>
           
<to-view-id>/Redir.jsp?navValue=#{testBean.navValue}&amp;navValue1=#{testBean.navValue1}</to-view-id>
           <redirect />
       </navigation-case>
   </navigation-rule>

http://localhost:8080/Tomahawk/faces/Redir.jsp?navValue=1768707082&navValue1=998817167


navValue and navValue1 are the two parameters..

what i am doin is right???
pls comment..
if it is so i can add sme javadocs and delete the TODO as a patch.

Regards
David Brainard Sounthiraraj