You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "santiago garcía pimentel (JIRA)" <ji...@apache.org> on 2015/01/19 14:42:36 UTC

[jira] [Commented] (SLING-4212) Sling Models: Allow multiple values from ValueMap in the resource-path injector

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

santiago garcía pimentel commented on SLING-4212:
-------------------------------------------------

[~sseifert@pro-vision.de] do you think we can integrate this or is there anything else that you think can be improved?

> Sling Models: Allow multiple values from ValueMap in the resource-path injector
> -------------------------------------------------------------------------------
>
>                 Key: SLING-4212
>                 URL: https://issues.apache.org/jira/browse/SLING-4212
>             Project: Sling
>          Issue Type: Improvement
>          Components: Extensions
>            Reporter: santiago garcía pimentel
>         Attachments: resourcePath-API.patch
>
>
> The current implementation of the resource-path injector does not support multiple values. I think it could be useful to inject a list of paths from the valuemap.
>  I have created a small patch to allow this. Right now it only allows them from the value map since I didn't want to change the API without consulting you first. I you agree I can do this change as well. I also added a test case for it.
> You can see a pull request in https://github.com/apache/sling/pull/51
> If there anything I can do to improve this patch, please let me know.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)