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

Jukka Zitting updated JCR-2453:
-------------------------------

    Attachment: JCR-2453.patch

Attached an initial draft of the required changes.

All the shared constants are moved to jackrabbit-webdav.

For now the patch simply copies the relevant code in the shared classes to 
o.a.j.spi2dav.server. More work is needed to avoid the duplication.

> Break the spi2dav dependency to jcr-server
> ------------------------------------------
>
>                 Key: JCR-2453
>                 URL: https://issues.apache.org/jira/browse/JCR-2453
>             Project: Jackrabbit Content Repository
>          Issue Type: Improvement
>          Components: jackrabbit-jcr-server, jackrabbit-spi2dav, 
> jackrabbit-webdav
>            Reporter: Jukka Zitting
>         Attachments: JCR-2453.patch
>
>
> Currently the spi2dav component has a dependency on the jcr-server component, 
> which is troublesome due to the extra transitive dependencies and which 
> strictly speaking should not be necessary from an architectural point of view.
> The dependency exists mostly for sharing a number of JCR-specific WebDAV 
> constants. I'd like to push those constants down to jackrabbit-webdav as they 
> are essentially just shared strings and as jackrabbit-webdav already contains 
> a number of constants used by JCR extensions.
> In addition to constant values, code in the following classes is shared 
> between jcr-server and spi2dav: JcrValueType, NamespacesProperty, 
> NodeTypesProperty, SearchResultProperty, SubscriptionImpl, ValuesProperty. 
> The shared code in JcrValueType and SubscriptionImpl is mostly just about 
> mapping constant value mappings and could fairly easily be moved to 
> jackrabbit-webdav. The Property classes are a but trickier, but it looks like 
> it would be possible to split the code to separate server- and client-side 
> classes for jcr-server and spi2dav.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to