[ https://issues.apache.org/jira/browse/SLING-1899?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12972774#action_12972774 ]
Carsten Ziegeler commented on SLING-1899: ----------------------------------------- > But I agree that a map() method that does not do url-encoding would make > sense. I would simply add a new one, make the javadocs on the existing ones > more > explicit, but _not_ deprecate them, since they do have a purpose. It should > not be moved to yet-another utility class. Deprecating does not mean removing, for obvious reasons this method has to stay as is for a long term And no, we're not talking about an utility class here either - I think we should add the methods to the resource resolver; And I agree with Justin, that "map" might not be the best name for a new method. > JcrResourceResolver map method is not reverse operation of resolve method > with special chars in path > ---------------------------------------------------------------------------------------------------- > > Key: SLING-1899 > URL: https://issues.apache.org/jira/browse/SLING-1899 > Project: Sling > Issue Type: Bug > Components: JCR > Affects Versions: JCR Resource 2.0.6 > Reporter: Stefan Seifert > Attachments: 101216_mapresolv_specialchars_fix.patch, > 101216_mapresolve_specialchars_test.patch > > > according to the javadocs of the ResourceResolver API interface, the map > method is intended as the reverse operation of the resolve method. > this is not the case if the path contains special chars like spaces - they > are url-encoded, although the javadocs of the API does not require this. > additionally the resolve method does not url-decode the path (which is > correct in my opinion because the servlet engine already decodes the path). > the map method should be fixed, so that not url-encoding takes place in it. > the application is responsible for url-encoding a path if needed, not the > sling api. > i've attached a patch which contains a simple unit tests that makes the > current implementation fail when resolving a path generated with the map > method: [^101216_mapresolve_specialchars_test.patch] -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.