diff --git a/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java b/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java index e315ed20f..5e512d39d 100644 --- a/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java +++ b/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java @@ -342,7 +342,7 @@ public static String toString( * whether to include transient fields * @param outputStatics * whether to include static fields - * @param excludeNulls + * @param excludeNullValues * whether to exclude fields whose values are null * @param reflectUpToClass * the superclass to reflect up to (inclusive), may be null @@ -351,12 +351,12 @@ public static String toString( * if the Object is null * * @see ToStringExclude - * @since 2.1 + * @since 3.6 */ public static String toString( final T object, final ToStringStyle style, final boolean outputTransients, - final boolean outputStatics, boolean excludeNulls, final Class reflectUpToClass) { - return new ReflectionToStringBuilder(object, style, null, reflectUpToClass, outputTransients, outputStatics, excludeNulls) + final boolean outputStatics, boolean excludeNullValues, final Class reflectUpToClass) { + return new ReflectionToStringBuilder(object, style, null, reflectUpToClass, outputTransients, outputStatics, excludeNullValues) .toString(); } @@ -565,7 +565,7 @@ public ReflectionToStringBuilder( * whether to include static fields * @param excludeNullValues * whether to exclude fields who value is null - * @since 2.1 + * @since 3.6 */ public ReflectionToStringBuilder( final T object, final ToStringStyle style, final StringBuffer buffer, @@ -641,7 +641,7 @@ protected void appendFieldsIn(final Class clazz) { // Warning: Field.get(Object) creates wrappers objects // for primitive types. final Object fieldValue = this.getValue(field); - if(!excludeNullValues || fieldValue != null){ + if (!excludeNullValues || fieldValue != null) { this.append(fieldName, fieldValue); } } catch (final IllegalAccessException ex) { @@ -722,6 +722,7 @@ public boolean isAppendTransients() { *

* * @return Whether or not to append fields whose values are null. + * @since 3.6 */ public boolean isExcludeNullValues() { return this.excludeNullValues; @@ -773,6 +774,7 @@ public void setAppendTransients(final boolean appendTransients) { * * @param excludeNullValues * Whether or not to append fields whose values are null. + * @since 3.6 */ public void setExcludeNullValues(final boolean excludeNullValues) { this.excludeNullValues = excludeNullValues; diff --git a/src/test/java/org/apache/commons/lang3/builder/ReflectionToStringBuilderExcludeNullValuesTest.java b/src/test/java/org/apache/commons/lang3/builder/ReflectionToStringBuilderExcludeNullValuesTest.java index 0a91609c5..dd94c51a7 100644 --- a/src/test/java/org/apache/commons/lang3/builder/ReflectionToStringBuilderExcludeNullValuesTest.java +++ b/src/test/java/org/apache/commons/lang3/builder/ReflectionToStringBuilderExcludeNullValuesTest.java @@ -17,17 +17,20 @@ package org.apache.commons.lang3.builder; -import static org.junit.Assert.*; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertTrue; import org.junit.Test; public class ReflectionToStringBuilderExcludeNullValuesTest { - class TestFixture{ + static class TestFixture { + @SuppressWarnings("unused") private Integer testIntegerField; + @SuppressWarnings("unused") private String testStringField; - - public TestFixture(Integer a, String b){ + + public TestFixture(Integer a, String b) { this.testIntegerField = a; this.testStringField = b; }