Skip to content

Commit f058495

Browse files
author
Richard Hua
authored
Merge pull request #443 from Microsoft/fix-not-empty-directory-warning
Fix ENOTEMPTY warning
2 parents 12efdb0 + 9299748 commit f058495

File tree

1 file changed

+6
-22
lines changed

1 file changed

+6
-22
lines changed

android/app/src/main/java/com/microsoft/codepush/react/FileUtils.java

+6-22
Original file line numberDiff line numberDiff line change
@@ -54,25 +54,11 @@ public static void copyDirectoryContents(String sourceDirectoryPath, String dest
5454
}
5555
}
5656

57-
public static void deleteDirectory(File directory) {
58-
if (directory.exists()) {
59-
File[] files = directory.listFiles();
60-
if (files != null) {
61-
for (File file : files) {
62-
if(file.isDirectory()) {
63-
deleteDirectory(file);
64-
}
65-
else {
66-
file.delete();
67-
}
68-
}
69-
}
70-
}
71-
directory.delete();
72-
}
73-
7457
public static void deleteDirectoryAtPath(String directoryPath) {
75-
deleteDirectory(new File(directoryPath));
58+
File file = new File(directoryPath);
59+
if (file.exists()) {
60+
deleteFileOrFolderSilently(file);
61+
}
7662
}
7763

7864
public static void deleteFileAtPathSilently(String path) {
@@ -86,9 +72,7 @@ public static void deleteFileOrFolderSilently(File file) {
8672
if (fileEntry.isDirectory()) {
8773
deleteFileOrFolderSilently(fileEntry);
8874
} else {
89-
if (!file.delete()) {
90-
fileEntry.delete();
91-
}
75+
fileEntry.delete();
9276
}
9377
}
9478
}
@@ -147,7 +131,7 @@ public static void unzipFile(File zipFile, String destination) throws IOExceptio
147131

148132
File destinationFolder = new File(destination);
149133
if (destinationFolder.exists()) {
150-
deleteDirectory(destinationFolder);
134+
deleteFileOrFolderSilently(destinationFolder);
151135
}
152136

153137
destinationFolder.mkdirs();

0 commit comments

Comments
 (0)