jackrabbit-oak-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michael Dürig <mdue...@apache.org>
Subject Re: Branch.move contract
Date Tue, 17 Apr 2012 16:20:05 GMT


Hi,

On 17.4.12 16:15, Julian Reschke wrote:
> Hi,
>
> I just noticed that move() defaults to "do nothing" for all kinds of
> error conditions:
>
> - source not existing
>
> - parent of target not existing
>
> - target existing
>
> At least the latter will require the JCR layer to do an existence check
> on the target before proceeding.
>
> Is this intentional? What's the rational?

It is intentional in so far, that I wanted to avoid having every one 
doing these kind of checks all over again.

We might want to add a utility class which does this checks on top of 
the current interface.

Michael

>
> Best regards, Julian

Mime
View raw message