Hi,

I only had a brief look at the current code, and currently it is
exporting the package org.apache.sling.resourceresolver.impl.mapping
which looks wrong to me. If we need new API this should go into the API
bundle. But I would prefer if we can get away without a new interface

Carsten


Andreas Schaefer wrote
> The placeholder was renamed to sling interpolation, changed the delimiter to 
> ${} and added a jUnit test suite where the StringInterpolationProvider 
> replacement is tested through Map Entries. 
> 
> I am wondering if Sling has some Integration Tests (IT). I could not find any 
> further tests regarding /etc/map.
> 
> Cheers - Andreas Schaefer
> 
>> On Jul 12, 2018, at 1:15 AM, Bertrand Delacretaz <[email protected]> 
>> wrote:
>>
>> Hi,
>>
>> On Mon, Jul 9, 2018 at 4:18 PM Andreas Schaefer
>> <[email protected]> wrote:
>>> ...Ruben and I think it could be helpful to make the Placeholder Provider 
>>> more generic...
>>
>> I agree with that and I think "string interpolation" is a better name
>> than "placeholder" which is quite generic - as per
>> https://en.wikipedia.org/wiki/String_interpolation
>>
>> -Bertrand
> 
-- 
Carsten Ziegeler
Adobe Research Switzerland
[email protected]

Reply via email to