modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/bundle/group/BundleGroupEditView.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
New commits: commit 5727aa8d54b13521bc791d561a8f17abaf64f177 Author: John Mazzitelli mazz@redhat.com Date: Thu Oct 3 09:45:45 2013 -0400
BZ 1014593 - make sure we do the initial permission check. currently, the code mistakenly thought it had cached the permissions when it didn't.
diff --git a/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/bundle/group/BundleGroupEditView.java b/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/bundle/group/BundleGroupEditView.java index 9da0e4c..f898680 100644 --- a/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/bundle/group/BundleGroupEditView.java +++ b/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/bundle/group/BundleGroupEditView.java @@ -56,7 +56,7 @@ public class BundleGroupEditView extends AbstractRecordEditor<BundleGroupsDataSo private Tab bundlesTab; private BundleSelector bundleSelector; private Set<Permission> globalPermissions; - private int permissionCheckBundleGroupId = 0; + private Integer permissionCheckBundleGroupId = null; private boolean canManage; private boolean canAssign; private boolean canUnassign; @@ -72,7 +72,7 @@ public class BundleGroupEditView extends AbstractRecordEditor<BundleGroupsDataSo final int bundleGroupId = Integer.parseInt(viewPath.getCurrent().getPath());
// if we have already determined permissions for this bundle, just proceed - if (permissionCheckBundleGroupId == bundleGroupId) { + if (null != permissionCheckBundleGroupId && permissionCheckBundleGroupId.intValue() == bundleGroupId) { authorizedRenderView(bundleGroupId, viewPath); return; }