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

Carsten Ziegeler commented on SLING-4807:
-----------------------------------------

[[email protected]] I think the code was actually correct :) The method 
is called from two places: merge and getEffectiveModel - variable replacement 
should not take place when called from merge, therefore the null check. So I 
think we should revert your change. I thought I added tests for this, but maybe 
I did not :(

> Variables in configurations are not replaced
> --------------------------------------------
>
>                 Key: SLING-4807
>                 URL: https://issues.apache.org/jira/browse/SLING-4807
>             Project: Sling
>          Issue Type: Bug
>          Components: Tooling
>    Affects Versions: Sling Provisioning Model 1.0.0, Sling Provisioning Model 
> 1.1.0, Sling Provisioning Model 1.2.0
>            Reporter: Carsten Ziegeler
>            Assignee: Carsten Ziegeler
>             Fix For: Sling Provisioning Model 1.3.0
>
>
> If a configuration contains variables, these are not replaced. We should add 
> support for it



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

Reply via email to