Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Using f:viewAction to navigate into a flow via redirect misses required query parameters #1403

Closed
eclipse-faces-bot opened this issue Aug 26, 2015 · 6 comments

Comments

@eclipse-faces-bot
Copy link

UIViewAction says:

Spec> If a navigation case is matched that causes the new viewId to be
Spec> different from the current viewId, the runtime must force a
Spec> redirect to that matched navigation case with different viewId,
Spec> regardless of whether or not the matched navigation case with
Spec> different viewId called for a redirect.

When using <f:viewAction> to enter a flow, this is functionally
equivalent to using GET based navigation to enter the flow, however,
this code path misses out on attaching the necessary parameters required
by FlowHandler.clientWindowTransition() to successfully enter the flow
in the GET based navigation case. Specifically, the
FlowHandler.TO_FLOW_DOCUMENT_ID_REQUEST_PARAM_NAME and the
FlowHandler.FLOW_ID_REQUEST_PARAM_NAME must be added to the query string
when entering the flow in this way.

The trick here will be to find the right place to specify that these
must be tacked on.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Reported by @edburns

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
In progress.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
File: changebundle.txt
Attached By: @edburns

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Marked as fixed on Monday, August 31st 2015, 3:48:39 pm

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
This issue was imported from java.net JIRA JAVASERVERFACES_SPEC_PUBLIC-1403

@eclipse-faces-bot
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant