[ 
https://issues.apache.org/jira/browse/JCR-1500?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12582204#action_12582204
 ] 

angela commented on JCR-1500:
-----------------------------

and so do I.
the Batch currently does not specify that there a multiple ways to remove a 
property... but it insinuates this usage by not explicitely mentioning the 
expected behaviour and by refering to the corresponding jcr methods. 

Let's clarify that in the javadoc and adjust the current implementations if 
necessary.

> removing properties through SPI: two ways to do it
> --------------------------------------------------
>
>                 Key: JCR-1500
>                 URL: https://issues.apache.org/jira/browse/JCR-1500
>             Project: Jackrabbit
>          Issue Type: Improvement
>          Components: jackrabbit-spi
>            Reporter: Julian Reschke
>            Priority: Minor
>
> Batch currently provides two ways to delete a property, similarly to JCR:
> - Batch.remove()
> - Batch.setValue(..., null)
> JCR2SPI currently uses (AFAIK) Batch.remove().
> Proposal:
> - clarify that the QValue argument in setValue must be non-null (same for 
> setValues)

-- 
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