You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nifi.apache.org by mc...@apache.org on 2017/05/05 16:38:22 UTC

nifi git commit: [NIFI-3813] fix broken PG config dialog when user has read only perms on the PG. This closes #1757

Repository: nifi
Updated Branches:
  refs/heads/master d9410d640 -> 85405dae1


[NIFI-3813] fix broken PG config dialog when user has read only perms on the PG. This closes #1757


Project: http://git-wip-us.apache.org/repos/asf/nifi/repo
Commit: http://git-wip-us.apache.org/repos/asf/nifi/commit/85405dae
Tree: http://git-wip-us.apache.org/repos/asf/nifi/tree/85405dae
Diff: http://git-wip-us.apache.org/repos/asf/nifi/diff/85405dae

Branch: refs/heads/master
Commit: 85405dae15ee76128818634c7bd45201a8b1a2d2
Parents: d9410d6
Author: Scott Aslan <sc...@gmail.com>
Authored: Fri May 5 11:07:46 2017 -0400
Committer: Matt Gilman <ma...@gmail.com>
Committed: Fri May 5 12:37:56 2017 -0400

----------------------------------------------------------------------
 .../webapp/js/nf/canvas/nf-process-group-configuration.js     | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/nifi/blob/85405dae/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-process-group-configuration.js
----------------------------------------------------------------------
diff --git a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-process-group-configuration.js b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-process-group-configuration.js
index 00810c7..498fbd7 100644
--- a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-process-group-configuration.js
+++ b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-process-group-configuration.js
@@ -177,8 +177,9 @@
                 // store the process group
                 $('#process-group-configuration').data('process-group', response);
 
+                var processGroup = response.component;
+
                 if (response.permissions.canWrite) {
-                    var processGroup = response.component;
 
                     // populate the process group settings
                     $('#process-group-name').removeClass('unset').val(processGroup.name);
@@ -196,8 +197,8 @@
                 } else {
                     if (response.permissions.canRead) {
                         // populate the process group settings
-                        $('#read-only-process-group-name').removeClass('unset').text(response.component.name);
-                        $('#read-only-process-group-comments').removeClass('unset').text(response.component.comments);
+                        $('#read-only-process-group-name').removeClass('unset').text(processGroup.name);
+                        $('#read-only-process-group-comments').removeClass('unset').text(processGroup.comments);
 
                         // populate the header
                         $('#process-group-configuration-header-text').text(processGroup.name + ' Configuration');