]> source.dussan.org Git - pf4j.git/commitdiff
Fix some warnings (javadoc, pom, sonar)
authorDecebal Suiu <decebal.suiu@gmail.com>
Wed, 1 Feb 2023 19:10:19 +0000 (21:10 +0200)
committerDecebal Suiu <decebal.suiu@gmail.com>
Wed, 1 Feb 2023 19:10:19 +0000 (21:10 +0200)
pf4j/src/main/java/org/pf4j/AbstractExtensionFinder.java
pf4j/src/main/java/org/pf4j/AbstractPluginManager.java
pf4j/src/main/java/org/pf4j/ClassLoadingStrategy.java
pf4j/src/main/java/org/pf4j/util/ClassUtils.java
pf4j/src/main/java/org/pf4j/util/StringUtils.java
pom.xml

index 8288ddc83a724f2ed4551bd7b8cf0d0f7a2b48eb..998a29dfccb38e16eeba4bf172a9012151d53759 100644 (file)
@@ -41,7 +41,7 @@ public abstract class AbstractExtensionFinder implements ExtensionFinder, Plugin
     protected volatile Map<String, ExtensionInfo> extensionInfos; // cache extension infos by class name
     protected Boolean checkForExtensionDependencies = null;
 
-    public AbstractExtensionFinder(PluginManager pluginManager) {
+    protected AbstractExtensionFinder(PluginManager pluginManager) {
         this.pluginManager = pluginManager;
     }
 
@@ -50,7 +50,6 @@ public abstract class AbstractExtensionFinder implements ExtensionFinder, Plugin
     public abstract Map<String, Set<String>> readClasspathStorages();
 
     @Override
-    @SuppressWarnings("unchecked")
     public <T> List<ExtensionWrapper<T>> find(Class<T> type) {
         log.debug("Finding extensions of extension point '{}'", type.getName());
         Map<String, Set<String>> entries = getEntries();
index d57b12e9195d50dbd75f52ec52b53b6e9d5be793..f56a94f1146bbb802c918b166c27ceffa56be769 100644 (file)
@@ -113,7 +113,7 @@ public abstract class AbstractPluginManager implements PluginManager {
     /**
      * The plugins roots are supplied as comma-separated list by {@code System.getProperty("pf4j.pluginsDir", "plugins")}.
      */
-    public AbstractPluginManager() {
+    protected AbstractPluginManager() {
         initialize();
     }
 
@@ -122,7 +122,7 @@ public abstract class AbstractPluginManager implements PluginManager {
      *
      * @param pluginsRoots the roots to search for plugins
      */
-    public AbstractPluginManager(Path... pluginsRoots) {
+    protected AbstractPluginManager(Path... pluginsRoots) {
         this(Arrays.asList(pluginsRoots));
     }
 
@@ -131,7 +131,7 @@ public abstract class AbstractPluginManager implements PluginManager {
      *
      * @param pluginsRoots the roots to search for plugins
      */
-    public AbstractPluginManager(List<Path> pluginsRoots) {
+    protected AbstractPluginManager(List<Path> pluginsRoots) {
         this.pluginsRoots.addAll(pluginsRoots);
 
         initialize();
@@ -877,10 +877,10 @@ public abstract class AbstractPluginManager implements PluginManager {
 
         return pluginWrapper;
     }
-    
+
     /**
      * creates the plugin wrapper. override this if you want to prevent plugins having full access to the plugin manager
-     * 
+     *
      * @return
      */
     protected PluginWrapper createPluginWrapper(PluginDescriptor pluginDescriptor, Path pluginPath, ClassLoader pluginClassLoader) {
@@ -933,7 +933,7 @@ public abstract class AbstractPluginManager implements PluginManager {
     /**
      * Set to true to allow requires expression to be exactly x.y.z.
      * The default is false, meaning that using an exact version x.y.z will
-     * implicitly mean the same as >=x.y.z
+     * implicitly mean the same as &gt;=x.y.z
      *
      * @param exactVersionAllowed set to true or false
      */
index 7ef3952f95604e50c8dfbd535153cf882e40a75e..3dfe4259235a40bb6616c62ae566cd7355a590e8 100644 (file)
@@ -30,32 +30,32 @@ import java.util.List;
 public class ClassLoadingStrategy {
 
     /**
-     * application(parent) -> plugin -> dependencies
+     * application(parent) -&gt; plugin -&gt; dependencies
      */
     public static final ClassLoadingStrategy APD = new ClassLoadingStrategy(Arrays.asList(Source.APPLICATION, Source.PLUGIN, Source.DEPENDENCIES));
 
     /**
-     * application(parent) -> dependencies -> plugin
+     * application(parent) -&gt; dependencies -&gt; plugin
      */
     public static final ClassLoadingStrategy ADP = new ClassLoadingStrategy(Arrays.asList(Source.APPLICATION, Source.DEPENDENCIES, Source.PLUGIN));
 
     /**
-     * plugin -> application(parent) -> dependencies
+     * plugin -&gt; application(parent) -&gt; dependencies
      */
     public static final ClassLoadingStrategy PAD = new ClassLoadingStrategy(Arrays.asList(Source.PLUGIN, Source.APPLICATION, Source.DEPENDENCIES));
 
     /**
-     * dependencies -> application(parent) -> plugin
+     * dependencies -&gt; application(parent) -&gt; plugin
      */
     public static final ClassLoadingStrategy DAP = new ClassLoadingStrategy(Arrays.asList(Source.DEPENDENCIES, Source.APPLICATION, Source.PLUGIN));
 
     /**
-     * dependencies -> plugin -> application(parent)
+     * dependencies -&gt; plugin -&gt; application(parent)
      */
     public static final ClassLoadingStrategy DPA = new ClassLoadingStrategy(Arrays.asList(Source.DEPENDENCIES, Source.PLUGIN, Source.APPLICATION));
 
     /**
-     * plugin -> dependencies -> application(parent)
+     * plugin -&gt; dependencies -&gt; application(parent)
      */
     public static final ClassLoadingStrategy PDA = new ClassLoadingStrategy(Arrays.asList(Source.PLUGIN, Source.DEPENDENCIES, Source.APPLICATION));
 
index 97daa27998d36608a533a13c6a4ce9b706d6e17b..c64ea4720130cb5b512b24a4cf6745643d45a5f0 100644 (file)
@@ -28,6 +28,8 @@ import java.util.Map;
  */
 public class ClassUtils {
 
+    private ClassUtils() {}
+
     public static List<String> getAllInterfacesNames(Class<?> aClass) {
         return toString(getAllInterfaces(aClass));
     }
@@ -56,26 +58,6 @@ public class ClassUtils {
         return list;
     }
 
-    /*
-    public static List<String> getAllAbstractClassesNames(Class<?> aClass) {
-        return toString(getAllInterfaces(aClass));
-    }
-
-    public static List getAllAbstractClasses(Class aClass) {
-        List<Class<?>> list = new ArrayList<>();
-
-        Class<?> superclass = aClass.getSuperclass();
-        while (superclass != null) {
-            if (Modifier.isAbstract(superclass.getModifiers())) {
-                list.add(superclass);
-            }
-            superclass = superclass.getSuperclass();
-        }
-
-        return list;
-    }
-    */
-
     /**
      * Get a certain annotation of a {@link TypeElement}.
      * See <a href="https://stackoverflow.com/a/10167558">stackoverflow.com</a> for more information.
@@ -96,13 +78,6 @@ public class ClassUtils {
         return null;
     }
 
-    /*
-    public static Element getAnnotationMirrorElement(TypeElement typeElement, Class<?> annotationClass) {
-        AnnotationMirror annotationMirror = getAnnotationMirror(typeElement, annotationClass);
-        return annotationMirror != null ? annotationMirror.getAnnotationType().asElement() : null;
-    }
-    */
-
     /**
      * Get a certain parameter of an {@link AnnotationMirror}.
      * See <a href="https://stackoverflow.com/a/10167558">stackoverflow.com</a> for more information.
@@ -139,9 +114,6 @@ public class ClassUtils {
 
     /**
      * Uses {@link Class#getSimpleName()} to convert from {@link Class} to {@link String}.
-     *
-     * @param classes
-     * @return
      */
     private static List<String> toString(List<Class<?>> classes) {
         List<String> list = new ArrayList<>();
index 72087ca96479d9c4d5e1c1dfba58915b75c6a743..aa84ff48cc05cd7af2822f7e60340f66b253fd24 100644 (file)
@@ -20,6 +20,8 @@ package org.pf4j.util;
  */
 public class StringUtils {
 
+    private StringUtils() {}
+
     public static boolean isNullOrEmpty(String str) {
         return (str == null) || str.isEmpty();
     }
diff --git a/pom.xml b/pom.xml
index f19d17074bec0c68405df32f33467b2df3b698f7..97e2c30a97b5077b62e0964f5e7ad26b6368b518 100644 (file)
--- a/pom.xml
+++ b/pom.xml
@@ -19,7 +19,7 @@
     <licenses>
         <license>
             <name>The Apache Software License, Version 2.0</name>
-            <url>http://www.apache.org/licenses/LICENSE-2.0.txt</url>
+            <url>https://www.apache.org/licenses/LICENSE-2.0.txt</url>
             <distribution>repo</distribution>
         </license>
     </licenses>
@@ -66,6 +66,7 @@
         <pluginManagement>
             <plugins>
                 <plugin>
+                    <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-compiler-plugin</artifactId>
                     <version>3.8.0</version>
                     <configuration>
                 </plugin>
 
                 <plugin>
+                    <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-surefire-plugin</artifactId>
                     <version>2.22.1</version>
                 </plugin>
 
                 <plugin>
+                    <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-jar-plugin</artifactId>
                     <version>2.6</version>
                 </plugin>
             </plugin>
 
             <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-jar-plugin</artifactId>
                 <configuration>
                     <archive>