[4269] Rename id to leverage lombok-generated getters

#4269
openrewrite · Jul 31, 2024 · 1db298d · 1db298d
1 parent f9bea1a
commit 1db298d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 13 deletions.
diff --git a/rewrite-maven/src/main/java/org/openrewrite/maven/MavenSettings.java b/rewrite-maven/src/main/java/org/openrewrite/maven/MavenSettings.java
@@ -328,11 +328,6 @@ public static class Profile implements WithProfiles.Profile {
         @Nullable
         RawRepositories repositories;
 
-        @Override
-        public String id() {
-            return id;
-        }
-
         @Override
         public boolean isActive() {
             return activation != null && activation.isActive();

diff --git a/rewrite-maven/src/main/java/org/openrewrite/maven/WithProfiles.java b/rewrite-maven/src/main/java/org/openrewrite/maven/WithProfiles.java
@@ -5,7 +5,7 @@
 
 public interface WithProfiles<P extends WithProfiles.Profile> {
     interface Profile {
-        String id();
+        String getId();
         boolean isActive();
         boolean isActiveByDefault();
     }
@@ -33,9 +33,9 @@ default List<P> activeProfiles(final Iterable<String> userSpecifiedProfiles) {
     }
 
     default boolean isActivated(final P profile, final Iterable<String> userSpecifiedProfiles) {
-        if (profile.id() != null) {
+        if (profile.getId() != null) {
             for (String activeProfile : userSpecifiedProfiles) {
-                if (activeProfile.trim().equals(profile.id())) {
+                if (activeProfile.trim().equals(profile.getId())) {
                     return true;
                 }
             }

diff --git a/rewrite-maven/src/main/java/org/openrewrite/maven/tree/Profile.java b/rewrite-maven/src/main/java/org/openrewrite/maven/tree/Profile.java
@@ -45,11 +45,6 @@ public class Profile implements WithProfiles.Profile {
     List<Plugin> plugins;
     List<Plugin> pluginManagement;
 
-    @Override
-    public String id() {
-        return id;
-    }
-
     @Override
     public boolean isActive() {
         return activation != null && activation.isActive();