Github user mcgilman commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/2990#discussion_r217405219
  
    --- Diff: 
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-remote-process-group.js
 ---
    @@ -741,8 +740,8 @@
                 })
                 .attr('font-family', function (d) {
                     var family = '';
    -                if (d.permissions.canRead) {
    -                    if (hasIssues(d) || d.component.transmitting) {
    +                if (d.permissions.canRead || 
d.operatePermissions.canWrite) {
    --- End diff --
    
    Actually, for determining which icon we show and the appropriate style it 
appears that we are now only using the status. This is available to everyone so 
I don't think we need any permission checks here. We should only need them when 
adding the tooltips. This should be very similar to the Processor Invalid icon 
and the tooltips for the validation errors.


---

Reply via email to