natas_hw
/
dubbo
0
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

add empty check at the start of setter.

This commit is contained in:
ken.lj 2019-01-07 21:25:14 +08:00
parent a66784ae12
commit d4532c6f8f
4 changed files with 24 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/AbstractInterfaceConfig.java
View File

@ -673,6 +673,9 @@ public abstract class AbstractInterfaceConfig extends AbstractMethodConfig {
}
public void setApplication(ApplicationConfig application) {
if (application == null) {
return;
}
application.refresh();
this.application = application;
ConfigManager.getInstance().setApplication(this.application);
@ -691,6 +694,9 @@ public abstract class AbstractInterfaceConfig extends AbstractMethodConfig {
}
public void setModule(ModuleConfig module) {
if (module == null) {
return;
}
this.module = module;
ConfigManager.getInstance().setModule(module);
}
@ -711,6 +717,9 @@ public abstract class AbstractInterfaceConfig extends AbstractMethodConfig {
@SuppressWarnings({"unchecked"})
public void setRegistries(List<? extends RegistryConfig> registries) {
if (registries == null) {
return;
}
for (RegistryConfig registryConfig : registries) {
registryConfig.refresh();
if (StringUtils.isNotEmpty(registryConfig.getId())) {
@ -740,6 +749,9 @@ public abstract class AbstractInterfaceConfig extends AbstractMethodConfig {
}
public void setMonitor(MonitorConfig monitor) {
if (monitor == null) {
return;
}
monitor.refresh();
this.monitor = monitor;
ConfigManager.getInstance().setMonitor(monitor);
@ -759,6 +771,9 @@ public abstract class AbstractInterfaceConfig extends AbstractMethodConfig {
}
public void setConfigCenter(ConfigCenterConfig configCenter) {
if (configCenter == null) {
return;
}
// give jvm properties the chance to override local configs, e.g., -Ddubbo.configcenter.highestPriority
configCenter.refresh();
this.configCenter = configCenter;

3
dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/AbstractServiceConfig.java
View File

@ -201,6 +201,9 @@ public abstract class AbstractServiceConfig extends AbstractInterfaceConfig {
@SuppressWarnings({"unchecked"})
public void setProtocols(List<? extends ProtocolConfig> protocols) {
if (protocols == null) {
return;
}
for (ProtocolConfig protocolConfig : protocols) {
if (StringUtils.isEmpty(protocolConfig.getName())) {
protocolConfig.setName(Constants.DUBBO_VERSION_KEY);

3
dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/ReferenceConfig.java
View File

@ -534,6 +534,9 @@ public class ReferenceConfig<T> extends AbstractReferenceConfig {
}
public void setConsumer(ConsumerConfig consumer) {
if (consumer == null) {
return;
}
this.consumer = consumer;
ConfigManager.getInstance().addConsumer(consumer);
}

3
dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/ServiceConfig.java
View File

@ -920,6 +920,9 @@ public class ServiceConfig<T> extends AbstractServiceConfig {
}
public void setProvider(ProviderConfig provider) {
if (provider == null) {
return;
}
provider.refresh();
this.provider = provider;
ConfigManager.getInstance().addProvider(provider);