From facc0b055707eaab37eb7cc8763a8504726a2301 Mon Sep 17 00:00:00 2001 From: Stefan Seifert Date: Thu, 15 Aug 2024 12:32:47 +0200 Subject: [PATCH] cosmetic: eliminate code warnings --- .../conga/plugins/aem/maven/allpackage/AllPackageBuilder.java | 2 +- .../aem/maven/allpackage/TemporaryContentPackageFile.java | 4 +--- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/tooling/conga-aem-maven-plugin/src/main/java/io/wcm/devops/conga/plugins/aem/maven/allpackage/AllPackageBuilder.java b/tooling/conga-aem-maven-plugin/src/main/java/io/wcm/devops/conga/plugins/aem/maven/allpackage/AllPackageBuilder.java index b892598..1058151 100644 --- a/tooling/conga-aem-maven-plugin/src/main/java/io/wcm/devops/conga/plugins/aem/maven/allpackage/AllPackageBuilder.java +++ b/tooling/conga-aem-maven-plugin/src/main/java/io/wcm/devops/conga/plugins/aem/maven/allpackage/AllPackageBuilder.java @@ -579,7 +579,7 @@ private List processContentPackage(ContentPackageFi File tempFile = File.createTempFile(FilenameUtils.getBaseName(pkg.getFile().getName()), ".zip"); // open original content package - try (ZipFile zipFileIn = new ZipFile(pkg.getFile())) { + try (ZipFile zipFileIn = new ZipFile.Builder().setFile(pkg.getFile()).get()) { // iterate through entries and write them to the temp. zip file try (FileOutputStream fos = new FileOutputStream(tempFile); diff --git a/tooling/conga-aem-maven-plugin/src/main/java/io/wcm/devops/conga/plugins/aem/maven/allpackage/TemporaryContentPackageFile.java b/tooling/conga-aem-maven-plugin/src/main/java/io/wcm/devops/conga/plugins/aem/maven/allpackage/TemporaryContentPackageFile.java index 249d5b7..e106551 100644 --- a/tooling/conga-aem-maven-plugin/src/main/java/io/wcm/devops/conga/plugins/aem/maven/allpackage/TemporaryContentPackageFile.java +++ b/tooling/conga-aem-maven-plugin/src/main/java/io/wcm/devops/conga/plugins/aem/maven/allpackage/TemporaryContentPackageFile.java @@ -83,9 +83,7 @@ public String getPackageInfoWithDependencies() { StringBuilder sb = new StringBuilder(); sb.append(getPackageInfo()); if (this.dependencies != null) { - sb.append(" (dependencies: "); - sb.append(this.dependencies); - sb.append(")"); + sb.append(" (dependencies: ").append(this.dependencies).append(")"); } return sb.toString(); }