[TAP5-2424] Application folder doesn't support "sub folders" (slash in path)

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.4, 5.3.8
• Fix Version/s: 5.4
• Component/s: tapestry-core
• Labels: easyfix newbie patch
• Assignee: Jochen Kemnade
• Reporter: Nicolas Bouillon
• Created: 04/Dec/14 15:54
• Updated: 09/Dec/14 08:20
• Resolved: 08/Dec/14 16:42

Description

I've set my tapstry servlet filter as :

<filter-mapping>
<filter-name>app</filter-name>
<url-pattern>/path1/path2/*</url-pattern>
</filter-mapping>

And in the AppModule

configuration.add(SymbolConstants.APPLICATION_FOLDER, "path1/path2");

It works for Page Requests, but for Event requests it doesn't work.

The probleme is here org.apache.tapestry5.internal.services.ComponentEventLinkEncoderImpl#decodeComponentEventRequest

The method used to remotve the application folder is not the same as the one used in org.apache.tapestry5.internal.services.ComponentEventLinkEncoderImpl#decodePageRenderRequest

In decodeComponentEventRequest, the request path comparison is done between the applicationFolderPrefix and the substring until the first "/". In decodePageRenderRequest, the comparison is done otherwise.