apache/cloudstack
1
0
Fork 0
mirror of https://github.com/apache/cloudstack.git synced 2025-10-26 08:42:29 +01:00
Code Issues Packages Projects Releases Wiki Activity

CLOUDSTACK-10360: Change the method name. (#2598)

Browse Source 
The 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".
This commit is contained in:
Kui LIU 2018-04-30 09:24:47 +02:00 committed by Rohit Yadav
parent 4277b92abe
commit 93509a431c
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
framework/config/src/main/java/org/apache/cloudstack/framework/config/ConfigKey.java
View File

@ -152,7 +152,7 @@ public class ConfigKey<T> {
return value(); return value();
} }
String value = s_depot != null ? s_depot.scoped(this).getConfigValue(id, this) : null; String value = s_depot != null ? s_depot.findScopedConfigStorage(this).getConfigValue(id, this) : null;
if (value == null) { if (value == null) {
return value(); return value();
} else { } else {

2
framework/config/src/main/java/org/apache/cloudstack/framework/config/impl/ConfigDepotImpl.java
View File

@ -166,7 +166,7 @@ public class ConfigDepotImpl implements ConfigDepot, ConfigDepotAdmin {
return _configDao; return _configDao;
} }
public ScopedConfigStorage scoped(ConfigKey<?> config) { public ScopedConfigStorage findScopedConfigStorage(ConfigKey<?> config) {
for (ScopedConfigStorage storage : _scopedStorages) { for (ScopedConfigStorage storage : _scopedStorages) {
if (storage.getScope() == config.scope()) { if (storage.getScope() == config.scope()) {
return storage; return storage;