CLOUDSTACK-6436: list* api commands with forDisplay parameter - don't return resources with display=false by default. ForDisplay=false has to be explicitly specified in the command

Alena Prokharchyk · Alena Prokharchyk · commit 664e5e3190ef · 2014-04-17T13:51:30.000-07:00
diff --git a/api/src/org/apache/cloudstack/api/BaseListAccountResourcesCmd.java b/api/src/org/apache/cloudstack/api/BaseListAccountResourcesCmd.java
@@ -16,9 +16,6 @@
 // under the License.
 package org.apache.cloudstack.api;
 
-import org.apache.cloudstack.context.CallContext;
-
-import com.cloud.user.Account;
 
 public abstract class BaseListAccountResourcesCmd extends BaseListDomainResourcesCmd {
 
@@ -30,10 +27,6 @@ public String getAccountName() {
     }
 
     public Boolean getDisplay() {
-        Account caller = CallContext.current().getCallingAccount();
-        if (caller.getType() == Account.ACCOUNT_TYPE_NORMAL) {
-            return true;
-        }
-        return null;
+        return true;
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -16,9 +16,6 @@`
`16`	`16`	`// under the License.`
`17`	`17`	`package org.apache.cloudstack.api;`
`18`	`18`
`19`		`-import org.apache.cloudstack.context.CallContext;`
`20`		`-`
`21`		`-import com.cloud.user.Account;`
`22`	`19`
`23`	`20`	`public abstract class BaseListAccountResourcesCmd extends BaseListDomainResourcesCmd {`
`24`	`21`
`@@ -30,10 +27,6 @@ public String getAccountName() {`
`30`	`27`	`}`
`31`	`28`
`32`	`29`	`public Boolean getDisplay() {`
`33`		`- Account caller = CallContext.current().getCallingAccount();`
`34`		`- if (caller.getType() == Account.ACCOUNT_TYPE_NORMAL) {`
`35`		`- return true;`
`36`		`- }`
`37`		`- return null;`
	`30`	`+ return true;`
`38`	`31`	`}`
`39`	`32`	`}`