[MNG-7416] Simplify Boolean expressions and returns (#333)

Use this link to re-run the recipe: https://app.moderne.io/recipes/builder/SEvWu02zw?organizationId=QXBhY2hlIE1hdmVu Co-authored-by: Moderne <team@moderne.io>
apache · Sep 26, 2023 · 20d5424 · 20d5424
1 parent 33d385e
commit 20d5424
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 26 deletions.
diff --git a/...resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultMetadataResolver.java b/...resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultMetadataResolver.java
@@ -435,11 +435,8 @@ private boolean isEnabled(RemoteRepository repository, Metadata.Nature nature) {
                 && repository.getPolicy(false).isEnabled()) {
             return true;
         }
-        if (!Metadata.Nature.RELEASE.equals(nature)
-                && repository.getPolicy(true).isEnabled()) {
-            return true;
-        }
-        return false;
+        return !Metadata.Nature.RELEASE.equals(nature)
+                && repository.getPolicy(true).isEnabled();
     }
 
     private RepositoryPolicy getPolicy(

diff --git a/maven-resolver-util/src/main/java/org/eclipse/aether/util/artifact/ArtifactIdUtils.java b/maven-resolver-util/src/main/java/org/eclipse/aether/util/artifact/ArtifactIdUtils.java
@@ -165,10 +165,7 @@ public static boolean equalsId(Artifact artifact1, Artifact artifact2) {
         if (!Objects.equals(artifact1.getClassifier(), artifact2.getClassifier())) {
             return false;
         }
-        if (!Objects.equals(artifact1.getVersion(), artifact2.getVersion())) {
-            return false;
-        }
-        return true;
+        return Objects.equals(artifact1.getVersion(), artifact2.getVersion());
     }
 
     /**
@@ -196,10 +193,7 @@ public static boolean equalsBaseId(Artifact artifact1, Artifact artifact2) {
         if (!Objects.equals(artifact1.getClassifier(), artifact2.getClassifier())) {
             return false;
         }
-        if (!Objects.equals(artifact1.getBaseVersion(), artifact2.getBaseVersion())) {
-            return false;
-        }
-        return true;
+        return Objects.equals(artifact1.getBaseVersion(), artifact2.getBaseVersion());
     }
 
     /**
@@ -225,9 +219,6 @@ public static boolean equalsVersionlessId(Artifact artifact1, Artifact artifact2
         if (!Objects.equals(artifact1.getExtension(), artifact2.getExtension())) {
             return false;
         }
-        if (!Objects.equals(artifact1.getClassifier(), artifact2.getClassifier())) {
-            return false;
-        }
-        return true;
+        return Objects.equals(artifact1.getClassifier(), artifact2.getClassifier());
     }
 }
diff --git a/...esolver-util/src/main/java/org/eclipse/aether/util/filter/ExclusionsDependencyFilter.java b/...esolver-util/src/main/java/org/eclipse/aether/util/filter/ExclusionsDependencyFilter.java
@@ -65,11 +65,7 @@ public boolean accept(DependencyNode node, List<DependencyNode> parents) {
 
         id = dependency.getArtifact().getGroupId() + ':' + id;
 
-        if (excludes.contains(id)) {
-            return false;
-        }
-
-        return true;
+        return !(excludes.contains(id));
     }
 
     @Override

diff --git a/...til/src/main/java/org/eclipse/aether/util/graph/selector/ExclusionDependencySelector.java b/...til/src/main/java/org/eclipse/aether/util/graph/selector/ExclusionDependencySelector.java
@@ -90,10 +90,7 @@ private boolean matches(Exclusion exclusion, Artifact artifact) {
         if (!matches(exclusion.getExtension(), artifact.getExtension())) {
             return false;
         }
-        if (!matches(exclusion.getClassifier(), artifact.getClassifier())) {
-            return false;
-        }
-        return true;
+        return matches(exclusion.getClassifier(), artifact.getClassifier());
     }
 
     private boolean matches(String pattern, String value) {