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

[#7929] jOOQ-checker should not allow any jOOQ API usage in the absence of an Allow annotation

Browse Source
This commit is contained in:
lukaseder 2018-10-11 17:59:17 +02:00
parent e33e0e55ff
commit 5a958a8df9
1 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
jOOQ-checker/src/main/java/org/jooq/checker/SQLDialectChecker.java
View File

@ -74,12 +74,17 @@ public class SQLDialectChecker extends AbstractChecker {
ExecutableElement elementFromUse = elementFromUse(node);
Support support = elementFromUse.getAnnotation(Support.class);
// In the absence of a @Support annotation, or if no SQLDialect is supplied,
// all jOOQ API method calls will type check.
if (support != null && support.value().length > 0) {
// In the absence of a @Support annotation, all jOOQ API method calls will type check.
if (support != null) {
Element enclosing = enclosing(getPath(root, node));
EnumSet<SQLDialect> supported = EnumSet.copyOf(asList(support.value()));
// [#7929] "Empty" @Support annotations expand to all SQLDialects
EnumSet<SQLDialect> supported = EnumSet.copyOf(
support.value().length > 0
? asList(support.value())
: asList(SQLDialect.values())
);
EnumSet<SQLDialect> allowed = EnumSet.noneOf(SQLDialect.class);
EnumSet<SQLDialect> required = EnumSet.noneOf(SQLDialect.class);