This is an automated email from the ASF dual-hosted git repository.

zehnder pushed a commit to branch dev
in repository https://gitbox.apache.org/repos/asf/streampipes.git


The following commit(s) were added to refs/heads/dev by this push:
     new c2fb93864b refactor(#3139): Remove deprecated 
IParameterExtractor.selectedTreeNodesInternalNames (#3459)
c2fb93864b is described below

commit c2fb93864b0433fc89e1570a1b9d1808ddd7c394
Author: Philipp Zehnder <[email protected]>
AuthorDate: Fri Feb 7 16:40:53 2025 +0100

    refactor(#3139): Remove deprecated 
IParameterExtractor.selectedTreeNodesInternalNames (#3459)
---
 .../api/extractor/IParameterExtractor.java         |  8 ----
 .../sdk/extractor/AbstractParameterExtractor.java  | 55 ----------------------
 2 files changed, 63 deletions(-)

diff --git 
a/streampipes-extensions-api/src/main/java/org/apache/streampipes/extensions/api/extractor/IParameterExtractor.java
 
b/streampipes-extensions-api/src/main/java/org/apache/streampipes/extensions/api/extractor/IParameterExtractor.java
index 2fd8e40f76..e315dba534 100644
--- 
a/streampipes-extensions-api/src/main/java/org/apache/streampipes/extensions/api/extractor/IParameterExtractor.java
+++ 
b/streampipes-extensions-api/src/main/java/org/apache/streampipes/extensions/api/extractor/IParameterExtractor.java
@@ -65,14 +65,6 @@ public interface IParameterExtractor {
   <V> List<V> selectedTreeNodesInternalNames(String internalName,
                                              Class<V> targetClass);
 
-  /**
-   * @deprecated use {@link #selectedTreeNodesInternalNames(String, Class)} 
instead
-   */
-  @Deprecated(since = "0.97.0", forRemoval = true)
-  <V> List<V> selectedTreeNodesInternalNames(String internalName,
-                                             Class<V> targetClass,
-                                             boolean onlyDataNodes);
-
   <W extends StaticProperty> W getStaticPropertyByName(String internalName, 
Class<W>
       spType);
 
diff --git 
a/streampipes-sdk/src/main/java/org/apache/streampipes/sdk/extractor/AbstractParameterExtractor.java
 
b/streampipes-sdk/src/main/java/org/apache/streampipes/sdk/extractor/AbstractParameterExtractor.java
index 844f26af2c..599a13c39d 100644
--- 
a/streampipes-sdk/src/main/java/org/apache/streampipes/sdk/extractor/AbstractParameterExtractor.java
+++ 
b/streampipes-sdk/src/main/java/org/apache/streampipes/sdk/extractor/AbstractParameterExtractor.java
@@ -47,7 +47,6 @@ import 
org.apache.streampipes.model.staticproperty.StaticPropertyAlternative;
 import org.apache.streampipes.model.staticproperty.StaticPropertyAlternatives;
 import org.apache.streampipes.model.staticproperty.StaticPropertyGroup;
 import org.apache.streampipes.model.staticproperty.StaticPropertyType;
-import org.apache.streampipes.model.staticproperty.TreeInputNode;
 import org.apache.streampipes.sdk.utils.Datatypes;
 
 import com.github.drapostolos.typeparser.TypeParser;
@@ -279,60 +278,6 @@ public abstract class AbstractParameterExtractor<T extends 
InvocableStreamPipesE
                                                    .toList();
   }
 
-  @Override
-  @Deprecated(since = "0.97.0", forRemoval = true)
-  /**
-   * This method returns a list of all nodes.
-   * Therefore, it requires both the property selectedNodesInternalNames and 
the nodes property to be set.
-   * The nodes are used to check for the data type. The problem with this 
implementation is, is that the client (e.g.
-   * UI) must get the nodes from the OPC UA server.
-   *
-   * @deprecated use {@link #selectedTreeNodesInternalNames(String, Class)} 
instead
-   */
-  public <V> List<V> selectedTreeNodesInternalNames(
-      String internalName,
-      Class<V> targetClass,
-      boolean onlyDataNodes
-  ) {
-    List<TreeInputNode> allNodes = new ArrayList<>();
-    RuntimeResolvableTreeInputStaticProperty sp =
-        getStaticPropertyByName(internalName, 
RuntimeResolvableTreeInputStaticProperty.class);
-    if (!sp.getNodes()
-           .isEmpty()) {
-      sp.getNodes()
-          .forEach(node -> buildFlatTree(node, allNodes));
-    }
-
-    if (!allNodes.isEmpty()) {
-      return sp.getSelectedNodesInternalNames()
-               .stream()
-               .filter(node -> {
-                 if (!onlyDataNodes) {
-                   return true;
-                 } else {
-                   var existingNode = allNodes.stream()
-                                              .filter(n -> 
n.getInternalNodeName()
-                                                            .equals(node))
-                                              .findFirst();
-                   return existingNode.map(TreeInputNode::isDataNode)
-                                      .orElse(false);
-                 }
-               })
-               .map(node -> typeParser.parse(node, targetClass))
-               .collect(Collectors.toList());
-    } else {
-      return new ArrayList<>();
-    }
-  }
-
-  private void buildFlatTree(TreeInputNode parent, List<TreeInputNode> 
collector) {
-    collector.add(parent);
-    if (parent.hasChildren()) {
-      parent.getChildren()
-            .forEach(child -> buildFlatTree(child, collector));
-    }
-  }
-
   @Override
   public <W extends StaticProperty> W getStaticPropertyByName(
       String internalName, Class<W>

Reply via email to