sardell commented on code in PR #7924:
URL: https://github.com/apache/nifi/pull/7924#discussion_r1372133318


##########
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-policy-management.js:
##########
@@ -1611,60 +1611,6 @@
             return loadPolicy().always(showPolicy);
         },
 
-        /**
-         * Shows the template policy.
-         *
-         * @param d
-         */
-        showTemplatePolicy: function (d) {
-            initializingComponentPolicy = true;
-
-            // reset the policy message
-            resetPolicyMessage();
-
-            // update the policy controls visibility
-            $('#component-policy-controls').show();
-            $('#global-policy-controls').hide();
-
-            // update the visibility
-            if (d.permissions.canRead === true) {
-                $('#policy-selected-template-container 
div.policy-selected-component-name').text(d.template.name);
-            } else {
-                $('#policy-selected-template-container 
div.policy-selected-component-name').text(d.id);
-            }
-            $('#policy-selected-template-container').show();
-
-            // populate the initial resource
-            $('#selected-policy-component-id').text(d.id);
-            $('#selected-policy-component-type').text('templates');
-            $('#component-policy-target')
-                .combo('setOptionEnabled', {
-                    value: 'operate-component'
-                }, false)
-                .combo('setOptionEnabled', {
-                    value: 'write-receive-data'
-                }, false)
-                .combo('setOptionEnabled', {
-                    value: 'write-send-data'
-                }, false)
-                .combo('setOptionEnabled', {
-                    value: 'read-data'
-                }, false)
-                .combo('setOptionEnabled', {
-                    value: 'read-provenance'
-                }, false)
-                .combo('setOptionEnabled', {
-                    value: 'write-data'
-                }, false)
-                .combo('setSelectedOption', {
-                    value: 'read-component'
-                });
-
-            initializingComponentPolicy = false;
-
-            return loadPolicy().always(showPolicy);
-        },
-

Review Comment:
   Yeah, apologies if I was heavy handed here. 'Template' the nifi term and 
html/js 'templates' were causing me some confusion. I'll investigate some more 
but interested to see what @scottyaslan says.



-- 
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: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to