[ https://issues.jboss.org/browse/JBSEAM-5085?page=com.atlassian.jira.plugi... ] Marek Novotny edited comment on JBSEAM-5085 at 5/17/13 8:50 AM: ---------------------------------------------------------------- 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}}. was (Author: manaRH): 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. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira