You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dubbo.apache.org by li...@apache.org on 2018/11/15 09:59:33 UTC
[incubator-dubbo] 02/02: ignore parameter(exclude=true).
This is an automated email from the ASF dual-hosted git repository.
liujun pushed a commit to branch dev-metadata
in repository https://gitbox.apache.org/repos/asf/incubator-dubbo.git
commit 453e323b4b955505b9c3ef10340f55ab8a3ca4a0
Author: ken.lj <ke...@gmail.com>
AuthorDate: Thu Nov 15 17:58:54 2018 +0800
ignore parameter(exclude=true).
---
.../src/main/java/org/apache/dubbo/config/AbstractConfig.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/AbstractConfig.java b/dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/AbstractConfig.java
index 130aa03..0be4c41 100644
--- a/dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/AbstractConfig.java
+++ b/dubbo-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/AbstractConfig.java
@@ -459,7 +459,9 @@ public abstract class AbstractConfig implements Serializable {
} else {
key = prop;
}
- if (method.getReturnType() == Object.class || parameter != null && parameter.excluded()) {
+ // treat url and configuration differently, the value should always present in configuration though it may not need to present in url.
+ //if (method.getReturnType() == Object.class || parameter != null && parameter.excluded()) {
+ if (method.getReturnType() == Object.class) {
metaData.put(key, null);
continue;
}