[jira] [Commented] (TAP5-1635) ComponentEventLinkEncoderImpl bug

["Alex Lumpov (Commented) (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TAP5-1635?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13115365#comment-13115365 ] 

Alex Lumpov commented on TAP5-1635:
-----------------------------------

To solve this problem we need the following changes in the ComponentEventLinkEncoderImpl.
1. private final String startPageName;
2. public ComponentEventLinkEncoderImpl(..., @Symbol(SymbolConstants.START_PAGE_NAME) String startPageName) {
        ...
        this.startPageName = startPageName.toLowerCase();
    }
3. private String encodePageName(String pageName) {
        if (pageName.equalsIgnoreCase(startPageName))
            return "";

        String encoded = pageName.toLowerCase();

        if (!encoded.endsWith("/" + startPageName))
            return encoded;

        return encoded.substring(0, encoded.length() - 1 - startPageName.length());
    }
                
> ComponentEventLinkEncoderImpl bug
> ---------------------------------
>
>                 Key: TAP5-1635
>                 URL: https://issues.apache.org/jira/browse/TAP5-1635
>             Project: Tapestry 5
>          Issue Type: Bug
>          Components: tapestry-core
>    Affects Versions: 5.2.6
>            Reporter: Alex Lumpov
>            Priority: Minor
>
> Method ComponentEventLinkEncoderImpl.encodePageName
> must omit value of tapestry.start-page-name
> not "index"

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira