You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ac...@apache.org on 2018/02/17 10:57:56 UTC
[camel] branch master updated (18634f4 -> 543c3a2)
This is an automated email from the ASF dual-hosted git repository.
acosentino pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git.
from 18634f4 CAMEL-12275 - Renamed classes to be compliant with other classes name
new 2e0410f camel-jsonpath - It is not necessary to explicitly set default jsonProvider, as ConfigurationBuilder.build will do that anyway.
new 543c3a2 camel-jsonpath - Remove duplications.
The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.../java/org/apache/camel/jsonpath/JsonPathEngine.java | 18 ++++++------------
1 file changed, 6 insertions(+), 12 deletions(-)
--
To stop receiving notification emails like this one, please contact
acosentino@apache.org.
[camel] 02/02: camel-jsonpath - Remove duplications.
Posted by ac...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
acosentino pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git
commit 543c3a283dbead9b826844e13264d4b1055ab071
Author: yihtserns <si...@yahoo.com>
AuthorDate: Sat Feb 17 13:22:05 2018 +0800
camel-jsonpath - Remove duplications.
---
.../java/org/apache/camel/jsonpath/JsonPathEngine.java | 17 ++++++-----------
1 file changed, 6 insertions(+), 11 deletions(-)
diff --git a/components/camel-jsonpath/src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java b/components/camel-jsonpath/src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java
index def309c..41d5bdf 100644
--- a/components/camel-jsonpath/src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java
+++ b/components/camel-jsonpath/src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java
@@ -67,19 +67,14 @@ public class JsonPathEngine {
this.writeAsString = writeAsString;
this.headerName = headerName;
+ Configuration.ConfigurationBuilder builder = Configuration.builder();
if (options != null) {
- Configuration.ConfigurationBuilder builder = Configuration.builder().options(options);
- if (suppressExceptions) {
- builder.options(SUPPRESS_EXCEPTIONS);
- }
- this.configuration = builder.build();
- } else {
- Configuration.ConfigurationBuilder builder = Configuration.builder();
- if (suppressExceptions) {
- builder.options(SUPPRESS_EXCEPTIONS);
- }
- this.configuration = builder.build();
+ builder.options(options);
+ }
+ if (suppressExceptions) {
+ builder.options(SUPPRESS_EXCEPTIONS);
}
+ this.configuration = builder.build();
boolean hasSimple = false;
if (allowSimple) {
--
To stop receiving notification emails like this one, please contact
acosentino@apache.org.
[camel] 01/02: camel-jsonpath - It is not necessary to explicitly
set default jsonProvider, as ConfigurationBuilder.build will do that anyway.
Posted by ac...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
acosentino pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git
commit 2e0410fe26da48c90d165dab17bfc91c00b3d539
Author: yihtserns <si...@yahoo.com>
AuthorDate: Sat Feb 17 13:13:30 2018 +0800
camel-jsonpath - It is not necessary to explicitly set default jsonProvider, as ConfigurationBuilder.build will do that anyway.
---
.../src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/components/camel-jsonpath/src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java b/components/camel-jsonpath/src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java
index 90ff3e8..def309c 100644
--- a/components/camel-jsonpath/src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java
+++ b/components/camel-jsonpath/src/main/java/org/apache/camel/jsonpath/JsonPathEngine.java
@@ -67,15 +67,14 @@ public class JsonPathEngine {
this.writeAsString = writeAsString;
this.headerName = headerName;
- Configuration defaults = Configuration.defaultConfiguration();
if (options != null) {
- Configuration.ConfigurationBuilder builder = Configuration.builder().jsonProvider(defaults.jsonProvider()).options(options);
+ Configuration.ConfigurationBuilder builder = Configuration.builder().options(options);
if (suppressExceptions) {
builder.options(SUPPRESS_EXCEPTIONS);
}
this.configuration = builder.build();
} else {
- Configuration.ConfigurationBuilder builder = Configuration.builder().jsonProvider(defaults.jsonProvider());
+ Configuration.ConfigurationBuilder builder = Configuration.builder();
if (suppressExceptions) {
builder.options(SUPPRESS_EXCEPTIONS);
}
--
To stop receiving notification emails like this one, please contact
acosentino@apache.org.