[ 
https://issues.apache.org/jira/browse/SLING-11506?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Konrad Windszus updated SLING-11506:
------------------------------------
    Description: 
In SLING-4340 the merging of resource types has been changed so that it gives 
the resourceType property precedence over {{Resource.getResourceType()}} 
(https://github.com/apache/sling-org-apache-sling-resourcemerger/blob/f821f844ba675e3850189debf8e45239026c5709/src/main/java/org/apache/sling/resourcemerger/impl/MergedResource.java#L74-L92).

This change is questionable and kind of unexpected from the user's side and 
also leads to undesired side effects like outlined in SLING-11504.

So in the best case the merging of resource types doesn't rely on properties 
(which are implementation details of the underlying resolvers) but clearly 
derives from either the lowest or highest resource without breaking the use 
cases outlined in SLING-4340.

  was:
In SLING-4340 the merging of resource types has been changed so that it gives 
the resourceType property precedence over {{Resource.getResourceType()}} in 

This change is questionable and kind of unexpected from the user's side and 
also leads to undesired side effects like outlined in SLING-11504.

So in the best case the merging of resource types doesn't rely on properties 
(which are implementation details of the underlying resolvers) but clearly 
derives from either the lowest or highest resource without breaking the use 
cases outlined in SLING-4340.


> Streamline and document resource (super) type merging
> -----------------------------------------------------
>
>                 Key: SLING-11506
>                 URL: https://issues.apache.org/jira/browse/SLING-11506
>             Project: Sling
>          Issue Type: Improvement
>          Components: Extensions
>    Affects Versions: Resource Merger 1.4.0
>            Reporter: Konrad Windszus
>            Priority: Major
>
> In SLING-4340 the merging of resource types has been changed so that it gives 
> the resourceType property precedence over {{Resource.getResourceType()}} 
> (https://github.com/apache/sling-org-apache-sling-resourcemerger/blob/f821f844ba675e3850189debf8e45239026c5709/src/main/java/org/apache/sling/resourcemerger/impl/MergedResource.java#L74-L92).
> This change is questionable and kind of unexpected from the user's side and 
> also leads to undesired side effects like outlined in SLING-11504.
> So in the best case the merging of resource types doesn't rely on properties 
> (which are implementation details of the underlying resolvers) but clearly 
> derives from either the lowest or highest resource without breaking the use 
> cases outlined in SLING-4340.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to