You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by si...@apache.org on 2019/06/18 14:42:30 UTC
[sling-org-apache-sling-feature-inventoryprinter] branch r2f
updated: [r2f] reducing redundant code
This is an automated email from the ASF dual-hosted git repository.
simonetripodi pushed a commit to branch r2f
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-feature-inventoryprinter.git
The following commit(s) were added to refs/heads/r2f by this push:
new deee9ce [r2f] reducing redundant code
deee9ce is described below
commit deee9ce56a22c6d7f46596ac62c40a1e875a44d2
Author: Simo Tripodi <st...@adobe.com>
AuthorDate: Tue Jun 18 16:42:22 2019 +0200
[r2f] reducing redundant code
---
...r.java => AbstractFeatureInventoryPrinter.java} | 41 +++++++++++---------
...ractRuntimeEnvironment2FeatureModelPrinter.java | 45 +++-------------------
.../impl/FeaturesInventoryPrinter.java | 42 +++++++++-----------
3 files changed, 46 insertions(+), 82 deletions(-)
diff --git a/src/main/java/org/apache/sling/feature/inventoryservice/impl/FeaturesInventoryPrinter.java b/src/main/java/org/apache/sling/feature/inventoryservice/impl/AbstractFeatureInventoryPrinter.java
similarity index 59%
copy from src/main/java/org/apache/sling/feature/inventoryservice/impl/FeaturesInventoryPrinter.java
copy to src/main/java/org/apache/sling/feature/inventoryservice/impl/AbstractFeatureInventoryPrinter.java
index ac9710c..c11efa2 100644
--- a/src/main/java/org/apache/sling/feature/inventoryservice/impl/FeaturesInventoryPrinter.java
+++ b/src/main/java/org/apache/sling/feature/inventoryservice/impl/AbstractFeatureInventoryPrinter.java
@@ -18,36 +18,41 @@
*/
package org.apache.sling.feature.inventoryservice.impl;
-import org.apache.felix.inventory.Format;
-import org.apache.felix.inventory.InventoryPrinter;
-import org.osgi.framework.BundleContext;
-import org.osgi.service.component.annotations.Activate;
-import org.osgi.service.component.annotations.Component;
+import static java.nio.charset.StandardCharsets.UTF_8;
+import static java.nio.file.Files.newBufferedReader;
+import java.io.BufferedReader;
import java.io.PrintWriter;
import java.net.URI;
-import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
-@Component(
-property = {InventoryPrinter.NAME + "=launch_feature",
- InventoryPrinter.TITLE + "=Launch Feature",
- InventoryPrinter.FORMAT + "=JSON"})
-public class FeaturesInventoryPrinter implements InventoryPrinter
-{
+import org.apache.felix.inventory.Format;
+import org.apache.felix.inventory.InventoryPrinter;
+import org.osgi.framework.BundleContext;
+import org.osgi.service.component.annotations.Activate;
+
+abstract class AbstractFeatureInventoryPrinter implements InventoryPrinter {
+
+ private static final String SLING_FEATURE_PROPERTY_NAME = "sling.feature";
+
@Activate
- BundleContext bc;
+ BundleContext bundleContext;
@Override
- public void print(PrintWriter printWriter, Format format, boolean isZip) {
- try {
- Path path = Paths.get(new URI(bc.getProperty("sling.feature")));
- byte[] bytes = Files.readAllBytes(path);
- printWriter.print(new String(bytes));
+ public final void print(PrintWriter printWriter, Format format, boolean isZip) {
+ String featureLocation = bundleContext.getProperty(SLING_FEATURE_PROPERTY_NAME);
+ URI previousFeatureURI = URI.create(featureLocation);
+ Path previousFeaturePath = Paths.get(previousFeatureURI);
+
+ try (BufferedReader reader = newBufferedReader(previousFeaturePath, UTF_8)) {
+ onFeature(featureLocation, reader, printWriter);
} catch (Exception e) {
e.printStackTrace(printWriter);
}
printWriter.println();
}
+
+ protected abstract void onFeature(String featureLocation, BufferedReader reader, PrintWriter printWriter) throws Exception;
+
}
diff --git a/src/main/java/org/apache/sling/feature/inventoryservice/impl/AbstractRuntimeEnvironment2FeatureModelPrinter.java b/src/main/java/org/apache/sling/feature/inventoryservice/impl/AbstractRuntimeEnvironment2FeatureModelPrinter.java
index bb3e322..6d71c9d 100644
--- a/src/main/java/org/apache/sling/feature/inventoryservice/impl/AbstractRuntimeEnvironment2FeatureModelPrinter.java
+++ b/src/main/java/org/apache/sling/feature/inventoryservice/impl/AbstractRuntimeEnvironment2FeatureModelPrinter.java
@@ -16,54 +16,27 @@
*/
package org.apache.sling.feature.inventoryservice.impl;
-import static java.nio.charset.StandardCharsets.UTF_8;
-import static java.nio.file.Files.newBufferedReader;
import static org.apache.sling.feature.io.json.FeatureJSONReader.read;
import static org.apache.sling.feature.io.json.FeatureJSONWriter.write;
-import java.io.IOException;
+import java.io.BufferedReader;
import java.io.PrintWriter;
-import java.io.Reader;
-import java.net.URI;
-import java.nio.file.Path;
-import java.nio.file.Paths;
-import org.apache.felix.inventory.Format;
-import org.apache.felix.inventory.InventoryPrinter;
import org.apache.sling.feature.ArtifactId;
import org.apache.sling.feature.Feature;
import org.apache.sling.feature.r2f.ConversionRequest;
import org.apache.sling.feature.r2f.DefaultConversionRequest;
import org.apache.sling.feature.r2f.RuntimeEnvironment2FeatureModel;
-import org.osgi.framework.BundleContext;
-import org.osgi.service.component.annotations.Activate;
import org.osgi.service.component.annotations.Reference;
-abstract class AbstractRuntimeEnvironment2FeatureModelPrinter implements InventoryPrinter {
-
- private static final String SLING_FEATURE_PROPERTY_NAME = "sling.feature";
+abstract class AbstractRuntimeEnvironment2FeatureModelPrinter extends AbstractFeatureInventoryPrinter {
@Reference
RuntimeEnvironment2FeatureModel generator;
- @Activate
- BundleContext bundleContext;
-
@Override
- public final void print(PrintWriter printWriter, Format format, boolean isZip) {
- String previousFeatureLocation = bundleContext.getProperty(SLING_FEATURE_PROPERTY_NAME);
- URI previousFeatureURI = URI.create(previousFeatureLocation);
- Path previousFeaturePath = Paths.get(previousFeatureURI);
- Feature previousFeature = null;
-
- try (Reader reader = newBufferedReader(previousFeaturePath, UTF_8)) {
- previousFeature = read(reader, previousFeatureLocation);
- } catch (IOException e) {
- throw new RuntimeException("An error occurred while reading 'sling.feature' framework-property "
- + previousFeatureLocation
- + ", see causing error(s):",
- e);
- }
+ protected void onFeature(String featureLocation, BufferedReader reader, PrintWriter printWriter) throws Exception {
+ Feature previousFeature = read(reader, featureLocation);
String groupId = previousFeature.getId().getGroupId();
String artifactId = previousFeature.getId().getArtifactId();
@@ -77,15 +50,7 @@ abstract class AbstractRuntimeEnvironment2FeatureModelPrinter implements Invento
Feature computedFeature = compute(previousFeature, currentFeature);
- try {
- write(printWriter, computedFeature);
- } catch (IOException e) {
- printWriter.append("An error occured while searlizing ")
- .append(computedFeature.toString())
- .append(":\n");
-
- e.printStackTrace(printWriter);
- }
+ write(printWriter, computedFeature);
}
protected abstract Feature compute(Feature previousFeature, Feature currentFeature);
diff --git a/src/main/java/org/apache/sling/feature/inventoryservice/impl/FeaturesInventoryPrinter.java b/src/main/java/org/apache/sling/feature/inventoryservice/impl/FeaturesInventoryPrinter.java
index ac9710c..76bff52 100644
--- a/src/main/java/org/apache/sling/feature/inventoryservice/impl/FeaturesInventoryPrinter.java
+++ b/src/main/java/org/apache/sling/feature/inventoryservice/impl/FeaturesInventoryPrinter.java
@@ -18,36 +18,30 @@
*/
package org.apache.sling.feature.inventoryservice.impl;
-import org.apache.felix.inventory.Format;
-import org.apache.felix.inventory.InventoryPrinter;
-import org.osgi.framework.BundleContext;
-import org.osgi.service.component.annotations.Activate;
-import org.osgi.service.component.annotations.Component;
+import static org.apache.felix.inventory.InventoryPrinter.FORMAT;
+import static org.apache.felix.inventory.InventoryPrinter.NAME;
+import static org.apache.felix.inventory.InventoryPrinter.TITLE;
+import java.io.BufferedReader;
import java.io.PrintWriter;
-import java.net.URI;
-import java.nio.file.Files;
-import java.nio.file.Path;
-import java.nio.file.Paths;
+
+import org.osgi.service.component.annotations.Component;
@Component(
-property = {InventoryPrinter.NAME + "=launch_feature",
- InventoryPrinter.TITLE + "=Launch Feature",
- InventoryPrinter.FORMAT + "=JSON"})
-public class FeaturesInventoryPrinter implements InventoryPrinter
-{
- @Activate
- BundleContext bc;
+ property = {
+ NAME + "=launch_feature",
+ TITLE + "=Launch Feature",
+ FORMAT + "=JSON"
+ }
+)
+public class FeaturesInventoryPrinter extends AbstractFeatureInventoryPrinter {
@Override
- public void print(PrintWriter printWriter, Format format, boolean isZip) {
- try {
- Path path = Paths.get(new URI(bc.getProperty("sling.feature")));
- byte[] bytes = Files.readAllBytes(path);
- printWriter.print(new String(bytes));
- } catch (Exception e) {
- e.printStackTrace(printWriter);
+ protected void onFeature(String featureLocation, BufferedReader reader, PrintWriter printWriter) throws Exception {
+ String line = null;
+ while ((line = reader.readLine()) != null) {
+ printWriter.println(line);
}
- printWriter.println();
}
+
}