You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-issues@jackrabbit.apache.org by "Michael Dürig (JIRA)" <ji...@apache.org> on 2013/11/27 10:49:37 UTC

[jira] [Updated] (OAK-1232) Improve implementation of Tree.get(Property)Status

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

Michael Dürig updated OAK-1232:
-------------------------------

    Attachment: OAK-1232.patch

At http://svn.apache.org/r1545978 I implemented the property status calculation in {{AbstractTree}} using the new methods from {{NodeBuilder}}. 

However as the attached patch shows using this new implementation causes {{org.apache.jackrabbit.oak.jcr.RepositoryTest#setStringProperty}} to fail with the MongoNS fixture. 

> Improve implementation of Tree.get(Property)Status 
> ---------------------------------------------------
>
>                 Key: OAK-1232
>                 URL: https://issues.apache.org/jira/browse/OAK-1232
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: core
>            Reporter: Michael Dürig
>            Priority: Minor
>         Attachments: OAK-1232.patch
>
>
> OAK-928 introduced methods for determining the status of a property from a {{NodeBuilder}}. The implementations of {{Tree.getPropertyStatus}} should change using these new methods instead of duplicating the logic. 
> Furthermore the since the {{get(Propert)Status}} methods pre-dates the introduction of the {{exists}} method, we should also clarify the effect of calling such methods on a non existing tree.



--
This message was sent by Atlassian JIRA
(v6.1#6144)