Merge pull request #359 from jeffgbutler/master

Checkstyle and Spotbugs Updates
mybatis · May 6, 2021 · 82c4fdf · 82c4fdf
2 parents 4eeb56f + a95d5f1
commit 82c4fdf
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 22 deletions.
diff --git a/src/main/java/org/mybatis/dynamic/sql/util/SqlProviderAdapter.java b/src/main/java/org/mybatis/dynamic/sql/util/SqlProviderAdapter.java
@@ -80,8 +80,8 @@ public String insertMultipleWithGeneratedKeys(Map<String, Object> parameterMap)
         if (entries.size() == 1) {
             return entries.get(0);
         } else {
-            throw new IllegalArgumentException("The parameters for insertMultipleWithGeneratedKeys" + //$NON-NLS-1$
-                    " must contain exactly one parameter of type String"); //$NON-NLS-1$
+            throw new IllegalArgumentException("The parameters for insertMultipleWithGeneratedKeys" //$NON-NLS-1$
+                    + " must contain exactly one parameter of type String"); //$NON-NLS-1$
         }
     }
 

diff --git a/src/main/java/org/mybatis/dynamic/sql/where/condition/IsInCaseInsensitive.java b/src/main/java/org/mybatis/dynamic/sql/where/condition/IsInCaseInsensitive.java
@@ -44,8 +44,9 @@ protected  IsInCaseInsensitive(Collection<String> values, Callback emptyCallback
 
     @Override
     public String renderCondition(String columnName, Stream<String> placeholders) {
-        return "upper(" + columnName + ") " + //$NON-NLS-1$ //$NON-NLS-2$
-                placeholders.collect(Collectors.joining(",", "in (", ")")); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
+        return "upper(" + columnName + ") " //$NON-NLS-1$ //$NON-NLS-2$
+                + placeholders.collect(
+                        Collectors.joining(",", "in (", ")")); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
     }
 
     @Override

diff --git a/src/main/java/org/mybatis/dynamic/sql/where/condition/IsNotInCaseInsensitive.java b/src/main/java/org/mybatis/dynamic/sql/where/condition/IsNotInCaseInsensitive.java
@@ -44,8 +44,8 @@ protected IsNotInCaseInsensitive(Collection<String> values, Callback emptyCallba
 
     @Override
     public String renderCondition(String columnName, Stream<String> placeholders) {
-        return "upper(" + columnName + ") " + //$NON-NLS-1$ //$NON-NLS-2$
-                placeholders.collect(
+        return "upper(" + columnName + ") " //$NON-NLS-1$ //$NON-NLS-2$
+                + placeholders.collect(
                         Collectors.joining(",", "not in (", ")")); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
     }
 

diff --git a/src/main/kotlin/org/mybatis/dynamic/sql/util/kotlin/KotlinBaseBuilders.kt b/src/main/kotlin/org/mybatis/dynamic/sql/util/kotlin/KotlinBaseBuilders.kt
@@ -105,18 +105,18 @@ abstract class KotlinBaseBuilder<D : AbstractWhereSupport<*>, B : KotlinBaseBuil
 
     fun allRows(): B = self()
 
-    private fun applyToWhere(block: AbstractWhereDSL<*>.() -> Unit): B =
-        self().also {
-            getDsl().where().apply(block)
-        }
+    private fun applyToWhere(block: AbstractWhereDSL<*>.() -> Unit): B {
+        getDsl().where().apply(block)
+        return self()
+    }
 
     private fun applyToWhere(
         subCriteria: CriteriaReceiver,
         block: AbstractWhereDSL<*>.(List<SqlCriterion>) -> Unit
-    ): B =
-        self().also {
-            getDsl().where().block(CriteriaCollector().apply(subCriteria).criteria)
-        }
+    ): B {
+        getDsl().where().block(CriteriaCollector().apply(subCriteria).criteria)
+        return self()
+    }
 
     protected abstract fun self(): B
 
@@ -199,17 +199,17 @@ abstract class KotlinBaseJoiningBuilder<D : AbstractQueryExpressionDSL<*, *>, B
             rightJoin(sq, sq.correlationName, jc.onJoinCriterion, jc.andJoinCriteria)
         }
 
-    private fun applyToDsl(joinCriteria: JoinReceiver, applyJoin: D.(JoinCollector) -> Unit): B =
-        self().also {
-            getDsl().applyJoin(JoinCollector().apply(joinCriteria))
-        }
+    private fun applyToDsl(joinCriteria: JoinReceiver, applyJoin: D.(JoinCollector) -> Unit): B {
+        getDsl().applyJoin(JoinCollector().apply(joinCriteria))
+        return self()
+    }
 
     private fun applyToDsl(
         subQuery: KotlinQualifiedSubQueryBuilder.() -> Unit,
         joinCriteria: JoinReceiver,
         applyJoin: D.(KotlinQualifiedSubQueryBuilder, JoinCollector) -> Unit
-    ): B =
-        self().also {
-            getDsl().applyJoin(KotlinQualifiedSubQueryBuilder().apply(subQuery), JoinCollector().apply(joinCriteria))
-        }
+    ): B {
+        getDsl().applyJoin(KotlinQualifiedSubQueryBuilder().apply(subQuery), JoinCollector().apply(joinCriteria))
+        return self()
+    }
 }