Minor improvements

diff --git a/pf4j/src/main/java/ro/fortsoft/pf4j/DefaultPluginDescriptorFinder.java b/pf4j/src/main/java/ro/fortsoft/pf4j/DefaultPluginDescriptorFinder.java
index 8b7eb379e44017cdfafbd092a37be9ed6a317b70..2696fcdaab677a9b4793318f3c64ce8f572647ee 100644 (file)
--- a/pf4j/src/main/java/ro/fortsoft/pf4j/DefaultPluginDescriptorFinder.java
+++ b/pf4j/src/main/java/ro/fortsoft/pf4j/DefaultPluginDescriptorFinder.java
@@ -51,7 +51,7 @@ public class DefaultPluginDescriptorFinder extends ManifestPluginDescriptorFinde
         Path manifestPath = pluginPath.resolve(Paths.get(classes,"/META-INF/MANIFEST.MF"));
         log.debug("Lookup plugin descriptor in '{}'", manifestPath);
         if (Files.notExists(manifestPath)) {
-            throw new PluginException("Cannot find '" + manifestPath + "' path");
+            throw new PluginException("Cannot find '{}' path", manifestPath);
         }
 
         try (InputStream input = Files.newInputStream(manifestPath)) {

diff --git a/pf4j/src/main/java/ro/fortsoft/pf4j/DependencyResolver.java b/pf4j/src/main/java/ro/fortsoft/pf4j/DependencyResolver.java
index c26881cafc3ba99e249d13febb03afbaf07dcfb8..d45f677a76e533e439231076dce6fb32a71ac9e1 100644 (file)
--- a/pf4j/src/main/java/ro/fortsoft/pf4j/DependencyResolver.java
+++ b/pf4j/src/main/java/ro/fortsoft/pf4j/DependencyResolver.java
@@ -71,7 +71,7 @@ public class DependencyResolver {
                List<String> pluginsId = dependenciesGraph.reverseTopologicalSort();
 
                if (pluginsId == null) {
-                       throw new PluginException("Cyclic dependencies !!!" + dependenciesGraph.toString());
+                       throw new PluginException("Cyclic dependencies !!! {}", dependenciesGraph.toString());
                }
 
                log.debug("Plugins order: {}", pluginsId);

diff --git a/pf4j/src/main/java/ro/fortsoft/pf4j/PluginManager.java b/pf4j/src/main/java/ro/fortsoft/pf4j/PluginManager.java
index 2ed5b24f9a55afedb2dd128cf54602a8ffcb02e9..e7f7f58bbe91f1f2e0d8929a2d23338d5f4d35f3 100644 (file)
--- a/pf4j/src/main/java/ro/fortsoft/pf4j/PluginManager.java
+++ b/pf4j/src/main/java/ro/fortsoft/pf4j/PluginManager.java
@@ -178,5 +178,6 @@ public interface PluginManager {
      * Gets the path of the folder where plugins are installed
      * @return Path of plugins root
      */
-    public Path getPluginsRoot();
+    Path getPluginsRoot();
+
 }

diff --git a/pf4j/src/main/java/ro/fortsoft/pf4j/PropertiesPluginDescriptorFinder.java b/pf4j/src/main/java/ro/fortsoft/pf4j/PropertiesPluginDescriptorFinder.java
index 8af426975af83c2d986fa24a4f286cd0d1c57e63..7484b4d59d432ecd565f4307aab5a6283ca510aa 100644 (file)
--- a/pf4j/src/main/java/ro/fortsoft/pf4j/PropertiesPluginDescriptorFinder.java
+++ b/pf4j/src/main/java/ro/fortsoft/pf4j/PropertiesPluginDescriptorFinder.java
@@ -59,7 +59,7 @@ public class PropertiesPluginDescriptorFinder implements PluginDescriptorFinder
         Path propertiesPath = pluginPath.resolve(Paths.get(propertiesFileName));
         log.debug("Lookup plugin descriptor in '{}'", propertiesPath);
         if (Files.notExists(propertiesPath)) {
-            throw new PluginException("Cannot find '" + pluginPath + "' path");
+            throw new PluginException("Cannot find '{}' path", pluginPath);
         }
 
         Properties properties = new Properties();