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

Bertrand Delacretaz commented on SLING-4825:
--------------------------------------------

Your patch looks good to me but it requires building the jackrabbit-jcr-server 
snapshot locally, if I remove 
$HOME.m2/repository/org/apache/jackrabbit/jackrabbit-jcr-server before building 
it fails.

Could you arrange with the jackrabbit team to have that snapshot deployed, or 
better a release of the jackrabbit-jcr-server module with the required changes? 
I'll wait for that to apply your patch, as our Jenkins builds will fail 
otherwise.

Also, if you have suggestions (or a patch...) to test this new feature that 
would be cool, for now we have some tests for those handlers in 
SlingWebDavServletTest [1], using some handlers defined in the 
launchpad/test-services module.



> Add ability to handle webdav based DELETE requests
> --------------------------------------------------
>
>                 Key: SLING-4825
>                 URL: https://issues.apache.org/jira/browse/SLING-4825
>             Project: Sling
>          Issue Type: Improvement
>          Components: JCR
>            Reporter: Satya Deep Maheshwari
>            Assignee: Bertrand Delacretaz
>
> Recently 'delete handler' was added in jackrabbit. This provides the ability 
> to add handlers for handling webdav based 'delete' requests. See JCR-3884 
> containing details on this feature. Earlier this ability was just available 
> for other webdav operations like to copy/move etc. but was missing for 
> 'delete'  . 
> Sling has written its own managers to keep track of these handlers. See [1] 
> which has pre-existing handler managers like  SlingCopyMoveManager. 
> The intent of this jira issue is to add a delete handler manager so that the 
> delete handler functionality can now be leveraged in Sling.
> [1] - 
> https://github.com/apache/sling/tree/trunk/bundles/jcr/webdav/src/main/java/org/apache/sling/jcr/webdav/impl/handler



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

Reply via email to