mcgilman commented on code in PR #7117:
URL: https://github.com/apache/nifi/pull/7117#discussion_r1185464464


##########
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/jquery/propertytable/jquery.propertytable.js:
##########
@@ -1931,13 +1948,20 @@
                                 if (property.hidden === false) {
                                     // Get the property value by propertyName
                                     var propertyValue = 
properties[dependency.propertyName];
+                                    referencingParameter = null;
+
+                                    // check if the property references a 
parameter
+                                    if (!_.isEmpty(currentParameterContext)) {

Review Comment:
   I believe we already have a `getParameterContext` callback in the original 
`options` with the Property Table to support actions like Convert to Parameter. 
We should be able to simply invoke that callback above, within this event, to 
get the parameter context if applicable. The callback will not be set when it's 
not applicable. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to