You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Konrad Windszus (JIRA)" <ji...@apache.org> on 2017/02/27 08:53:45 UTC

[jira] [Resolved] (SLING-6562) Remove getAdministrativeResourceResolver from Sling Validation

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

Konrad Windszus resolved SLING-6562.
------------------------------------
    Resolution: Fixed

Fixed in [r1784509|http://svn.apache.org/r1784509].
I also refactored the SPI {{ValidationModelProvider}} a bit, so that there is no longer a ResourceResolver being passed as argument. That way each {{ValidationModelProvider}} needs to take care to open its own service resource resolver (if necessary).
Still a resource resolver is necessary in {{2.}} because that one needs to resolve the resource inheritance of the to be validated resource type.

> Remove getAdministrativeResourceResolver from Sling Validation
> --------------------------------------------------------------
>
>                 Key: SLING-6562
>                 URL: https://issues.apache.org/jira/browse/SLING-6562
>             Project: Sling
>          Issue Type: Improvement
>          Components: Validation
>    Affects Versions: Validation 1.0.0
>            Reporter: Konrad Windszus
>            Assignee: Konrad Windszus
>             Fix For: Validation 1.0.0
>
>
> 3 occurrences in
> # {{o.a.s.validation.impl.resourcemodel.ResourceValidationModelProviderImpl}},
> # {{o.a.s.validation.impl.ValidationModelRetrieverImpl}} (most probably the latter should not deal with resource resolvers at all (compare with http://www.mail-archive.com/dev@sling.apache.org/msg64770.html))
> # {{o.a.s.validation.impl.ValidationServiceImpl}} to retrieve the search paths (for relativizing resource types)



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)