[ https://issues.apache.org/jira/browse/CLOUDSTACK-10362?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16469009#comment-16469009 ]
ASF GitHub Bot commented on CLOUDSTACK-10362: --------------------------------------------- rhtyd closed pull request #2600: CLOUDSTACK-10362: Change the "getXXX" method names to "isXXX". URL: https://github.com/apache/cloudstack/pull/2600 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/api/src/main/java/org/apache/cloudstack/api/command/admin/offering/CreateServiceOfferingCmd.java b/api/src/main/java/org/apache/cloudstack/api/command/admin/offering/CreateServiceOfferingCmd.java index 0bde79bbd7b..3448698cd37 100644 --- a/api/src/main/java/org/apache/cloudstack/api/command/admin/offering/CreateServiceOfferingCmd.java +++ b/api/src/main/java/org/apache/cloudstack/api/command/admin/offering/CreateServiceOfferingCmd.java @@ -166,15 +166,15 @@ public String getServiceOfferingName() { return serviceOfferingName; } - public Boolean getOfferHa() { + public Boolean isOfferHa() { return offerHa == null ? Boolean.FALSE : offerHa; } - public Boolean GetLimitCpuUse() { + public Boolean isLimitCpuUse() { return limitCpuUse == null ? Boolean.FALSE : limitCpuUse; } - public Boolean getVolatileVm() { + public Boolean isVolatileVm() { return isVolatile == null ? Boolean.FALSE : isVolatile; } diff --git a/api/src/main/java/org/apache/cloudstack/api/command/admin/vlan/ListVlanIpRangesCmd.java b/api/src/main/java/org/apache/cloudstack/api/command/admin/vlan/ListVlanIpRangesCmd.java index d4f2d5ad611..d5edd301a66 100644 --- a/api/src/main/java/org/apache/cloudstack/api/command/admin/vlan/ListVlanIpRangesCmd.java +++ b/api/src/main/java/org/apache/cloudstack/api/command/admin/vlan/ListVlanIpRangesCmd.java @@ -119,7 +119,7 @@ public Long getNetworkId() { return networkId; } - public Boolean getForVirtualNetwork() { + public Boolean isForVirtualNetwork() { return forVirtualNetwork; } diff --git a/api/src/main/java/org/apache/cloudstack/api/command/admin/vpc/UpdateVPCCmdByAdmin.java b/api/src/main/java/org/apache/cloudstack/api/command/admin/vpc/UpdateVPCCmdByAdmin.java index 8606c32a71e..d7761def204 100644 --- a/api/src/main/java/org/apache/cloudstack/api/command/admin/vpc/UpdateVPCCmdByAdmin.java +++ b/api/src/main/java/org/apache/cloudstack/api/command/admin/vpc/UpdateVPCCmdByAdmin.java @@ -34,7 +34,7 @@ @Override public void execute(){ - Vpc result = _vpcService.updateVpc(getId(), getVpcName(), getDisplayText(), getCustomId(), getDisplayVpc()); + Vpc result = _vpcService.updateVpc(getId(), getVpcName(), getDisplayText(), getCustomId(), isDisplayVpc()); if (result != null) { VpcResponse response = _responseGenerator.createVpcResponse(ResponseView.Full, result); response.setResponseName(getCommandName()); diff --git a/api/src/main/java/org/apache/cloudstack/api/command/user/address/ListPublicIpAddressesCmd.java b/api/src/main/java/org/apache/cloudstack/api/command/user/address/ListPublicIpAddressesCmd.java index 0a0e7a12076..d590081104a 100644 --- a/api/src/main/java/org/apache/cloudstack/api/command/user/address/ListPublicIpAddressesCmd.java +++ b/api/src/main/java/org/apache/cloudstack/api/command/user/address/ListPublicIpAddressesCmd.java @@ -133,11 +133,11 @@ public Long getAssociatedNetworkId() { return associatedNetworkId; } - public Boolean getIsSourceNat() { + public Boolean isSourceNat() { return isSourceNat; } - public Boolean getIsStaticNat() { + public Boolean isStaticNat() { return isStaticNat; } diff --git a/api/src/main/java/org/apache/cloudstack/api/command/user/network/ListNetworksCmd.java b/api/src/main/java/org/apache/cloudstack/api/command/user/network/ListNetworksCmd.java index 041d6417044..a61c59734c6 100644 --- a/api/src/main/java/org/apache/cloudstack/api/command/user/network/ListNetworksCmd.java +++ b/api/src/main/java/org/apache/cloudstack/api/command/user/network/ListNetworksCmd.java @@ -123,11 +123,11 @@ public Long getPhysicalNetworkId() { return supportedServices; } - public Boolean getRestartRequired() { + public Boolean isRestartRequired() { return restartRequired; } - public Boolean getSpecifyIpRanges() { + public Boolean isSpecifyIpRanges() { return specifyIpRanges; } diff --git a/api/src/main/java/org/apache/cloudstack/api/command/user/vm/DeployVMCmd.java b/api/src/main/java/org/apache/cloudstack/api/command/user/vm/DeployVMCmd.java index 8161fb2564b..7fc54884099 100644 --- a/api/src/main/java/org/apache/cloudstack/api/command/user/vm/DeployVMCmd.java +++ b/api/src/main/java/org/apache/cloudstack/api/command/user/vm/DeployVMCmd.java @@ -256,7 +256,7 @@ public HypervisorType getHypervisor() { return HypervisorType.getType(hypervisor); } - public Boolean getDisplayVm() { + public Boolean isDisplayVm() { return displayVm; } diff --git a/api/src/main/java/org/apache/cloudstack/api/command/user/vpc/UpdateVPCCmd.java b/api/src/main/java/org/apache/cloudstack/api/command/user/vpc/UpdateVPCCmd.java index 94d0e6efcd4..1309334a496 100644 --- a/api/src/main/java/org/apache/cloudstack/api/command/user/vpc/UpdateVPCCmd.java +++ b/api/src/main/java/org/apache/cloudstack/api/command/user/vpc/UpdateVPCCmd.java @@ -73,7 +73,7 @@ public Long getId() { return id; } - public Boolean getDisplayVpc() { + public Boolean isDisplayVpc() { return display; } @@ -97,7 +97,7 @@ public long getEntityOwnerId() { @Override public void execute() { - Vpc result = _vpcService.updateVpc(getId(), getVpcName(), getDisplayText(), getCustomId(), getDisplayVpc()); + Vpc result = _vpcService.updateVpc(getId(), getVpcName(), getDisplayText(), getCustomId(), isDisplayVpc()); if (result != null) { VpcResponse response = _responseGenerator.createVpcResponse(ResponseView.Restricted, result); response.setResponseName(getCommandName()); diff --git a/api/src/main/java/org/apache/cloudstack/api/response/HostResponse.java b/api/src/main/java/org/apache/cloudstack/api/response/HostResponse.java index b9667eca241..3d53682d3b9 100644 --- a/api/src/main/java/org/apache/cloudstack/api/response/HostResponse.java +++ b/api/src/main/java/org/apache/cloudstack/api/response/HostResponse.java @@ -630,7 +630,7 @@ public String getClusterType() { return clusterType; } - public Boolean getLocalStorageActive() { + public Boolean isLocalStorageActive() { return localStorageActive; } @@ -646,11 +646,11 @@ public String getEvents() { return events; } - public Boolean getHasEnoughCapacity() { + public Boolean hasEnoughCapacity() { return hasEnoughCapacity; } - public Boolean getSuitableForMigration() { + public Boolean isSuitableForMigration() { return suitableForMigration; } diff --git a/api/src/main/java/org/apache/cloudstack/api/response/ServiceOfferingResponse.java b/api/src/main/java/org/apache/cloudstack/api/response/ServiceOfferingResponse.java index d004e7ea071..f66d2f82fee 100644 --- a/api/src/main/java/org/apache/cloudstack/api/response/ServiceOfferingResponse.java +++ b/api/src/main/java/org/apache/cloudstack/api/response/ServiceOfferingResponse.java @@ -180,7 +180,7 @@ public void setIsSystemOffering(Boolean isSystem) { this.isSystem = isSystem; } - public Boolean getDefaultUse() { + public Boolean isDefaultUse() { return defaultUse; } diff --git a/api/src/main/java/org/apache/cloudstack/api/response/SnapshotPolicyResponse.java b/api/src/main/java/org/apache/cloudstack/api/response/SnapshotPolicyResponse.java index 39178a3afd6..10710c64803 100644 --- a/api/src/main/java/org/apache/cloudstack/api/response/SnapshotPolicyResponse.java +++ b/api/src/main/java/org/apache/cloudstack/api/response/SnapshotPolicyResponse.java @@ -104,7 +104,7 @@ public void setTimezone(String timezone) { this.timezone = timezone; } - public Boolean getForDisplay() { + public Boolean isForDisplay() { return forDisplay; } diff --git a/api/src/main/java/org/apache/cloudstack/api/response/VolumeResponse.java b/api/src/main/java/org/apache/cloudstack/api/response/VolumeResponse.java index 895e13c5c5c..d845e414808 100644 --- a/api/src/main/java/org/apache/cloudstack/api/response/VolumeResponse.java +++ b/api/src/main/java/org/apache/cloudstack/api/response/VolumeResponse.java @@ -276,7 +276,7 @@ public String getObjectId() { return this.getId(); } - public Boolean getDestroyed() { + public Boolean isDestroyed() { return destroyed; } diff --git a/plugins/database/quota/src/main/java/org/apache/cloudstack/api/command/QuotaSummaryCmd.java b/plugins/database/quota/src/main/java/org/apache/cloudstack/api/command/QuotaSummaryCmd.java index 88466e08c6b..21441ef233a 100644 --- a/plugins/database/quota/src/main/java/org/apache/cloudstack/api/command/QuotaSummaryCmd.java +++ b/plugins/database/quota/src/main/java/org/apache/cloudstack/api/command/QuotaSummaryCmd.java @@ -63,7 +63,7 @@ public void execute() { if (getAccountName() != null && getDomainId() != null) responses = _responseBuilder.createQuotaSummaryResponse(caller.getAccountName(), caller.getDomainId()); else - responses = _responseBuilder.createQuotaSummaryResponse(getListAll()); + responses = _responseBuilder.createQuotaSummaryResponse(isListAll()); } else { responses = _responseBuilder.createQuotaSummaryResponse(caller.getAccountName(), caller.getDomainId()); } @@ -94,7 +94,7 @@ public String getCommandName() { return s_name; } - public Boolean getListAll() { + public Boolean isListAll() { return listAll == null ? false: listAll; } diff --git a/plugins/storage/volume/nexenta/src/main/java/org/apache/cloudstack/storage/datastore/lifecylce/NexentaPrimaryDataStoreLifeCycle.java b/plugins/storage/volume/nexenta/src/main/java/org/apache/cloudstack/storage/datastore/lifecylce/NexentaPrimaryDataStoreLifeCycle.java index 4fffb348a8b..32735664a25 100644 --- a/plugins/storage/volume/nexenta/src/main/java/org/apache/cloudstack/storage/datastore/lifecylce/NexentaPrimaryDataStoreLifeCycle.java +++ b/plugins/storage/volume/nexenta/src/main/java/org/apache/cloudstack/storage/datastore/lifecylce/NexentaPrimaryDataStoreLifeCycle.java @@ -102,7 +102,7 @@ public DataStore initialize(Map<String, Object> dsInfos) { details.put(NexentaUtil.NMS_URL, params.getNmsUrl().toString()); details.put(NexentaUtil.VOLUME, params.getVolume()); - details.put(NexentaUtil.SPARSE_VOLUMES, params.getSparseVolumes().toString()); + details.put(NexentaUtil.SPARSE_VOLUMES, params.isSparseVolumes().toString()); details.put(NexentaUtil.STORAGE_TYPE, params.getStorageType().toString()); details.put(NexentaUtil.STORAGE_HOST, params.getStorageHost()); diff --git a/plugins/storage/volume/nexenta/src/main/java/org/apache/cloudstack/storage/datastore/util/NexentaStorAppliance.java b/plugins/storage/volume/nexenta/src/main/java/org/apache/cloudstack/storage/datastore/util/NexentaStorAppliance.java index c1efc75e124..fbb6645b8e0 100644 --- a/plugins/storage/volume/nexenta/src/main/java/org/apache/cloudstack/storage/datastore/util/NexentaStorAppliance.java +++ b/plugins/storage/volume/nexenta/src/main/java/org/apache/cloudstack/storage/datastore/util/NexentaStorAppliance.java @@ -337,7 +337,7 @@ NexentaStorZvol createIscsiVolume(String volumeName, Long volumeSize) { final String zvolName = getVolumeName(volumeName); String volumeSizeString = String.format("%dB", volumeSize); - client.execute(NmsResponse.class, "zvol", "create", zvolName, volumeSizeString, parameters.getVolumeBlockSize(), parameters.getSparseVolumes()); + client.execute(NmsResponse.class, "zvol", "create", zvolName, volumeSizeString, parameters.getVolumeBlockSize(), parameters.isSparseVolumes()); final String targetName = getTargetName(volumeName); final String targetGroupName = getTargetGroupName(volumeName); diff --git a/plugins/storage/volume/nexenta/src/main/java/org/apache/cloudstack/storage/datastore/util/NexentaUtil.java b/plugins/storage/volume/nexenta/src/main/java/org/apache/cloudstack/storage/datastore/util/NexentaUtil.java index ee6a78ff49c..7eef85b281e 100644 --- a/plugins/storage/volume/nexenta/src/main/java/org/apache/cloudstack/storage/datastore/util/NexentaUtil.java +++ b/plugins/storage/volume/nexenta/src/main/java/org/apache/cloudstack/storage/datastore/util/NexentaUtil.java @@ -194,7 +194,7 @@ public void setSparseVolumes(String sparseVolumes) { this.sparseVolumes = Boolean.TRUE.toString().equalsIgnoreCase(sparseVolumes); } - public Boolean getSparseVolumes() { + public Boolean isSparseVolumes() { return sparseVolumes; } diff --git a/plugins/storage/volume/nexenta/src/test/java/org/apache/cloudstack/storage/datastore/util/NexentaUtilTest.java b/plugins/storage/volume/nexenta/src/test/java/org/apache/cloudstack/storage/datastore/util/NexentaUtilTest.java index 8f4498557c1..45bffb09107 100644 --- a/plugins/storage/volume/nexenta/src/test/java/org/apache/cloudstack/storage/datastore/util/NexentaUtilTest.java +++ b/plugins/storage/volume/nexenta/src/test/java/org/apache/cloudstack/storage/datastore/util/NexentaUtilTest.java @@ -74,7 +74,7 @@ public void testParseNexentaPluginUrl() { assertEquals(parameters.getStorageHost(), "192.168.1.1"); assertEquals((int) parameters.getStoragePort(), NexentaUtil.DEFAULT_ISCSI_TARGET_PORTAL_PORT); assertNull(parameters.getStoragePath()); - assertEquals((boolean) parameters.getSparseVolumes(), false); + assertEquals((boolean) parameters.isSparseVolumes(), false); assertEquals(parameters.getVolumeBlockSize(), "8K"); url += "volume=cloudstack"; @@ -109,7 +109,7 @@ public void testParseNexentaPluginUrl() { url += "sparseVolumes=true;"; parameters = NexentaUtil.parseNexentaPluginUrl(url); - assertEquals(parameters.getSparseVolumes(), Boolean.TRUE); + assertEquals(parameters.isSparseVolumes(), Boolean.TRUE); url += "volumeBlockSize=128K;"; parameters = NexentaUtil.parseNexentaPluginUrl(url); @@ -124,7 +124,7 @@ public void testParseNexentaPluginUrl() { assertEquals(parameters.getStorageHost(), "192.168.1.2"); assertEquals((int) parameters.getStoragePort(), 3000); assertEquals(parameters.getStoragePath(), "/volumes/cloudstack"); - assertEquals(parameters.getSparseVolumes(), Boolean.TRUE); + assertEquals(parameters.isSparseVolumes(), Boolean.TRUE); assertEquals(parameters.getVolumeBlockSize(), "128K"); exception.expect(RuntimeException.class); diff --git a/server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java b/server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java index 80642f51375..a0ce7f3ea1d 100755 --- a/server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java +++ b/server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java @@ -2255,7 +2255,7 @@ public ServiceOffering createServiceOffering(final CreateServiceOfferingCmd cmd) throw new InvalidParameterValueException("Please specify a valid domain id"); } - final Boolean offerHA = cmd.getOfferHa(); + final Boolean offerHA = cmd.isOfferHa(); boolean localStorageRequired = false; final String storageType = cmd.getStorageType(); @@ -2270,8 +2270,8 @@ public ServiceOffering createServiceOffering(final CreateServiceOfferingCmd cmd) } } - final Boolean limitCpuUse = cmd.GetLimitCpuUse(); - final Boolean volatileVm = cmd.getVolatileVm(); + final Boolean limitCpuUse = cmd.isLimitCpuUse(); + final Boolean volatileVm = cmd.isVolatileVm(); final String vmTypeString = cmd.getSystemVmType(); VirtualMachine.Type vmType = null; diff --git a/server/src/main/java/com/cloud/network/NetworkServiceImpl.java b/server/src/main/java/com/cloud/network/NetworkServiceImpl.java index c121d1e75a1..8d93b7d4dfe 100644 --- a/server/src/main/java/com/cloud/network/NetworkServiceImpl.java +++ b/server/src/main/java/com/cloud/network/NetworkServiceImpl.java @@ -1427,10 +1427,10 @@ public Network doInTransaction(TransactionStatus status) throws InsufficientCapa String path = null; Long physicalNetworkId = cmd.getPhysicalNetworkId(); List<String> supportedServicesStr = cmd.getSupportedServices(); - Boolean restartRequired = cmd.getRestartRequired(); + Boolean restartRequired = cmd.isRestartRequired(); boolean listAll = cmd.listAll(); boolean isRecursive = cmd.isRecursive(); - Boolean specifyIpRanges = cmd.getSpecifyIpRanges(); + Boolean specifyIpRanges = cmd.isSpecifyIpRanges(); Long vpcId = cmd.getVpcId(); Boolean canUseForDeploy = cmd.canUseForDeploy(); Map<String, String> tags = cmd.getTags(); diff --git a/server/src/main/java/com/cloud/server/ManagementServerImpl.java b/server/src/main/java/com/cloud/server/ManagementServerImpl.java index 3082f1dfa72..faa81f317b0 100644 --- a/server/src/main/java/com/cloud/server/ManagementServerImpl.java +++ b/server/src/main/java/com/cloud/server/ManagementServerImpl.java @@ -1588,7 +1588,7 @@ private boolean hasSuitablePoolsForVolume(final VolumeVO volume, final Host host final Long domainId = cmd.getDomainId(); Long accountId = null; final Long networkId = cmd.getNetworkId(); - final Boolean forVirtual = cmd.getForVirtualNetwork(); + final Boolean forVirtual = cmd.isForVirtualNetwork(); String vlanType = null; final Long projectId = cmd.getProjectId(); final Long physicalNetworkId = cmd.getPhysicalNetworkId(); @@ -1823,8 +1823,8 @@ private boolean hasSuitablePoolsForVolume(final VolumeVO volume, final Host host final Boolean forVirtualNetwork = cmd.isForVirtualNetwork(); final Boolean forLoadBalancing = cmd.isForLoadBalancing(); final Long ipId = cmd.getId(); - final Boolean sourceNat = cmd.getIsSourceNat(); - final Boolean staticNat = cmd.getIsStaticNat(); + final Boolean sourceNat = cmd.isSourceNat(); + final Boolean staticNat = cmd.isStaticNat(); final Long vpcId = cmd.getVpcId(); final Boolean forDisplay = cmd.getDisplay(); final Map<String, String> tags = cmd.getTags(); diff --git a/server/src/main/java/com/cloud/storage/TemplateProfile.java b/server/src/main/java/com/cloud/storage/TemplateProfile.java index 410b519968a..304b652a589 100644 --- a/server/src/main/java/com/cloud/storage/TemplateProfile.java +++ b/server/src/main/java/com/cloud/storage/TemplateProfile.java @@ -164,7 +164,7 @@ public void setBits(Integer bits) { this.bits = bits; } - public Boolean getPasswordEnabled() { + public Boolean isPasswordEnabled() { return passwordEnabled; } @@ -172,7 +172,7 @@ public void setPasswordEnabled(Boolean enabled) { this.passwordEnabled = enabled; } - public Boolean getRequiresHVM() { + public Boolean isRequiresHVM() { return requiresHvm; } @@ -188,7 +188,7 @@ public void setUrl(String url) { this.url = url; } - public Boolean getIsPublic() { + public Boolean isPublic() { return isPublic; } @@ -196,7 +196,7 @@ public void setIsPublic(Boolean is) { this.isPublic = is; } - public Boolean getFeatured() { + public Boolean isFeatured() { return featured; } @@ -204,7 +204,7 @@ public void setFeatured(Boolean featured) { this.featured = featured; } - public Boolean getIsExtractable() { + public Boolean isExtractable() { return isExtractable; } @@ -264,7 +264,7 @@ public void setCheckSum(String chksum) { this.chksum = chksum; } - public Boolean getBootable() { + public Boolean isBootable() { return this.bootable; } @@ -300,7 +300,7 @@ public void setSshKeyEnabled(Boolean enabled) { this.sshKeyEnbaled = enabled; } - public Boolean getSshKeyEnabled() { + public Boolean isSshKeyEnabled() { return this.sshKeyEnbaled; } diff --git a/server/src/main/java/com/cloud/template/TemplateAdapterBase.java b/server/src/main/java/com/cloud/template/TemplateAdapterBase.java index 7467e30aa01..4a51b695157 100644 --- a/server/src/main/java/com/cloud/template/TemplateAdapterBase.java +++ b/server/src/main/java/com/cloud/template/TemplateAdapterBase.java @@ -338,10 +338,10 @@ public TemplateProfile prepare(RegisterIsoCmd cmd) throws ResourceAllocationExce protected VMTemplateVO persistTemplate(TemplateProfile profile, VirtualMachineTemplate.State initialState) { List<Long> zoneIdList = profile.getZoneIdList(); VMTemplateVO template = - new VMTemplateVO(profile.getTemplateId(), profile.getName(), profile.getFormat(), profile.getIsPublic(), profile.getFeatured(), profile.getIsExtractable(), - profile.getTemplateType(), profile.getUrl(), profile.getRequiresHVM(), profile.getBits(), profile.getAccountId(), profile.getCheckSum(), - profile.getDisplayText(), profile.getPasswordEnabled(), profile.getGuestOsId(), profile.getBootable(), profile.getHypervisorType(), - profile.getTemplateTag(), profile.getDetails(), profile.getSshKeyEnabled(), profile.IsDynamicallyScalable(), profile.isDirectDownload()); + new VMTemplateVO(profile.getTemplateId(), profile.getName(), profile.getFormat(), profile.isPublic(), profile.isFeatured(), profile.isExtractable(), + profile.getTemplateType(), profile.getUrl(), profile.isRequiresHVM(), profile.getBits(), profile.getAccountId(), profile.getCheckSum(), + profile.getDisplayText(), profile.isPasswordEnabled(), profile.getGuestOsId(), profile.isBootable(), profile.getHypervisorType(), + profile.getTemplateTag(), profile.getDetails(), profile.isSshKeyEnabled(), profile.IsDynamicallyScalable(), profile.isDirectDownload()); template.setState(initialState); if (profile.isDirectDownload()) { diff --git a/server/src/main/java/com/cloud/vm/UserVmManagerImpl.java b/server/src/main/java/com/cloud/vm/UserVmManagerImpl.java index df81dd3ff2b..dc4440d0cca 100644 --- a/server/src/main/java/com/cloud/vm/UserVmManagerImpl.java +++ b/server/src/main/java/com/cloud/vm/UserVmManagerImpl.java @@ -4791,7 +4791,7 @@ public UserVm createVirtualMachine(DeployVMCmd cmd) throws InsufficientCapacityE String group = cmd.getGroup(); String userData = cmd.getUserData(); String sshKeyPairName = cmd.getSSHKeyPairName(); - Boolean displayVm = cmd.getDisplayVm(); + Boolean displayVm = cmd.isDisplayVm(); String keyboard = cmd.getKeyboard(); Map<Long, DiskOffering> dataDiskTemplateToDiskOfferingMap = cmd.getDataDiskTemplateToDiskOfferingMap(); if (zone.getNetworkType() == NetworkType.Basic) { ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Inconsistent method names > ------------------------- > > Key: CLOUDSTACK-10362 > URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10362 > Project: CloudStack > Issue Type: Improvement > Security Level: Public(Anyone can view this level - this is the > default.) > Reporter: KuiLIU > Priority: Major > > Some Boolean-return methods are named "getXXX", > but other Boolean-return methods are named "isXXX", such as the following two > methods. > They will return boolean values, rename them as "isXXX" should be more clear > than "getXXX". > api/src/main/java/org/apache/cloudstack/api/command/admin/vlan/ListVlanIpRangesCmd.java > {code:java} > public Boolean getForVirtualNetwork() { > return forVirtualNetwork; > } > {code} > api/src/main/java/org/apache/cloudstack/api/command/user/address/ListPublicIpAddressesCmd.java > {code:java} > public Boolean isForVirtualNetwork() { > return forVirtualNetwork; > } > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)