modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/groups/definitions/SingleGroupDefinitionView.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
New commits: commit 6689f8e0e7057c2e183879a7a3bbbbb05f36d34b Author: Joseph Marques joseph@redhat.com Date: Wed Sep 22 14:35:10 2010 -0400
only read from viewPath if the user is allowed to access this view
diff --git a/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/groups/definitions/SingleGroupDefinitionView.java b/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/groups/definitions/SingleGroupDefinitionView.java index bcdb9ca..ef55f63 100644 --- a/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/groups/definitions/SingleGroupDefinitionView.java +++ b/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/groups/definitions/SingleGroupDefinitionView.java @@ -333,10 +333,7 @@ public class SingleGroupDefinitionView extends LocatableVLayout implements Bookm }
@Override - public void renderView(ViewPath viewPath) { - groupDefinitionId = viewPath.getCurrentAsInt(); - viewId = viewPath.getCurrent(); - basePath = viewPath.getPathToCurrent(); + public void renderView(final ViewPath viewPath) { GWTServiceLookup.getAuthorizationService().getExplicitGlobalPermissions(new AsyncCallback<Set<Permission>>() { @Override public void onFailure(Throwable caught) { @@ -355,6 +352,9 @@ public class SingleGroupDefinitionView extends LocatableVLayout implements Bookm if (result.contains(Permission.MANAGE_INVENTORY) == false) { handleAuthorizationFailure(); } else { + groupDefinitionId = viewPath.getCurrentAsInt(); + viewId = viewPath.getCurrent(); + basePath = viewPath.getPathToCurrent(); lookupDetails(groupDefinitionId); } }
rhq-commits@lists.fedorahosted.org