You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by bd...@apache.org on 2021/05/07 15:42:48 UTC
[sling-whiteboard] branch master updated: Consistent naming
This is an automated email from the ASF dual-hosted git repository.
bdelacretaz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-whiteboard.git
The following commit(s) were added to refs/heads/master by this push:
new 77087e1 Consistent naming
77087e1 is described below
commit 77087e193e54973f51900ebb189e49f484cea042
Author: Bertrand Delacretaz <bd...@apache.org>
AuthorDate: Fri May 7 17:42:29 2021 +0200
Consistent naming
---
.../sling/documentaggregator/api/DocumentTree.java | 2 +-
.../{JsonTargetNode.java => JsonDocumentNode.java} | 10 ++++-----
...sonMappingTarget.java => JsonDocumentTree.java} | 8 ++++----
.../{MapTargetNode.java => MapDocumentNode.java} | 6 +++---
...{MapMappingTarget.java => MapDocumentTree.java} | 8 ++++----
.../impl/PropertiesSelector.java | 24 ----------------------
.../samples/graphql/DocumentDataFetcher.java | 2 +-
.../samples/graphql/DocumentsDataFetcher.java | 2 +-
8 files changed, 19 insertions(+), 43 deletions(-)
diff --git a/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/api/DocumentTree.java b/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/api/DocumentTree.java
index d8558e0..d4e6b2a 100644
--- a/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/api/DocumentTree.java
+++ b/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/api/DocumentTree.java
@@ -38,5 +38,5 @@ public interface DocumentTree {
void close();
}
- @NotNull DocumentNode newTargetNode();
+ @NotNull DocumentNode newDocumentNode();
}
\ No newline at end of file
diff --git a/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/JsonTargetNode.java b/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/JsonDocumentNode.java
similarity index 89%
rename from remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/JsonTargetNode.java
rename to remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/JsonDocumentNode.java
index 9cb7dce..76c41b7 100644
--- a/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/JsonTargetNode.java
+++ b/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/JsonDocumentNode.java
@@ -29,14 +29,14 @@ import javax.json.JsonObjectBuilder;
import org.apache.sling.documentaggregator.api.DocumentTree;
import org.apache.sling.documentaggregator.api.DocumentTree.DocumentNode;
-/** A TargetNode that outputs to a JSON document */
-public class JsonTargetNode implements DocumentTree.DocumentNode {
+/** A DocumentNode that outputs to a JSON document */
+public class JsonDocumentNode implements DocumentTree.DocumentNode {
private final String name;
private final JsonObjectBuilder builder;
- private List<JsonTargetNode> children;
+ private List<JsonDocumentNode> children;
- JsonTargetNode(String name) {
+ JsonDocumentNode(String name) {
this.name = name;
this.builder = Json.createObjectBuilder();
}
@@ -46,7 +46,7 @@ public class JsonTargetNode implements DocumentTree.DocumentNode {
if(children == null) {
children = new ArrayList<>();
}
- final JsonTargetNode child = new JsonTargetNode(name);
+ final JsonDocumentNode child = new JsonDocumentNode(name);
children.add(child);
return child;
}
diff --git a/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/JsonMappingTarget.java b/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/JsonDocumentTree.java
similarity index 82%
rename from remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/JsonMappingTarget.java
rename to remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/JsonDocumentTree.java
index 0f96063..7dad999 100644
--- a/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/JsonMappingTarget.java
+++ b/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/JsonDocumentTree.java
@@ -24,12 +24,12 @@ import org.apache.sling.documentaggregator.api.DocumentTree;
import org.jetbrains.annotations.NotNull;
import org.osgi.service.component.annotations.Component;
-/** MappingTarget that outputs to a JSON document */
+/** DocumentTree that generates a JSON document */
@Component(service = DocumentTree.class, property = { DocumentTree.TARGET_TYPE + "=json" })
-public class JsonMappingTarget implements DocumentTree {
+public class JsonDocumentTree implements DocumentTree {
@Override
- public @NotNull DocumentNode newTargetNode() {
- return new JsonTargetNode("ROOT_THIS_NAME_SHOULD_NOT_APPEAR_IN_OUTPUT");
+ public @NotNull DocumentNode newDocumentNode() {
+ return new JsonDocumentNode("ROOT_THIS_NAME_SHOULD_NOT_APPEAR_IN_OUTPUT");
}
}
\ No newline at end of file
diff --git a/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/MapTargetNode.java b/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/MapDocumentNode.java
similarity index 90%
rename from remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/MapTargetNode.java
rename to remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/MapDocumentNode.java
index 722d2ad..f066640 100644
--- a/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/MapTargetNode.java
+++ b/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/MapDocumentNode.java
@@ -26,14 +26,14 @@ import org.apache.sling.documentaggregator.api.DocumentTree;
import org.apache.sling.documentaggregator.api.DocumentTree.DocumentNode;
/** A TargetNode that outputs to a Map document */
-public class MapTargetNode extends HashMap<String, Object> implements DocumentTree.DocumentNode {
+public class MapDocumentNode extends HashMap<String, Object> implements DocumentTree.DocumentNode {
- MapTargetNode(String name) {
+ MapDocumentNode(String name) {
}
@Override
public DocumentNode addChild(String name) {
- final MapTargetNode child = new MapTargetNode(name);
+ final MapDocumentNode child = new MapDocumentNode(name);
put(name, child);
return child;
}
diff --git a/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/MapMappingTarget.java b/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/MapDocumentTree.java
similarity index 82%
rename from remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/MapMappingTarget.java
rename to remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/MapDocumentTree.java
index 5aaf833..5e82718 100644
--- a/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/MapMappingTarget.java
+++ b/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/MapDocumentTree.java
@@ -24,12 +24,12 @@ import org.apache.sling.documentaggregator.api.DocumentTree;
import org.jetbrains.annotations.NotNull;
import org.osgi.service.component.annotations.Component;
-/** MappingTarget that outputs to a JSON document */
+/** DocumentTree that generates a Map of Maps */
@Component(service = DocumentTree.class, property = { DocumentTree.TARGET_TYPE + "=map" })
-public class MapMappingTarget implements DocumentTree {
+public class MapDocumentTree implements DocumentTree {
@Override
- public @NotNull DocumentNode newTargetNode() {
- return new MapTargetNode("ROOT_THIS_NAME_SHOULD_NOT_APPEAR_IN_OUTPUT");
+ public @NotNull DocumentNode newDocumentNode() {
+ return new MapDocumentNode("ROOT_THIS_NAME_SHOULD_NOT_APPEAR_IN_OUTPUT");
}
}
\ No newline at end of file
diff --git a/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/PropertiesSelector.java b/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/PropertiesSelector.java
deleted file mode 100644
index 187a62c..0000000
--- a/remote-content-api/document-aggregator/src/main/java/org/apache/sling/documentaggregator/impl/PropertiesSelector.java
+++ /dev/null
@@ -1,24 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements. See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership. The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied. See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-
-package org.apache.sling.documentaggregator.impl;
-
-interface PropertiesSelector {
- boolean renderProperty(String name);
-}
\ No newline at end of file
diff --git a/remote-content-api/sample-graphql-api/src/main/java/org/apache/sling/remotecontent/samples/graphql/DocumentDataFetcher.java b/remote-content-api/sample-graphql-api/src/main/java/org/apache/sling/remotecontent/samples/graphql/DocumentDataFetcher.java
index 48e6d1c..af048e3 100644
--- a/remote-content-api/sample-graphql-api/src/main/java/org/apache/sling/remotecontent/samples/graphql/DocumentDataFetcher.java
+++ b/remote-content-api/sample-graphql-api/src/main/java/org/apache/sling/remotecontent/samples/graphql/DocumentDataFetcher.java
@@ -54,7 +54,7 @@ public class DocumentDataFetcher implements SlingDataFetcher<Object> {
final Resource target = e.getCurrentResource().getResourceResolver().getResource(path);
// Use the aggregator to build the body
- final DocumentTree.DocumentNode body = mappingTarget.newTargetNode();
+ final DocumentTree.DocumentNode body = mappingTarget.newDocumentNode();
documentAggregator.aggregate(target, body, opt);
body.close();
data.put("body", body.adaptTo(Map.class));
diff --git a/remote-content-api/sample-graphql-api/src/main/java/org/apache/sling/remotecontent/samples/graphql/DocumentsDataFetcher.java b/remote-content-api/sample-graphql-api/src/main/java/org/apache/sling/remotecontent/samples/graphql/DocumentsDataFetcher.java
index 22a10cd..ea57780 100644
--- a/remote-content-api/sample-graphql-api/src/main/java/org/apache/sling/remotecontent/samples/graphql/DocumentsDataFetcher.java
+++ b/remote-content-api/sample-graphql-api/src/main/java/org/apache/sling/remotecontent/samples/graphql/DocumentsDataFetcher.java
@@ -46,7 +46,7 @@ public class DocumentsDataFetcher implements SlingDataFetcher<Object> {
private DocumentAggregator documentAggregator;
private void addDocumentData(final Map<String, Object> data, String key, Resource r, DocumentAggregator aggregator, DocumentAggregator.Options opt) {
- final DocumentTree.DocumentNode target = mappingTarget.newTargetNode();
+ final DocumentTree.DocumentNode target = mappingTarget.newDocumentNode();
aggregator.aggregate(r, target, opt);
target.close();
data.put(key, target.adaptTo(Map.class));