KuiLIU created CLOUDSTACK-10360:
-----------------------------------
Summary: Inconsistent method name
Key: CLOUDSTACK-10360
URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10360
Project: CloudStack
Issue Type: Improvement
Security Level: Public (Anyone can view this level - this is the default.)
Reporter: KuiLIU
The following method is named as "scoped" that seems to whether the variable
config is scoped in _scopedStorages or not.
Actually, the method tries to find a storage of which scope equals to the scope
of config.
So that, the method name "findStorage" should be more clear than "scoped".
{code:java}
public ScopedConfigStorage scoped(ConfigKey<?> config) {
for (ScopedConfigStorage storage : _scopedStorages) {
if (storage.getScope() == config.scope()) {
return storage;
}
}
throw new CloudRuntimeException("Unable to find config storage for this
scope: " + config.scope() + " for " + config.key());
}
{code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)