Browse Source

Update some code to Java 7

pull/159/head
Decebal Suiu 7 years ago
parent
commit
f5f5587e96
  1. 4
      pf4j/src/main/java/ro/fortsoft/pf4j/DefaultPluginStatusProvider.java
  2. 38
      pf4j/src/main/java/ro/fortsoft/pf4j/util/FileUtils.java

4
pf4j/src/main/java/ro/fortsoft/pf4j/DefaultPluginStatusProvider.java

@ -47,11 +47,11 @@ public class DefaultPluginStatusProvider implements PluginStatusProvider {
private void initialize() { private void initialize() {
try { try {
// create a list with plugin identifiers that should be only accepted by this manager (whitelist from plugins/enabled.txt file) // create a list with plugin identifiers that should be only accepted by this manager (whitelist from plugins/enabled.txt file)
enabledPlugins = FileUtils.readLines(pluginsRoot.resolve("enabled.txt").toFile(), true); enabledPlugins = FileUtils.readLines(pluginsRoot.resolve("enabled.txt"), true);
log.info("Enabled plugins: {}", enabledPlugins); log.info("Enabled plugins: {}", enabledPlugins);
// create a list with plugin identifiers that should not be accepted by this manager (blacklist from plugins/disabled.txt file) // create a list with plugin identifiers that should not be accepted by this manager (blacklist from plugins/disabled.txt file)
disabledPlugins = FileUtils.readLines(pluginsRoot.resolve("disabled.txt").toFile(), true); disabledPlugins = FileUtils.readLines(pluginsRoot.resolve("disabled.txt"), true);
log.info("Disabled plugins: {}", disabledPlugins); log.info("Disabled plugins: {}", disabledPlugins);
} catch (IOException e) { } catch (IOException e) {
log.error(e.getMessage(), e); log.error(e.getMessage(), e);

38
pf4j/src/main/java/ro/fortsoft/pf4j/util/FileUtils.java

@ -19,12 +19,11 @@ import org.slf4j.Logger;
import org.slf4j.LoggerFactory; import org.slf4j.LoggerFactory;
import java.io.BufferedReader; import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.File; import java.io.File;
import java.io.FileFilter; import java.io.FileFilter;
import java.io.FileReader; import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException; import java.io.IOException;
import java.nio.charset.StandardCharsets;
import java.nio.file.FileVisitResult; import java.nio.file.FileVisitResult;
import java.nio.file.Files; import java.nio.file.Files;
import java.nio.file.Path; import java.nio.file.Path;
@ -42,44 +41,28 @@ public class FileUtils {
private static final Logger log = LoggerFactory.getLogger(FileUtils.class); private static final Logger log = LoggerFactory.getLogger(FileUtils.class);
public static List<String> readLines(File file, boolean ignoreComments) throws IOException { public static List<String> readLines(Path path, boolean ignoreComments) throws IOException {
File file = path.toFile();
if (!file.exists() || !file.isFile()) { if (!file.exists() || !file.isFile()) {
return new ArrayList<>(); return new ArrayList<>();
} }
List<String> lines = new ArrayList<>(); List<String> lines = new ArrayList<>();
BufferedReader reader = null; try (BufferedReader reader = new BufferedReader(new FileReader(file))) {
try {
reader = new BufferedReader(new FileReader(file));
String line; String line;
while ((line = reader.readLine()) != null) { while ((line = reader.readLine()) != null) {
if (ignoreComments && !line.startsWith("#") && !lines.contains(line)) { if (ignoreComments && !line.startsWith("#") && !lines.contains(line)) {
lines.add(line); lines.add(line);
} }
} }
} finally {
if (reader != null) {
reader.close();
}
} }
return lines; return lines;
} }
public static void writeLines(Collection<String> lines, File file) throws IOException { public static void writeLines(Collection<String> lines, File file) throws IOException {
BufferedWriter writer = null; Files.write(file.toPath(), lines, StandardCharsets.UTF_8);
try {
writer = new BufferedWriter(new FileWriter(file));
for (String line : lines) {
writer.write(line);
writer.newLine();
}
} finally {
if (writer != null) {
writer.close();
}
}
} }
/** /**
@ -136,7 +119,8 @@ public class FileUtils {
} }
/** /**
* Finds a path with various endings or null if not found * Finds a path with various endings or null if not found.
*
* @param basePath the base name * @param basePath the base name
* @param endings a list of endings to search for * @param endings a list of endings to search for
* @return new path or null if not found * @return new path or null if not found
@ -153,7 +137,8 @@ public class FileUtils {
} }
/** /**
* Delete a file (not recursively) and ignore any errors * Delete a file (not recursively) and ignore any errors.
*
* @param path the path to delete * @param path the path to delete
*/ */
public static void optimisticDelete(Path path) { public static void optimisticDelete(Path path) {
@ -205,9 +190,10 @@ public class FileUtils {
} }
/** /**
* Return true only if path is a zip file * Return true only if path is a zip file.
*
* @param path to a file/dir * @param path to a file/dir
* @return true if file with .zip ending * @return true if file with {@code .zip} ending
*/ */
public static boolean isZipFile(Path path) { public static boolean isZipFile(Path path) {
return Files.isRegularFile(path) && path.toString().toLowerCase().endsWith(".zip"); return Files.isRegularFile(path) && path.toString().toLowerCase().endsWith(".zip");

Loading…
Cancel
Save