Also keep in mind that the redirect tag in faces-config.xml uses an
include-view-params attribute.
Cheers,
Cay
On 12/13/2009 11:38 AM, Dan Allen wrote:
Same story as issue 698. I noticed this during a presentation, and
haven't remembered to submit until now. I should have caught this before
final draft.
When using implicit navigation, you can pass flags through the query
string that
alter the behavior of the navigation. The first is a redirect flag, the
second
is a hint to include view parameters in the outgoing query string.
- faces-redirect
- includeViewParams
As you can see, there are two problems with the includeViewParams flag.
First,
it uses camel case instead of hyphen delimiters between words and it is not
prefixed with faces-.
When this feature was being discussed, there was a concern that the
hints would
collide with query string parameters used by the application. As a result, a
prefix was added to the redirect hint, making it faces-redirect. However,
includeViewParams was not updated.
Here's the following change that needs to happen in the spec:
includeViewParams ==> faces-include-view-params
And in use:
/entry?faces-redirect=true&faces-include-view-params=true
In the long run, we need to find a cleaner (perhaps type-safe) way of
indicating
an implicit navigation. Using the query string really isn't great.
Perhaps under
the covers the implicit navigation is communicated this way, but we
provide a
Java API that generates this in toString()
return new Navigation("entry", true, true);
-Dan
--
Dan Allen
Senior Software Engineer, Red Hat | Author of Seam in Action
Registered Linux User #231597
http://mojavelinux.com
http://mojavelinux.com/seaminaction
http://www.google.com/profiles/dan.j.allen
--
Cay S. Horstmann |