Fix the primitives for Methref (fixes #777)

jodd-proxetta/src/main/java/jodd/methref/InterfaceImplementation.java

@@ -25,8 +25,6 @@
 
 package jodd.methref;
 
-import java.lang.reflect.InvocationHandler;
-import java.lang.reflect.Method;
 import java.lang.reflect.Proxy;
 
 public class InterfaceImplementation<C> {
@@ -42,18 +40,11 @@ public static <T> InterfaceImplementation<T> of(final Class<T> target) {
 
 	@SuppressWarnings("unchecked")
 	public C createInstanceFor(final Methref m) {
-		return (C) Proxy.newProxyInstance(target.getClassLoader(), new Class[]{target}, new InvocationHandler() {
-			final Methref methref = m;
-			@Override
-			public Object invoke(final Object proxy, final Method method, final Object[] args) {
-				final String method_name = method.getName();
-				m.lastName(method_name);
-				if (method.getReturnType().isPrimitive()) {
-					return 0;
-				}
-				return null;
-			}
-		});
+		return (C) Proxy.newProxyInstance(
+				target.getClassLoader(),
+				new Class[]{target},
+				new InterfaceMethodInvocationHandler(m)
+		);
 	}
 
 }

jodd-proxetta/src/main/java/jodd/methref/InterfaceMethodInvocationHandler.java

@@ -0,0 +1,27 @@
+package jodd.methref;
+
+import java.lang.reflect.InvocationHandler;
+import java.lang.reflect.Method;
+
+@SuppressWarnings("rawtypes")
+public class InterfaceMethodInvocationHandler implements InvocationHandler {
+	private final Methref methref;
+
+	public InterfaceMethodInvocationHandler(final Methref methref) {
+		this.methref = methref;
+	}
+
+	@Override
+	public Object invoke(final Object proxy, final Method method, final Object[] args) {
+		final String method_name = method.getName();
+		methref.lastName(method_name);
+		if (method.getReturnType().isPrimitive()) {
+			final Class primitiveReturnType = method.getReturnType();
+			if (primitiveReturnType.equals(boolean.class)) {
+				return false;
+			}
+			return 0;
+		}
+		return null;
+	}
+}

jodd-proxetta/src/test/java/jodd/methref/InterfaceMethodInvocationHandlerTest.java

@@ -0,0 +1,39 @@
+package jodd.methref;
+
+import jodd.util.ClassUtil;
+import org.junit.jupiter.api.Test;
+
+import java.lang.reflect.Method;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+class InterfaceMethodInvocationHandlerTest {
+	interface Foo {
+		boolean b();
+		int i();
+		double d();
+		char c();
+		float f();
+	}
+
+	@Test
+	void testPrimitives() {
+		final Methref methref = new Methref(this.getClass());
+		final InterfaceMethodInvocationHandler imih = new InterfaceMethodInvocationHandler(methref);
+
+		Method m = ClassUtil.findMethod(Foo.class, "b");
+		assertEquals(false, imih.invoke(null, m, null));
+
+		m = ClassUtil.findMethod(Foo.class, "i");
+		assertEquals(0, imih.invoke(null, m, null));
+
+		m = ClassUtil.findMethod(Foo.class, "d");
+		assertEquals(0, imih.invoke(null, m, null));
+
+		m = ClassUtil.findMethod(Foo.class, "c");
+		assertEquals(0, imih.invoke(null, m, null));
+
+		m = ClassUtil.findMethod(Foo.class, "f");
+		assertEquals(0, imih.invoke(null, m, null));
+	}
+}