Merge pull request #77 from cobratbq/fix-bad-filtering-conditional

Require checking scopes and types conditions in all cases.
nicoulaj · May 25, 2020 · 2b18928 · 2b18928
2 parents 8a6b8fd + 1f9f27b
commit 2b18928
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/main/java/net/nicoulaj/maven/plugins/checksum/mojo/DependenciesCheckMojo.java b/src/main/java/net/nicoulaj/maven/plugins/checksum/mojo/DependenciesCheckMojo.java
@@ -214,8 +214,8 @@ private List<Artifact> getArtifactsToProcess()
     Set<Artifact> artifacts = transitive ? project.getArtifacts() : project.getDependencyArtifacts();
     for ( Artifact artifact : artifacts )
     {
-      if ((!allProjectModules.contains(artifact)) || (scopes == null || scopes.contains(artifact.getScope()))
-                                                     && (types == null || types.contains(artifact.getType())))
+      if (!allProjectModules.contains(artifact) && (scopes == null || scopes.contains(artifact.getScope()))
+          && (types == null || types.contains(artifact.getType())))
       {
         getLog().debug(String.format("check artifact %s", artifact));
         result.add(artifact);