[ 
https://issues.apache.org/jira/browse/SLING-234?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12566546#action_12566546
 ] 

Felix Meschberger commented on SLING-234:
-----------------------------------------

In addition to fixing the parsing (and documenting it), the include(String, 
RequestDispatcherOptions) method removed in Rev. 618697 should be reintroduced.

> RequestDispatcherOption string parsing in SlingScriptHelper.include(String, 
> String)
> -----------------------------------------------------------------------------------
>
>                 Key: SLING-234
>                 URL: https://issues.apache.org/jira/browse/SLING-234
>             Project: Sling
>          Issue Type: Bug
>          Components: API, Scripting
>            Reporter: Felix Meschberger
>            Assignee: Felix Meschberger
>             Fix For: 2.0.0
>
>
> Currently the RequestDispatcherOptions given to the 
> SlingScriptHelper.include(String path, String options) method are parsed as 
> follows:
>     * If the string neither contains a comma nor a colon, the string is 
> interpreted as a resource type. Hence a RequestDispatcherOption object is 
> created setting the "forceResourceType" option to the given string
>     * Otherwise the string is parsed using the comma as a separator for the 
> option list and the colon as a separator between an option name and its value.
> This definition of using the colon as a name/value separator collides with 
> the use of the colon as a namespace separator also used in resource types. We 
> should change this to use the equals sign also used in properties files as 
> the separator between names and values.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to