Description of problem: Having this in the pages.xml: <page view-id="/entry.xhtml"> <rewrite pattern="/entry/{blogEntryId}" /> <rewrite pattern="/entry" /> <param name="blogEntryId" value="#{blogEntry.id}"/> <action execute="#{entryAction.loadBlogEntry(blogEntry.id)}"/> </page> and a link to entry in a page: <s:link id="link" view="/entry.xhtml" propagation="none" value="#{blogEntry.title}"> <f:param name="blogEntryId" value="#{blogEntry.id}"/> </s:link> this should render as a link to "/entry/foo", but it renders as "/entry?blogEntryId=roadmap11" Version-Release number of selected component (if applicable): WFK 2.2.0.CR1 How reproducible: Always Steps to Reproduce: 1. build and deploy the seam-blog example 2. Go to http://127.0.0.1:8080/seam-blog/ 3. Notice the URL in [Link] link in the Seam Text article. Actual results: http://127.0.0.1:8080/seam-blog/entry?blogEntryId=seamtext Expected results: http://127.0.0.1:8080/seam-blog/entry/seamtext
Interesting observation: removing the '<rewrite pattern="/entry" />' rule makes the other rule work.
This issue goes away with reverted patch for https://bugzilla.redhat.com/show_bug.cgi?id=921660
My previous comment was a mistake, it doesn't work.
Edited release note and inserted it into the 2.2.0 RN book. Docs stage URL: https://documentation-devel.engineering.redhat.com/docs/en-US/JBoss_Web_Framework_Kit/2.2/html-single/2.2.0_Release_Notes/index.html Build: JBoss_Web_Framework_Kit-2.2.0_Release_Notes-2.2-en-US-0-23
Marek Novotny <mnovotny> made a comment on jira JBSEAM-5085 I figured out how to change the incorrect behavior of org.jboss.seam.ui.util.ViewUrlBuilder. The existence of viewHandler.getBookmarkableUrl was premature to call in ViewUrlBuilder initialization, because we need added parameters in URL. So after discussion with Marek Schmidt I prepared the change. However that change broke the calling of page actions due double url encoding - experienced in org.jboss.seam.test.integration.faces.conversations.ConversationPropagationsTest.testSButtonActionPropagationNone(). So I changed default url encoding to false in org.jboss.seam.ui.util.ViewUrlBuilder.ViewUrlBuilder(String, String). The test case for Page rewrite is added as org.jboss.seam.test.integration.faces.ViewUrlBuilderTest.
Marek Novotny <mnovotny> made a comment on jira JBSEAM-5085 I figured out how to change the incorrect behavior of {{org.jboss.seam.ui.util.ViewUrlBuilder}}. The existence of {{viewHandler.getBookmarkableUrl}} was premature to call in {{ViewUrlBuilder}} initialization, because we need added parameters in URL. So after discussion with Marek Schmidt I prepared the change. However that change broke the calling of page actions due double url encoding - experienced in {{org.jboss.seam.test.integration.faces.conversations.ConversationPropagationsTest.testSButtonActionPropagationNone()}}. So I changed default url encoding to false in {{org.jboss.seam.ui.util.ViewUrlBuilder.ViewUrlBuilder(String, String)}}. The test case for Page rewrite is added as {{org.jboss.seam.test.integration.faces.ViewUrlBuilderTest}}.
Marek Novotny <mnovotny> updated the status of jira JBSEAM-5085 to Resolved
back ported the fix as https://source.jboss.org/changelog/Seam?cs=15510 and
Marek Novotny <mnovotny> updated the status of jira JBSEAM-5085 to Closed
Marek Novotny <mnovotny> made a comment on jira JBSEAM-5085 2.3.1.CR1 is now out, so closing all resolved issues
Verified on 2.3.0.ER1 release.
Updated doc text.