Fixed issue where invokeWriteMethod would attempt to apply a null value on a primitive value. Now checks for primitives and applies the default value instead of null.

slimjim2234 · slimjim2234 · commit e61bdb197936 · 2015-08-14T08:15:03.000-04:00
diff --git a/src/main/java/de/danielbechler/diff/introspection/PropertyAccessor.java b/src/main/java/de/danielbechler/diff/introspection/PropertyAccessor.java
@@ -16,6 +16,7 @@
 
 package de.danielbechler.diff.introspection;
 
+import com.google.common.base.Defaults;
 import de.danielbechler.diff.access.PropertyAwareAccessor;
 import de.danielbechler.diff.selector.BeanPropertyElementSelector;
 import de.danielbechler.util.Assert;
@@ -227,7 +228,9 @@ private void invokeWriteMethod(final Object target, final Object value)
 	{
 		try
 		{
-			writeMethod.invoke(target, value);
+            if (writeMethod.getParameterTypes()[0].isPrimitive() && value == null) {
+                writeMethod.invoke(target, Defaults.defaultValue(writeMethod.getParameterTypes()[0]));
+            } else writeMethod.invoke(target, value);
 		}
 		catch (final Exception cause)
 		{

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,7 @@`
`16`	`16`
`17`	`17`	`package de.danielbechler.diff.introspection;`
`18`	`18`
	`19`	`+import com.google.common.base.Defaults;`
`19`	`20`	`import de.danielbechler.diff.access.PropertyAwareAccessor;`
`20`	`21`	`import de.danielbechler.diff.selector.BeanPropertyElementSelector;`
`21`	`22`	`import de.danielbechler.util.Assert;`
`@@ -227,7 +228,9 @@ private void invokeWriteMethod(final Object target, final Object value)`
`227`	`228`	`{`
`228`	`229`	`try`
`229`	`230`	`{`
`230`		`- writeMethod.invoke(target, value);`
	`231`	`+ if (writeMethod.getParameterTypes()[0].isPrimitive() && value == null) {`
	`232`	`+ writeMethod.invoke(target, Defaults.defaultValue(writeMethod.getParameterTypes()[0]));`
	`233`	`+ } else writeMethod.invoke(target, value);`
`231`	`234`	`}`
`232`	`235`	`catch (final Exception cause)`
`233`	`236`	`{`