diff --git a/api/src/com/cloud/api/commands/ListIsosCmd.java b/api/src/com/cloud/api/commands/ListIsosCmd.java index 0b3cd18af64..c2765008620 100755 --- a/api/src/com/cloud/api/commands/ListIsosCmd.java +++ b/api/src/com/cloud/api/commands/ListIsosCmd.java @@ -126,8 +126,6 @@ public class ListIsosCmd extends BaseListCmd { } public boolean listInReadyState() { - return true; - /* Account account = UserContext.current().getCaller(); // It is account specific if account is admin type and domainId and accountName are not null boolean isAccountSpecific = (account == null || isAdmin(account.getType())) && (getAccountName() != null) && (getDomainId() != null); @@ -136,7 +134,6 @@ public class ListIsosCmd extends BaseListCmd { boolean onlyReady = (templateFilter == TemplateFilter.featured) || (templateFilter == TemplateFilter.selfexecutable) || (templateFilter == TemplateFilter.sharedexecutable) || (templateFilter == TemplateFilter.executable && isAccountSpecific) || (templateFilter == TemplateFilter.community); return onlyReady; - */ } ///////////////////////////////////////////////////// @@ -174,7 +171,7 @@ public class ListIsosCmd extends BaseListCmd { isAdmin = true; } - ListResponse response = _responseGenerator.createIsoResponse(isoZonePairSet, isAdmin, account, bootable, true); + ListResponse response = _responseGenerator.createIsoResponse(isoZonePairSet, isAdmin, account, bootable, listInReadyState()); response.setResponseName(getCommandName()); this.setResponseObject(response); } diff --git a/api/src/com/cloud/api/commands/ListTemplatesCmd.java b/api/src/com/cloud/api/commands/ListTemplatesCmd.java index bf5110d826d..6f58acd829f 100755 --- a/api/src/com/cloud/api/commands/ListTemplatesCmd.java +++ b/api/src/com/cloud/api/commands/ListTemplatesCmd.java @@ -107,8 +107,6 @@ public class ListTemplatesCmd extends BaseListCmd { } public boolean listInReadyState() { - return true; - /* Account account = UserContext.current().getCaller(); // It is account specific if account is admin type and domainId and accountName are not null boolean isAccountSpecific = (account == null || isAdmin(account.getType())) && (getAccountName() != null) && (getDomainId() != null); @@ -117,7 +115,6 @@ public class ListTemplatesCmd extends BaseListCmd { boolean onlyReady = (templateFilter == TemplateFilter.featured) || (templateFilter == TemplateFilter.selfexecutable) || (templateFilter == TemplateFilter.sharedexecutable) || (templateFilter == TemplateFilter.executable && isAccountSpecific) || (templateFilter == TemplateFilter.community); return onlyReady; - */ } ///////////////////////////////////////////////////// @@ -147,7 +144,7 @@ public class ListTemplatesCmd extends BaseListCmd { List templateResponses = new ArrayList(); for (Pair template : templateZonePairSet) { - _responseGenerator.createTemplateResponse(templateResponses, template, isAdmin, account, true); + _responseGenerator.createTemplateResponse(templateResponses, template, isAdmin, account, listInReadyState()); } response.setResponses(templateResponses);