fix: [BUG] - Download selected documents / folders : Zip creation iss…

…ue and empty zip - EXO-72976 (#1271) (#1274)
exoplatform · Jul 11, 2024 · cc3fad9 · cc3fad9
1 parent 6a7ec99
commit cc3fad9
Showing 1 changed file with 3 additions and 22 deletions.
diff --git a/...torage-jcr/src/main/java/org/exoplatform/documents/storage/jcr/util/JCRDocumentsUtil.java b/...torage-jcr/src/main/java/org/exoplatform/documents/storage/jcr/util/JCRDocumentsUtil.java
@@ -860,7 +860,7 @@ public static void createFile(Node node,
     Node jrcNode = node.getNode("jcr:content");
     InputStream inputStream = jrcNode.getProperty("jcr:data").getStream();
     String path = "";
-    String nodePath = getPath(node);
+    String nodePath = node.getPath();
     if (StringUtils.isNotEmpty(symlinkPath) || StringUtils.isNotEmpty(sourcePath)) {
       nodePath = symlinkPath + nodePath.replace(sourcePath, "");
     }
@@ -885,7 +885,7 @@ public static void createTempFilesAndFolders(Node node,
       return;
     }
     if (JCRDocumentsUtil.isFolder(node)) {
-      String nodePath = getPath(node);
+      String nodePath = node.getPath();
       if (StringUtils.isNotEmpty(symlinkPath) || StringUtils.isNotEmpty(sourcePath)) {
         nodePath = symlinkPath + nodePath.replace(sourcePath, "");
       }
@@ -901,32 +901,13 @@ public static void createTempFilesAndFolders(Node node,
         String sourceID = node.getProperty(NodeTypeConstants.EXO_SYMLINK_UUID).getString();
         Node sourceNode = JCRDocumentsUtil.getNodeByIdentifier(node.getSession(), sourceID);
         if (sourceNode != null) {
-          createTempFilesAndFolders(sourceNode, getPath(node), getPath(sourceNode), tempFolderPath, parentPath);
+          createTempFilesAndFolders(sourceNode, node.getPath(), sourceNode.getPath(), tempFolderPath, parentPath);
         }
       } else {
         createFile(node, symlinkPath, sourcePath, tempFolderPath, parentPath);
       }
     }
   }
-
-  static String getPath(Node node) {
-    Node parent = node;
-    LinkedList<String> parents = new LinkedList<String>();
-    while (parent != null) {
-      try {
-        if (parent.hasProperty(NodeTypeConstants.EXO_TITLE)) {
-          parents.addFirst(Utils.getStringProperty(parent, NodeTypeConstants.EXO_TITLE));
-        } else {
-          parents.addFirst(parent.getName());
-        }
-        parent = parent.getParent();
-      } catch (RepositoryException e) {
-        parent = null;
-      }
-    }
-    return String.join("/", parents);
-  }
-
   public static void cleanFiles(File file) {
     File[] files = file.listFiles();
     if (files != null) {