s9514171 commented on a change in pull request #5060:
URL: https://github.com/apache/nifi/pull/5060#discussion_r635921165
##########
File path:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-parameter-contexts.js
##########
@@ -2397,8 +2397,6 @@
* @param parameterToSelect Optional, name of the parameter to
select in the table.
*/
showParameterContext: function (id, readOnly, parameterToSelect) {
- parameterCount = 0;
Review comment:
@scottyaslan sorry for the late reply, the first fix idea is referenced
from variable handling..., but it seems to have the potential issue as you
mentioned. maybe we can also fix it too.
I'm reviewing it again, and currently, I find there is a better way to solve
it. I'll update later after testing
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
