[Orchestra] TODO in UrlParameterNavigationHandler

2007-10-30 Thread 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

Re: [Orchestra] TODO in UrlParameterNavigationHandler

2007-10-30 Thread Mike Kienenberger
Note: pure speculation. Possibly it's due to the misunderstanding that the #{blah} part of the string is the EL expression. Any string is an EL expression, and #{} is simply one operator that can be used in that string. So #{a} text #{b} text #{c} is one EL expression, not three. On 10/30/07,

Re: [Orchestra] TODO in UrlParameterNavigationHandler

2007-10-30 Thread Mario Ivankovits
Hi! // TODO: this only supports one EL expression at the moment; it would be nice // to support multiple. I do not exactly know why we stated that. In fact, the TODO should be: // TODO: Parse every #{} one by one, correctly convert it to a string (e.g. by using converters) and url encode them

Re: [Orchestra] TODO in UrlParameterNavigationHandler

2007-10-30 Thread David Brainard
Mario Ivankovits wrote: Hi! // TODO: this only supports one EL expression at the moment; it would be nice // to support multiple. I do not exactly know why we stated that. In fact, the TODO should be: // TODO: Parse every #{} one by one, correctly convert it to a string (e.g. by using