werpu commented on PR #502: URL: https://github.com/apache/myfaces/pull/502#issuecomment-1413427633
Thanks for the suggestion, does it really break the external libs? The thing is it basically just prefixes the viewState on the param map lookup if you go in with a standard parameter. I cannot omit that the request parameters are coming in prefixed for the naming container case that is per spec and thats what we have done wrongly so far! But if you pull the values in from the parameter map you have to be prefix aware anyway. The resolver just does this internally in a non intrusive way, is the param coming in prefixed it wont prefix, if it comes in non prefix it tries both a prefix and non prefixed lookup! But either way I will provide a second pull request with the scripts only, so that you can have a look at yourself, and will postpone the merge. Feel free to take over the server side for a better solution! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
