Use clearer condition.

harawata · harawata · commit 9ce0fcdce2ec · 2018-01-16T03:13:58.000+09:00
diff --git a/src/main/java/org/apache/ibatis/binding/MapperMethod.java b/src/main/java/org/apache/ibatis/binding/MapperMethod.java
@@ -85,7 +85,7 @@ public Object execute(SqlSession sqlSession, Object[] args) {
           Object param = method.convertArgsToSqlCommandParam(args);
           result = sqlSession.selectOne(command.getName(), param);
           if (method.returnsOptional() &&
-              (result == null || !method.getReturnType().equals(result.getClass()))) {
+              (result == null || !Optional.class.equals(result.getClass()))) {
             result = OptionalUtil.ofNullable(result);
           }
         }

Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ public Object execute(SqlSession sqlSession, Object[] args) {`
`85`	`85`	`Object param = method.convertArgsToSqlCommandParam(args);`
`86`	`86`	`result = sqlSession.selectOne(command.getName(), param);`
`87`	`87`	`if (method.returnsOptional() &&`
`88`		`- (result == null \|\| !method.getReturnType().equals(result.getClass()))) {`
	`88`	`+ (result == null \|\| !Optional.class.equals(result.getClass()))) {`
`89`	`89`	`result = OptionalUtil.ofNullable(result);`
`90`	`90`	`}`
`91`	`91`	`}`