diff --git a/gestalt-core/src/main/java/org/github/gestalt/config/builder/GestaltBuilder.java b/gestalt-core/src/main/java/org/github/gestalt/config/builder/GestaltBuilder.java index 1a75851d2..34abb98c8 100644 --- a/gestalt-core/src/main/java/org/github/gestalt/config/builder/GestaltBuilder.java +++ b/gestalt-core/src/main/java/org/github/gestalt/config/builder/GestaltBuilder.java @@ -608,7 +608,7 @@ public GestaltBuilder setTreatMissingDiscretionaryValuesAsErrors(boolean treatMi * @param treatNullValuesInClassAsErrors treat null values in classes after decoding as errors * @return GestaltBuilder builder * @deprecated This value is no longer used, Please use {@link #setTreatMissingDiscretionaryValuesAsErrors(boolean)} - * and {@link #setTreatMissingValuesAsErrors(Boolean)} + * and {@link #setTreatMissingValuesAsErrors(Boolean)} */ @Deprecated(since = "0.25.0", forRemoval = true) public GestaltBuilder setTreatNullValuesInClassAsErrors(Boolean treatNullValuesInClassAsErrors) { diff --git a/gestalt-core/src/main/java/org/github/gestalt/config/decoder/ProxyDecoder.java b/gestalt-core/src/main/java/org/github/gestalt/config/decoder/ProxyDecoder.java index 42d273305..0e711ec79 100644 --- a/gestalt-core/src/main/java/org/github/gestalt/config/decoder/ProxyDecoder.java +++ b/gestalt-core/src/main/java/org/github/gestalt/config/decoder/ProxyDecoder.java @@ -228,9 +228,10 @@ public Object invoke(Object proxy, Method method, Object[] args) throws Throwabl //noinspection DataFlowIssue assigning null for ease of reading here gestaltResult = null; } - } else + } else { throw new GestaltException("Failed to get pass through object from proxy config while calling method: " + methodName + " with type: " + returnType + " in path: " + path + "."); + } } return gestaltResult; @@ -301,7 +302,8 @@ static class ProxyCacheInvocationHandler extends ProxyPassThroughInvocationHandl private final Map methodResults; - private ProxyCacheInvocationHandler(String path, Tags tags, DecoderContext decoderContext, GestaltConfig config, Map methodResults) { + private ProxyCacheInvocationHandler(String path, Tags tags, DecoderContext decoderContext, GestaltConfig config, + Map methodResults) { super(path, tags, decoderContext, config); this.methodResults = methodResults; } diff --git a/gestalt-core/src/main/java/org/github/gestalt/config/entity/ValidationError.java b/gestalt-core/src/main/java/org/github/gestalt/config/entity/ValidationError.java index e35192079..32981011b 100644 --- a/gestalt-core/src/main/java/org/github/gestalt/config/entity/ValidationError.java +++ b/gestalt-core/src/main/java/org/github/gestalt/config/entity/ValidationError.java @@ -423,9 +423,6 @@ public OptionalMissingValueDecoding(String path, ConfigNode node, String decoder public String description() { StringBuilder description = new StringBuilder(62); description.append("Missing Optional Value while decoding ").append(decoder).append(" on path: ").append(path); - if (node != null) { - description.append(", from node: ").append(node.printer(path)); - } if (className != null) { description.append(", with class: ").append(className); } diff --git a/gestalt-core/src/main/java/org/github/gestalt/config/utils/ClassUtils.java b/gestalt-core/src/main/java/org/github/gestalt/config/utils/ClassUtils.java index 788270997..58725e6ce 100644 --- a/gestalt-core/src/main/java/org/github/gestalt/config/utils/ClassUtils.java +++ b/gestalt-core/src/main/java/org/github/gestalt/config/utils/ClassUtils.java @@ -251,7 +251,7 @@ public static Class wrapperToPrimitive(final Class cls) { * * @param type The class to query or null. * @return true if the given {@code type} is a primitive or primitive wrapper ({@link Boolean}, {@link Byte}, - * {@link Character}, {@link Short}, {@link Integer}, {@link Long}, {@link Double}, {@link Float}). + * {@link Character}, {@link Short}, {@link Integer}, {@link Long}, {@link Double}, {@link Float}). * @since 3.1 */ public static boolean isPrimitiveOrWrapper(final Class type) { @@ -267,7 +267,7 @@ public static boolean isPrimitiveOrWrapper(final Class type) { * * @param type The class to query or null. * @return true if the given {@code type} is a primitive wrapper ({@link Boolean}, {@link Byte}, {@link Character}, - * {@link Short}, {@link Integer}, {@link Long}, {@link Double}, {@link Float}). + * {@link Short}, {@link Integer}, {@link Long}, {@link Double}, {@link Float}). * @since 3.1 */ public static boolean isPrimitiveWrapper(final Class type) {