You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by an...@apache.org on 2014/01/28 05:09:11 UTC
[3/4] git commit: updated refs/heads/4.3 to ce1b393
CLOUDSTACK-5960: Domain admin or user cannot register a template using
S3/Swift object store.(cherry picked from commit 6d1475182cd9ea17010a142043dd9fe9c923cf81)
Signed-off-by: Animesh Chaturvedi <an...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/7e8b8e6c
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/7e8b8e6c
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/7e8b8e6c
Branch: refs/heads/4.3
Commit: 7e8b8e6cd05efe5098a662610a31571a9eaca684
Parents: 8d1d0c5
Author: Min Chen <mi...@citrix.com>
Authored: Mon Jan 27 11:54:38 2014 -0800
Committer: Animesh Chaturvedi <an...@apache.org>
Committed: Mon Jan 27 19:52:06 2014 -0800
----------------------------------------------------------------------
server/src/com/cloud/template/TemplateAdapterBase.java | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/7e8b8e6c/server/src/com/cloud/template/TemplateAdapterBase.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/template/TemplateAdapterBase.java b/server/src/com/cloud/template/TemplateAdapterBase.java
index 6194e6e..c75c1e4 100755
--- a/server/src/com/cloud/template/TemplateAdapterBase.java
+++ b/server/src/com/cloud/template/TemplateAdapterBase.java
@@ -162,8 +162,14 @@ public abstract class TemplateAdapterBase extends AdapterBase implements Templat
}
boolean isAdmin = _accountDao.findById(templateOwner.getId()).getType() == Account.ACCOUNT_TYPE_ADMIN;
-
- if (!isAdmin && zoneId == null) {
+ boolean isRegionStore = false;
+ List<ImageStoreVO> stores = _imgStoreDao.findRegionImageStores();
+ if (stores != null && stores.size() > 0) {
+ isRegionStore = true;
+ }
+
+ if (!isAdmin && zoneId == null && !isRegionStore ) {
+ // domain admin and user should also be able to register template on a region store
throw new InvalidParameterValueException("Please specify a valid zone Id.");
}