From 97ec3f6628bfcbe1ad89847005a6c03ce31e6c53 Mon Sep 17 00:00:00 2001 From: Min Chen Date: Tue, 11 Feb 2014 22:26:52 -0800 Subject: [PATCH] Fix an AclEntityType cast bug. --- .../apache/cloudstack/acl/api/AclApiServiceImpl.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/services/iam/plugin/src/org/apache/cloudstack/acl/api/AclApiServiceImpl.java b/services/iam/plugin/src/org/apache/cloudstack/acl/api/AclApiServiceImpl.java index b6f1caad84e..29aa25854c9 100644 --- a/services/iam/plugin/src/org/apache/cloudstack/acl/api/AclApiServiceImpl.java +++ b/services/iam/plugin/src/org/apache/cloudstack/acl/api/AclApiServiceImpl.java @@ -255,16 +255,16 @@ public class AclApiServiceImpl extends ManagerBase implements AclApiService, Man private void addDomainWideResourceAccess(Map params) { - String entityType = (String) params.get(ApiConstants.ENTITY_TYPE); + AclEntityType entityType = (AclEntityType)params.get(ApiConstants.ENTITY_TYPE); Long entityId = (Long) params.get(ApiConstants.ENTITY_ID); Long domainId = (Long) params.get(ApiConstants.DOMAIN_ID); Boolean isRecursive = (Boolean) params.get(ApiConstants.SUBDOMAIN_ACCESS); - if (AclEntityType.Network.toString().equals(entityType)) { - createPolicyAndAddToDomainGroup("DomainWideNetwork-" + entityId, "domain wide network", entityType, + if (entityType == AclEntityType.Network) { + createPolicyAndAddToDomainGroup("DomainWideNetwork-" + entityId, "domain wide network", entityType.toString(), entityId, "listNetworks", AccessType.UseEntry, domainId, isRecursive); - } else if (AclEntityType.AffinityGroup.toString().equals(entityType)) { - createPolicyAndAddToDomainGroup("DomainWideNetwork-" + entityId, "domain wide affinityGroup", entityType, + } else if (entityType == AclEntityType.AffinityGroup) { + createPolicyAndAddToDomainGroup("DomainWideNetwork-" + entityId, "domain wide affinityGroup", entityType.toString(), entityId, "listAffinityGroups", AccessType.UseEntry, domainId, isRecursive); }