[Yanel-dev] New Patch for Yarep: Bugfix

[Michael Wechner]

On 10/24/10 6:03 PM, Balz Schreier wrote:
> Hi,
>
> I have fixed a bug in Yarep in the existsNode() method.
> Some weeks ago, splited file paths were introduced. It was implemented =

> to be backwards compatible (see getNode() method), but backwards =

> compatibility was not considered in the exists-method.

you mean if you have a mix of splitted and non-splitted paths?

IIRC Claudio implemented something like an automagic data migration for such
cases, or did I miss something? I need to have a closer look at the code =

again ....

>
> Please let me know by when this gets submitted to yarep svn.

I cannot tell you right now :-(

Cheers

Michael
>
> Cheers
> - B
>
> Index: =

> src/impl/java/org/wyona/yarep/impl/repo/vfs/VirtualFileSystemRepository.j=
ava
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> --- =

> src/impl/java/org/wyona/yarep/impl/repo/vfs/VirtualFileSystemRepository.j=
ava(revision =

> 54092)
> +++ =

> src/impl/java/org/wyona/yarep/impl/repo/vfs/VirtualFileSystemRepository.j=
ava(working =

> copy)
> @@ -396,7 +396,7 @@
>              path =3D path.substring(0, path.length() - 1);
>          }
>          if (splitPathEnabled) {
> -            return map.exists(new Path(splitPath(path)));
> +            return map.exists(new Path(splitPath(path))) || =

> map.exists(new Path(path));
>          } else {
>              return map.exists(new Path(path));
>          }
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.wyona.org/pipermail/yanel-development/attachments/2010102=
5/86d2038a/attachment.htm