You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2020/08/13 08:08:11 UTC

[GitHub] [cloudstack] shwstppr commented on a change in pull request #4250: [VMware] OVF Full properties support

shwstppr commented on a change in pull request #4250:
URL: https://github.com/apache/cloudstack/pull/4250#discussion_r469772225



##########
File path: api/src/main/java/org/apache/cloudstack/api/command/user/template/ListTemplatesCmd.java
##########
@@ -99,7 +99,7 @@
     public EnumSet<ApiConstants.DomainDetails> getDetails() throws InvalidParameterValueException {
         EnumSet<ApiConstants.DomainDetails> dv;
         if (CollectionUtils.isEmpty(viewDetails)) {
-            dv = EnumSet.of(ApiConstants.DomainDetails.min);
+            dv = EnumSet.of(ApiConstants.DomainDetails.all);

Review comment:
       @nvazquez any particular reason, we are listing all details by default? We can set it min and call API with details=all in UI when needed like DeployVM or template view




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org