Konrad Windszus created SLING-8084:
--------------------------------------

             Summary: Clarify javadoc for ResourceResolver.map(...)
                 Key: SLING-8084
                 URL: https://issues.apache.org/jira/browse/SLING-8084
             Project: Sling
          Issue Type: Bug
          Components: API
    Affects Versions: API 2.18.6
            Reporter: Konrad Windszus


Currently the javadoc for {{ResourceResolver.map(...)}} 
(https://github.com/apache/sling-org-apache-sling-api/blob/ddeb4ec650813d816a8ffca3ac1eec92f7ea53fb/src/main/java/org/apache/sling/api/resource/ResourceResolver.java#L283)
 does not clearly state whether URI encoding is done within or not.

If I understand 
https://github.com/apache/sling-org-apache-sling-resourceresolver/blob/e017ace802aa55805727bd6f5637939599d19ac4/src/main/java/org/apache/sling/resourceresolver/impl/mapping/ResourceMapperImpl.java#L160
 and 
https://github.com/apache/sling-org-apache-sling-resourceresolver/blob/e017ace802aa55805727bd6f5637939599d19ac4/src/main/java/org/apache/sling/resourceresolver/impl/mapping/ResourceMapperImpl.java#L362
 correctly it will actually do the URI encoding!

It is important to mention that as calling {{ResourceResolver.map}} twice in a 
row might lead to a 404 in the end.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to