You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nifi.apache.org by ex...@apache.org on 2021/10/30 16:17:29 UTC

[nifi] branch main updated: NIFI-9352: Consider all property values in component references

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

exceptionfactory pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/nifi.git


The following commit(s) were added to refs/heads/main by this push:
     new 09feed9  NIFI-9352: Consider all property values in component references
09feed9 is described below

commit 09feed9df6002e3687e52d109c3dca351e945247
Author: Mark Payne <ma...@hotmail.com>
AuthorDate: Sat Oct 30 10:20:42 2021 -0400

    NIFI-9352: Consider all property values in component references
    
    - Ensure that we consider all property values to determine if a component references a variable, not just 'supported' (i.e., well-known/non-user-defined) properties
    
    This closes #5499
    
    Signed-off-by: David Handermann <ex...@apache.org>
---
 .../src/main/java/org/apache/nifi/groups/StandardProcessGroup.java      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-components/src/main/java/org/apache/nifi/groups/StandardProcessGroup.java b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-components/src/main/java/org/apache/nifi/groups/StandardProcessGroup.java
index 7b32364..3583444 100644
--- a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-components/src/main/java/org/apache/nifi/groups/StandardProcessGroup.java
+++ b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-components/src/main/java/org/apache/nifi/groups/StandardProcessGroup.java
@@ -3452,7 +3452,7 @@ public final class StandardProcessGroup implements ProcessGroup {
     }
 
     private boolean isComponentImpactedByVariable(final ComponentNode component, final String variableName) {
-        final List<PropertyDescriptor> propertyDescriptors = component.getPropertyDescriptors();
+        final Set<PropertyDescriptor> propertyDescriptors = component.getRawPropertyValues().keySet();
         for (final PropertyDescriptor descriptor : propertyDescriptors) {
             final PropertyConfiguration propertyConfiguration = component.getProperty(descriptor);
             if (propertyConfiguration.getVariableImpact().isImpacted(variableName)) {