github/jooq
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[#7953] [#8496] Search the hierarchy for a matching setter

Browse Source
This commit is contained in:
lukaseder 2019-04-09 12:33:43 +02:00
parent af286d96b7
commit baeae9a24b
1 changed files with 27 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
jOOQ/src/main/java/org/jooq/impl/Tools.java
View File

@ -2044,7 +2044,7 @@ final class Tools {
// [#4182] MySQL also supports # as a comment character, and requires
// -- to be followed by a whitespace, although the latter is also not
// handled correctly by the MySQL JDBC driver (yet). See
// http://bugs.mysql.com/bug.php?id=76623
// http://bugs.mysql.com/bug.php?id=76623
(mysql && peek(sqlChars, i, TOKEN_HASH))) {
// Consume the complete comment
@ -3335,7 +3335,9 @@ final class Tools {
if (suffix != null) {
try {
Method setter = type.getDeclaredMethod("set" + suffix, method.getReturnType());
// [#7953] [#8496] Search the hierarchy for a matching setter
Method setter = getInstanceMethod(type, "set" + suffix, new Class[] { method.getReturnType() });
// Setter annotation is more relevant
if (setter.getAnnotation(Column.class) == null)
@ -3528,6 +3530,29 @@ final class Tools {
}
}
private static final Method getInstanceMethod(Class<?> type, String name, Class<?>[] parameters) throws NoSuchMethodException {
// first priority: find a public method with exact signature match in class hierarchy
try {
return type.getMethod(name, parameters);
}
// second priority: find a private method with exact signature match on declaring class
catch (NoSuchMethodException e) {
do {
try {
return type.getDeclaredMethod(name, parameters);
}
catch (NoSuchMethodException ignore) {}
type = type.getSuperclass();
}
while (type != null);
throw new NoSuchMethodException();
}
}
/**
* All the public and declared methods of a type.
* <p>