Fix 'string builder buffer' can be replaced with 'string'

Vincent Potucek · Vincent Potucek · commit 5ad598d831da · 2025-05-31T12:22:06.000+02:00
diff --git a/compat/maven-artifact/src/main/java/org/apache/maven/artifact/repository/ArtifactRepositoryPolicy.java b/compat/maven-artifact/src/main/java/org/apache/maven/artifact/repository/ArtifactRepositoryPolicy.java
@@ -132,15 +132,7 @@ public boolean checkOutOfDate(Date lastModified) {
 
     @Override
     public String toString() {
-        StringBuilder buffer = new StringBuilder(64);
-        buffer.append("{enabled=");
-        buffer.append(enabled);
-        buffer.append(", checksums=");
-        buffer.append(checksumPolicy);
-        buffer.append(", updates=");
-        buffer.append(updatePolicy);
-        buffer.append('}');
-        return buffer.toString();
+        return "{enabled=" + enabled + ", checksums=" + checksumPolicy + ", updates=" + updatePolicy + '}';
     }
 
     public void merge(ArtifactRepositoryPolicy policy) {
diff --git a/compat/maven-compat/src/main/java/org/apache/maven/artifact/repository/layout/FlatRepositoryLayout.java b/compat/maven-compat/src/main/java/org/apache/maven/artifact/repository/layout/FlatRepositoryLayout.java
@@ -62,19 +62,11 @@ public String pathOf(Artifact artifact) {
     }
 
     public String pathOfLocalRepositoryMetadata(ArtifactMetadata metadata, ArtifactRepository repository) {
-        return pathOfRepositoryMetadata(metadata.getLocalFilename(repository));
-    }
-
-    private String pathOfRepositoryMetadata(String filename) {
-        StringBuilder path = new StringBuilder(128);
-
-        path.append(filename);
-
-        return path.toString();
+        return metadata.getLocalFilename(repository);
     }
 
     public String pathOfRemoteRepositoryMetadata(ArtifactMetadata metadata) {
-        return pathOfRepositoryMetadata(metadata.getRemoteFilename());
+        return metadata.getRemoteFilename();
     }
 
     @Override
diff --git a/compat/maven-compat/src/main/java/org/apache/maven/artifact/repository/metadata/AbstractRepositoryMetadata.java b/compat/maven-compat/src/main/java/org/apache/maven/artifact/repository/metadata/AbstractRepositoryMetadata.java
@@ -159,14 +159,11 @@ public void merge(ArtifactMetadata metadata) {
     }
 
     public String extendedToString() {
-        StringBuilder buffer = new StringBuilder(256);
 
-        buffer.append(LS).append("Repository Metadata").append(LS).append("--------------------------");
-        buffer.append(LS).append("GroupId: ").append(getGroupId());
-        buffer.append(LS).append("ArtifactId: ").append(getArtifactId());
-        buffer.append(LS).append("Metadata Type: ").append(getClass().getName());
-
-        return buffer.toString();
+        return LS + "Repository Metadata" + LS + "--------------------------" + LS
+                + "GroupId: " + getGroupId() + LS
+                + "ArtifactId: " + getArtifactId() + LS
+                + "Metadata Type: " + getClass().getName();
     }
 
     public int getNature() {
diff --git a/compat/maven-compat/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolutionRequest.java b/compat/maven-compat/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolutionRequest.java
@@ -207,21 +207,18 @@ public boolean isResolveTransitively() {
     }
 
     public String toString() {
-        StringBuilder sb = new StringBuilder()
-                .append("REQUEST: ")
-                .append(LS)
-                .append("artifact: ")
-                .append(artifact)
-                .append(LS)
-                .append(artifactDependencies)
-                .append(LS)
-                .append("localRepository: ")
-                .append(localRepository)
-                .append(LS)
-                .append("remoteRepositories: ")
-                .append(remoteRepositories);
-
-        return sb.toString();
+
+        return "REQUEST: " + LS
+                + "artifact: "
+                + artifact
+                + LS
+                + artifactDependencies
+                + LS
+                + "localRepository: "
+                + localRepository
+                + LS
+                + "remoteRepositories: "
+                + remoteRepositories;
     }
 
     public boolean isOffline() {
diff --git a/compat/maven-compat/src/main/java/org/apache/maven/repository/MetadataResolutionRequest.java b/compat/maven-compat/src/main/java/org/apache/maven/repository/MetadataResolutionRequest.java
@@ -134,22 +134,19 @@ public MetadataResolutionRequest setManagedVersionMap(Map managedVersionMap) {
     }
 
     public String toString() {
-        StringBuilder sb = new StringBuilder()
-                .append("REQUEST: ")
-                .append("\n")
-                .append("artifact: ")
-                .append(mad)
-                .append("\n")
-                .append(artifactDependencies)
-                .append("\n")
-                .append("localRepository: ")
-                .append(localRepository)
-                .append("\n")
-                .append("remoteRepositories: ")
-                .append(remoteRepositories)
-                .append("\n");
-
-        return sb.toString();
+
+        return "REQUEST: " + "\n"
+                + "artifact: "
+                + mad
+                + "\n"
+                + artifactDependencies
+                + "\n"
+                + "localRepository: "
+                + localRepository
+                + "\n"
+                + "remoteRepositories: "
+                + remoteRepositories
+                + "\n";
     }
 
     public boolean isAsList() {
diff --git a/compat/maven-compat/src/main/java/org/apache/maven/repository/legacy/metadata/AbstractArtifactMetadata.java b/compat/maven-compat/src/main/java/org/apache/maven/repository/legacy/metadata/AbstractArtifactMetadata.java
@@ -47,13 +47,10 @@ public String getArtifactId() {
     }
 
     public String extendedToString() {
-        StringBuilder buffer = new StringBuilder(256);
 
-        buffer.append(LS).append("Artifact Metadata").append(LS).append("--------------------------");
-        buffer.append(LS).append("GroupId: ").append(getGroupId());
-        buffer.append(LS).append("ArtifactId: ").append(getArtifactId());
-        buffer.append(LS).append("Metadata Type: ").append(getClass().getName());
-
-        return buffer.toString();
+        return LS + "Artifact Metadata" + LS + "--------------------------" + LS
+                + "GroupId: " + getGroupId() + LS
+                + "ArtifactId: " + getArtifactId() + LS
+                + "Metadata Type: " + getClass().getName();
     }
 }
diff --git a/compat/maven-compat/src/test/java/org/apache/maven/artifact/AbstractArtifactComponentTestCase.java b/compat/maven-compat/src/test/java/org/apache/maven/artifact/AbstractArtifactComponentTestCase.java
@@ -282,9 +282,8 @@ protected Artifact createArtifact(String artifactId, String version, String type
     }
 
     protected Artifact createArtifact(String groupId, String artifactId, String version, String type) throws Exception {
-        Artifact a = artifactFactory.createBuildArtifact(groupId, artifactId, version, type);
 
-        return a;
+        return artifactFactory.createBuildArtifact(groupId, artifactId, version, type);
     }
 
     protected void deleteLocalArtifact(Artifact artifact) throws Exception {
diff --git a/compat/maven-compat/src/test/java/org/apache/maven/project/AbstractMavenProjectTestCase.java b/compat/maven-compat/src/test/java/org/apache/maven/project/AbstractMavenProjectTestCase.java
@@ -112,10 +112,8 @@ protected static File getFileForClasspathResource(String resource)
     protected ArtifactRepository getLocalRepository() throws Exception {
         ArtifactRepositoryLayout repoLayout = getContainer().lookup(ArtifactRepositoryLayout.class);
 
-        ArtifactRepository r = repositorySystem.createArtifactRepository(
+        return repositorySystem.createArtifactRepository(
                 "local", "file://" + getLocalRepositoryPath().getAbsolutePath(), repoLayout, null, null);
-
-        return r;
     }
 
     // ----------------------------------------------------------------------
diff --git a/compat/maven-embedder/src/main/java/org/apache/maven/cli/transfer/AbstractMavenTransferListener.java b/compat/maven-embedder/src/main/java/org/apache/maven/cli/transfer/AbstractMavenTransferListener.java
@@ -199,11 +199,10 @@ public void transferInitiated(TransferEvent event) {
         String direction = event.getRequestType() == TransferEvent.RequestType.PUT ? "to" : "from";
 
         TransferResource resource = event.getResource();
-        StringBuilder message = new StringBuilder();
-        message.append(darkOn).append(action).append(' ').append(direction).append(' ');
-        message.append(darkOff).append(resource.getRepositoryId());
-        message.append(darkOn).append(": ").append(resource.getRepositoryUrl());
-        message.append(darkOff).append(resource.getResourceName());
+        String message = darkOn + action + ' ' + direction + ' ' + darkOff
+                + resource.getRepositoryId() + darkOn
+                + ": " + resource.getRepositoryUrl() + darkOff
+                + resource.getResourceName();
 
         out.println(message);
     }
diff --git a/compat/maven-embedder/src/main/java/org/apache/maven/cli/transfer/Slf4jMavenTransferListener.java b/compat/maven-embedder/src/main/java/org/apache/maven/cli/transfer/Slf4jMavenTransferListener.java
@@ -48,12 +48,11 @@ public void transferInitiated(TransferEvent event) {
         String direction = event.getRequestType() == TransferEvent.RequestType.PUT ? "to" : "from";
 
         TransferResource resource = event.getResource();
-        StringBuilder message = new StringBuilder();
-        message.append(action).append(' ').append(direction).append(' ').append(resource.getRepositoryId());
-        message.append(": ");
-        message.append(resource.getRepositoryUrl()).append(resource.getResourceName());
+        String message = action + ' ' + direction + ' ' + resource.getRepositoryId() + ": "
+                + resource.getRepositoryUrl()
+                + resource.getResourceName();
 
-        out.info(message.toString());
+        out.info(message);
     }
 
     @Override
diff --git a/compat/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java b/compat/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelBuilder.java
@@ -984,9 +984,7 @@ private ModelData readParentLocally(
          * if ( version == null || !version.equals( parent.getVersion() ) ) { return null; }
          */
 
-        ModelData parentData = new ModelData(candidateSource, candidateModel, groupId, artifactId, version);
-
-        return parentData;
+        return new ModelData(candidateSource, candidateModel, groupId, artifactId, version);
     }
 
     private boolean rawChildVersionReferencesParent(String rawChildModelVersion) {
@@ -1094,10 +1092,8 @@ public int getValidationLevel() {
             // MNG-2199: What else to check here ?
         }
 
-        ModelData parentData = new ModelData(
+        return new ModelData(
                 modelSource, parentModel, parent.getGroupId(), parent.getArtifactId(), parent.getVersion());
-
-        return parentData;
     }
 
     private Model getSuperModel() {
diff --git a/compat/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelProblem.java b/compat/maven-model-builder/src/main/java/org/apache/maven/model/building/DefaultModelProblem.java
@@ -165,12 +165,7 @@ public Version getVersion() {
 
     @Override
     public String toString() {
-        StringBuilder buffer = new StringBuilder(128);
 
-        buffer.append('[').append(getSeverity()).append("] ");
-        buffer.append(getMessage());
-        buffer.append(" @ ").append(ModelProblemUtils.formatLocation(this, null));
-
-        return buffer.toString();
+        return "[" + getSeverity() + "] " + getMessage() + " @ " + ModelProblemUtils.formatLocation(this, null);
     }
 }
diff --git a/compat/maven-model-builder/src/main/java/org/apache/maven/model/building/ModelData.java b/compat/maven-model-builder/src/main/java/org/apache/maven/model/building/ModelData.java
@@ -189,15 +189,8 @@ public void setVersion(String version) {
      * @return The effective identifier of the model, never {@code null}.
      */
     public String getId() {
-        StringBuilder buffer = new StringBuilder(128);
 
-        buffer.append(getGroupId())
-                .append(':')
-                .append(getArtifactId())
-                .append(':')
-                .append(getVersion());
-
-        return buffer.toString();
+        return getGroupId() + ':' + getArtifactId() + ':' + getVersion();
     }
 
     @Override
diff --git a/compat/maven-model-builder/src/main/java/org/apache/maven/model/building/ModelProblemUtils.java b/compat/maven-model-builder/src/main/java/org/apache/maven/model/building/ModelProblemUtils.java
@@ -98,15 +98,12 @@ static String toId(Model model) {
      * @return The user-friendly artifact id, never {@code null}.
      */
     static String toId(String groupId, String artifactId, String version) {
-        StringBuilder buffer = new StringBuilder(128);
-
-        buffer.append((groupId != null && groupId.length() > 0) ? groupId : "[unknown-group-id]");
-        buffer.append(':');
-        buffer.append((artifactId != null && artifactId.length() > 0) ? artifactId : "[unknown-artifact-id]");
-        buffer.append(':');
-        buffer.append((version != null && version.length() > 0) ? version : "[unknown-version]");
 
-        return buffer.toString();
+        return ((groupId != null && groupId.length() > 0) ? groupId : "[unknown-group-id]")
+                + ':'
+                + ((artifactId != null && artifactId.length() > 0) ? artifactId : "[unknown-artifact-id]")
+                + ':'
+                + ((version != null && version.length() > 0) ? version : "[unknown-version]");
     }
 
     /**
diff --git a/compat/maven-model-builder/src/test/java/org/apache/maven/model/profile/activation/JdkVersionProfileActivatorTest.java b/compat/maven-model-builder/src/test/java/org/apache/maven/model/profile/activation/JdkVersionProfileActivatorTest.java
@@ -47,9 +47,7 @@ void setUp() throws Exception {
     private Profile newProfile(String jdkVersion) {
         Activation a = Activation.newBuilder().jdk(jdkVersion).build();
 
-        Profile p = Profile.newBuilder().activation(a).build();
-
-        return p;
+        return Profile.newBuilder().activation(a).build();
     }
 
     private Properties newProperties(String javaVersion) {
diff --git a/compat/maven-model-builder/src/test/java/org/apache/maven/model/profile/activation/OperatingSystemProfileActivatorTest.java b/compat/maven-model-builder/src/test/java/org/apache/maven/model/profile/activation/OperatingSystemProfileActivatorTest.java
@@ -42,9 +42,7 @@ void setUp() throws Exception {
     private Profile newProfile(ActivationOS.Builder activationBuilder) {
         Activation a = Activation.newBuilder().os(activationBuilder.build()).build();
 
-        Profile p = Profile.newBuilder().activation(a).build();
-
-        return p;
+        return Profile.newBuilder().activation(a).build();
     }
 
     private Properties newProperties(String osName, String osVersion, String osArch) {
diff --git a/compat/maven-model-builder/src/test/java/org/apache/maven/model/profile/activation/PropertyProfileActivatorTest.java b/compat/maven-model-builder/src/test/java/org/apache/maven/model/profile/activation/PropertyProfileActivatorTest.java
@@ -45,9 +45,7 @@ private Profile newProfile(String key, String value) {
 
         Activation a = Activation.newBuilder().property(ap).build();
 
-        Profile p = Profile.newBuilder().activation(a).build();
-
-        return p;
+        return Profile.newBuilder().activation(a).build();
     }
 
     private Properties newProperties(String key, String value) {
diff --git a/impl/maven-cli/src/main/java/org/apache/maven/cling/transfer/Slf4jMavenTransferListener.java b/impl/maven-cli/src/main/java/org/apache/maven/cling/transfer/Slf4jMavenTransferListener.java
@@ -51,12 +51,11 @@ public void transferInitiated(TransferEvent event) {
         String direction = event.getRequestType() == TransferEvent.RequestType.PUT ? "to" : "from";
 
         TransferResource resource = event.getResource();
-        StringBuilder message = new StringBuilder();
-        message.append(action).append(' ').append(direction).append(' ').append(resource.getRepositoryId());
-        message.append(": ");
-        message.append(resource.getRepositoryUrl()).append(resource.getResourceName());
+        String message = action + ' ' + direction + ' ' + resource.getRepositoryId() + ": "
+                + resource.getRepositoryUrl()
+                + resource.getResourceName();
 
-        out.info(message.toString());
+        out.info(message);
     }
 
     @Override
diff --git a/impl/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultLifecycleRegistry.java b/impl/maven-core/src/main/java/org/apache/maven/internal/impl/DefaultLifecycleRegistry.java
@@ -142,9 +142,7 @@ public List<String> computePhases(Lifecycle lifecycle) {
         addPhases(graph, null, null, lifecycle.v3phases());
         List<String> allPhases = graph.visitAll();
         Collections.reverse(allPhases);
-        List<String> computed =
-                allPhases.stream().filter(s -> !s.startsWith("$")).collect(Collectors.toList());
-        return computed;
+        return allPhases.stream().filter(s -> !s.startsWith("$")).collect(Collectors.toList());
     }
 
     private static void addPhase(
diff --git a/impl/maven-core/src/main/java/org/apache/maven/internal/transformation/impl/DefaultConsumerPomBuilder.java b/impl/maven-core/src/main/java/org/apache/maven/internal/transformation/impl/DefaultConsumerPomBuilder.java
@@ -198,8 +198,7 @@ private ModelBuilderResult buildModel(RepositorySystemSession session, Path src)
         request.lifecycleBindingsInjector(lifecycleBindingsInjector::injectLifecycleBindings);
         ModelBuilder.ModelBuilderSession mbSession =
                 iSession.getData().get(SessionData.key(ModelBuilder.ModelBuilderSession.class));
-        ModelBuilderResult result = mbSession.build(request.build());
-        return result;
+        return mbSession.build(request.build());
     }
 
     static Model transformNonPom(Model model, MavenProject project) {
diff --git a/impl/maven-core/src/main/java/org/apache/maven/plugin/MojoExecution.java b/impl/maven-core/src/main/java/org/apache/maven/plugin/MojoExecution.java
@@ -146,12 +146,8 @@ public void setConfiguration(XmlNode configuration) {
     }
 
     public String identify() {
-        StringBuilder sb = new StringBuilder(256);
 
-        sb.append(executionId);
-        sb.append(configuration.toString());
-
-        return sb.toString();
+        return executionId + configuration.toString();
     }
 
     public String getLifecyclePhase() {
diff --git a/impl/maven-core/src/main/java/org/apache/maven/project/MavenProject.java b/impl/maven-core/src/main/java/org/apache/maven/project/MavenProject.java
@@ -1403,9 +1403,7 @@ private void deepCopy(MavenProject project) {
     }
 
     private static String getProjectReferenceId(String groupId, String artifactId, String version) {
-        StringBuilder buffer = new StringBuilder(128);
-        buffer.append(groupId).append(':').append(artifactId).append(':').append(version);
-        return buffer.toString();
+        return groupId + ':' + artifactId + ':' + version;
     }
 
     /**
diff --git a/impl/maven-core/src/main/java/org/apache/maven/project/ReactorModelPool.java b/impl/maven-core/src/main/java/org/apache/maven/project/ReactorModelPool.java
@@ -82,9 +82,7 @@ public int hashCode() {
 
         @Override
         public String toString() {
-            StringBuilder buffer = new StringBuilder(128);
-            buffer.append(groupId).append(':').append(artifactId).append(':').append(version);
-            return buffer.toString();
+            return groupId + ':' + artifactId + ':' + version;
         }
     }
 }
diff --git a/impl/maven-core/src/test/java/org/apache/maven/plugin/PluginParameterExpressionEvaluatorV4Test.java b/impl/maven-core/src/test/java/org/apache/maven/plugin/PluginParameterExpressionEvaluatorV4Test.java
@@ -421,9 +421,8 @@ private MojoExecution newMojoExecution(Session session) {
     }
 
     private DefaultSession newSession() throws Exception {
-        DefaultSession session = new DefaultSession(
+        return new DefaultSession(
                 newMavenSession(), mock(RepositorySystem.class), null, null, new DefaultLookup(container), null);
-        return session;
     }
 
     private MavenSession newMavenSession() throws Exception {
diff --git a/impl/maven-core/src/test/java/org/apache/maven/repository/TestRepositoryConnector.java b/impl/maven-core/src/test/java/org/apache/maven/repository/TestRepositoryConnector.java
diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/SettingsUtilsV4.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/SettingsUtilsV4.java
diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultInterpolator.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/model/DefaultInterpolator.java
diff --git a/impl/maven-impl/src/main/java/org/apache/maven/impl/resolver/LocalSnapshotMetadata.java b/impl/maven-impl/src/main/java/org/apache/maven/impl/resolver/LocalSnapshotMetadata.java
diff --git a/impl/maven-impl/src/test/java/org/apache/maven/impl/model/profile/ConditionProfileActivatorTest.java b/impl/maven-impl/src/test/java/org/apache/maven/impl/model/profile/ConditionProfileActivatorTest.java
diff --git a/impl/maven-impl/src/test/java/org/apache/maven/impl/model/profile/JdkVersionProfileActivatorTest.java b/impl/maven-impl/src/test/java/org/apache/maven/impl/model/profile/JdkVersionProfileActivatorTest.java
diff --git a/impl/maven-impl/src/test/java/org/apache/maven/impl/model/profile/OperatingSystemProfileActivatorTest.java b/impl/maven-impl/src/test/java/org/apache/maven/impl/model/profile/OperatingSystemProfileActivatorTest.java
diff --git a/impl/maven-impl/src/test/java/org/apache/maven/impl/model/profile/PropertyProfileActivatorTest.java b/impl/maven-impl/src/test/java/org/apache/maven/impl/model/profile/PropertyProfileActivatorTest.java
diff --git a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng2068ReactorRelativeParentsTest.java b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng2068ReactorRelativeParentsTest.java
diff --git a/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4745PluginVersionUpdateTest.java b/its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng4745PluginVersionUpdateTest.java
diff --git a/its/core-it-suite/src/test/java/org/apache/maven/it/utils/DeployedResource.java b/its/core-it-suite/src/test/java/org/apache/maven/it/utils/DeployedResource.java
diff --git a/its/core-it-support/core-it-plugins/maven-it-plugin-artifact/src/main/java/org/apache/maven/plugin/coreit/InstallArtifactsMojo.java b/its/core-it-support/core-it-plugins/maven-it-plugin-artifact/src/main/java/org/apache/maven/plugin/coreit/InstallArtifactsMojo.java

Original file line number	Diff line number	Diff line change
`@@ -62,19 +62,11 @@ public String pathOf(Artifact artifact) {`
`62`	`62`	`}`
`63`	`63`
`64`	`64`	`public String pathOfLocalRepositoryMetadata(ArtifactMetadata metadata, ArtifactRepository repository) {`
`65`		`- return pathOfRepositoryMetadata(metadata.getLocalFilename(repository));`
`66`		`- }`
`67`		`-`
`68`		`- private String pathOfRepositoryMetadata(String filename) {`
`69`		`- StringBuilder path = new StringBuilder(128);`
`70`		`-`
`71`		`- path.append(filename);`
`72`		`-`
`73`		`- return path.toString();`
	`65`	`+ return metadata.getLocalFilename(repository);`
`74`	`66`	`}`
`75`	`67`
`76`	`68`	`public String pathOfRemoteRepositoryMetadata(ArtifactMetadata metadata) {`
`77`		`- return pathOfRepositoryMetadata(metadata.getRemoteFilename());`
	`69`	`+ return metadata.getRemoteFilename();`
`78`	`70`	`}`
`79`	`71`
`80`	`72`	`@Override`
Original file line number	Diff line number	Diff line change
`@@ -47,13 +47,10 @@ public String getArtifactId() {`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`public String extendedToString() {`
`50`		`- StringBuilder buffer = new StringBuilder(256);`
`51`	`50`
`52`		`- buffer.append(LS).append("Artifact Metadata").append(LS).append("--------------------------");`
`53`		`- buffer.append(LS).append("GroupId: ").append(getGroupId());`
`54`		`- buffer.append(LS).append("ArtifactId: ").append(getArtifactId());`
`55`		`- buffer.append(LS).append("Metadata Type: ").append(getClass().getName());`
`56`		`-`
`57`		`- return buffer.toString();`
	`51`	`+ return LS + "Artifact Metadata" + LS + "--------------------------" + LS`
	`52`	`+ + "GroupId: " + getGroupId() + LS`
	`53`	`+ + "ArtifactId: " + getArtifactId() + LS`
	`54`	`+ + "Metadata Type: " + getClass().getName();`
`58`	`55`	`}`
`59`	`56`	`}`
Original file line number	Diff line number	Diff line change
`@@ -282,9 +282,8 @@ protected Artifact createArtifact(String artifactId, String version, String type`
`282`	`282`	`}`
`283`	`283`
`284`	`284`	`protected Artifact createArtifact(String groupId, String artifactId, String version, String type) throws Exception {`
`285`		`- Artifact a = artifactFactory.createBuildArtifact(groupId, artifactId, version, type);`
`286`	`285`
`287`		`- return a;`
	`286`	`+ return artifactFactory.createBuildArtifact(groupId, artifactId, version, type);`
`288`	`287`	`}`
`289`	`288`
`290`	`289`	`protected void deleteLocalArtifact(Artifact artifact) throws Exception {`