With trunk rev 2170 Joel's problem still manifests. I confirmed issue 352 solved, but this is something different. LocalReference.getFileReference() does not make the ./ go away. I think it should:
LocalReference.java:86 < return createFileReference(file.getAbsolutePath()); > return createFileReference(file.getCanonicalPath()); Unless there is a useful reason to create a LocalReference that still contains ./ and ../ etc -- I can't think of one. ----- Original Message ----- From: "Jerome Louvel" <[EMAIL PROTECTED]> To: [email protected] Sent: Tuesday, September 18, 2007 5:13:35 PM (GMT-0500) America/New_York Subject: RE: Bug: DirectoryResource is broken if used with relative paths Hi all, This bug lied in the File connector in is now fixed in SVN. Thanks for testing it. Best regards, Jerome > -----Message d'origine----- > De : Rob Heittman [mailto:[EMAIL PROTECTED] > Envoyé : mardi 18 septembre 2007 15:48 > À : [email protected] > Objet : Re: Bug: DirectoryResource is broken if used with > relative paths > > > Thanks for trying it anyway, Joel, I wasn't at an actual > computer yesterday. > > Something like this works around the issue for me: > > String path = null; > try{ > path=new File("./").getCanonicalPath(); > } catch (IOException handleMePlease) {} > LocalReference lr = LocalReference.createFileReference(path); > Directory directory = new Directory(getContext(),lr); > > You need to get the "." and ".." resolved away, which > getCanonicalPath() > does. > > I think LocalReference should do this for you when you pass a > File as in > your original example -- thoughts, Jerome, Thierry? > > Also, I think LocalReference.getTargetRef() oughta return a > LocalReference > (you'd have to cast it, but this should work). > > - Rob > > > ----- Original Message ----- > From: "Joel Nelson" <[EMAIL PROTECTED]> > To: [email protected] > Sent: Monday, September 17, 2007 8:00:26 PM (GMT-0500) > America/New_York > Subject: Re: Bug: DirectoryResource is broken if used with > relative paths > > On Mon, 17 Sep 2007 11:26:08 -0400, Rob Heittman wrote: > > > 1) > > // Create a directory able to expose a hierarchy of files > > Directory directory = new Directory(getContext(), > > LocalReference.createFileReference(new File("./").getPath())); > > > > This one is equally broken > > > 2) > > // Create a directory able to expose a hierarchy of files > > Directory directory = new Directory(getContext(), > > LocalReference.createFileReference(new File("./")).getTargetRef()); > > This one doesn't compile, "The constructor Directory(Context, > Reference) > is undefined" > >
