From d0c9ca3d483fa8f03099fd01eb8d53c558802cef Mon Sep 17 00:00:00 2001 From: David McKnight Date: Wed, 4 Jun 2008 15:49:15 +0000 Subject: [PATCH] [235471] DStoreHostFile.getParentPath() breaks API contract for Root files --- .../rse/internal/services/dstore/files/DStoreHostFile.java | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/rse/plugins/org.eclipse.rse.services.dstore/src/org/eclipse/rse/internal/services/dstore/files/DStoreHostFile.java b/rse/plugins/org.eclipse.rse.services.dstore/src/org/eclipse/rse/internal/services/dstore/files/DStoreHostFile.java index 1cabc3154e6..5b8d862e64b 100644 --- a/rse/plugins/org.eclipse.rse.services.dstore/src/org/eclipse/rse/internal/services/dstore/files/DStoreHostFile.java +++ b/rse/plugins/org.eclipse.rse.services.dstore/src/org/eclipse/rse/internal/services/dstore/files/DStoreHostFile.java @@ -16,6 +16,7 @@ * Xuan Chen (IBM) - [189041] incorrect file name after rename a file inside a zip file - DStore Windows * Xuan Chen (IBM) - [187548] Editor shows incorrect file name after renaming file on Linux dstore * Kevin Doyle (IBM) - [191548] Various NPE fixes + * David McKnight (IBM) - [235471] DStoreHostFile.getParentPath() breaks API contract for Root files *******************************************************************************/ package org.eclipse.rse.internal.services.dstore.files; @@ -143,6 +144,10 @@ public class DStoreHostFile implements IHostFile, IHostFilePermissionsContainer } else { + if (isRoot()){ // IFileService.getParentPath() contract states a root must return null + return null; + } + if (_element.getName().length() == 0) { // derive from value