Deprecate the "empty list" methods

The new statement configuration functionality is better and more flexible

Author: jeffgbutler

src/main/java/org/mybatis/dynamic/sql/AbstractListValueCondition.java

@@ -1,5 +1,5 @@
 /*
- *    Copyright 2016-2021 the original author or authors.
+ *    Copyright 2016-2022 the original author or authors.
  *
  *    Licensed under the Apache License, Version 2.0 (the "License");
  *    you may not use this file except in compliance with the License.
@@ -26,12 +26,23 @@
 
 public abstract class AbstractListValueCondition<T> implements VisitableCondition<T> {
     protected final Collection<T> values;
+
+    /**
+     * @deprecated in favor of the statement configuration functions
+     */
+    @Deprecated
     protected final Callback emptyCallback;
 
     protected AbstractListValueCondition(Collection<T> values) {
         this(values, () -> { });
     }
 
+    /**
+     * @deprecated in favor of the statement configuration functions
+     * @param values values
+     * @param emptyCallback empty callback
+     */
+    @Deprecated
     protected AbstractListValueCondition(Collection<T> values, Callback emptyCallback) {
         this.values = Objects.requireNonNull(values);
         this.emptyCallback = Objects.requireNonNull(emptyCallback);
@@ -96,6 +107,14 @@ protected <R, S extends AbstractListValueCondition<R>> S mapSupport(Function<? s
      */
     public abstract AbstractListValueCondition<T> filter(Predicate<? super T> predicate);
 
+    /**
+     * Specifies a callback function to be called if the value list is empty when rendered.
+     *
+     * @deprecated in favor of the statement configuration functions
+     * @param callback a callback function - typically throws an exception to block the statement from executing
+     * @return this condition
+     */
+    @Deprecated
     public abstract AbstractListValueCondition<T> withListEmptyCallback(Callback callback);
 
     public abstract String renderCondition(String columnName, Stream<String> placeholders);

src/main/java/org/mybatis/dynamic/sql/where/condition/IsIn.java

@@ -38,6 +38,11 @@ public static <T> IsIn<T> empty() {
         return t;
     }
 
+    /**
+     * @deprecated in favor of the statement configuration functions
+     * @return a new empty condition
+     */
+    @Deprecated
     private <S> IsIn<S> emptyWithCallBack() {
         return new IsIn<>(Collections.emptyList(), emptyCallback);
     }
@@ -46,6 +51,12 @@ protected IsIn(Collection<T> values) {
         super(values);
     }
 
+    /**
+     * @deprecated in favor of the statement configuration functions
+     * @param values values
+     * @param emptyCallback empty callback
+     */
+    @Deprecated
     protected IsIn(Collection<T> values, Callback emptyCallback) {
         super(values, emptyCallback);
     }
@@ -56,6 +67,12 @@ public String renderCondition(String columnName, Stream<String> placeholders) {
                 + placeholders.collect(Collectors.joining(",", "in (", ")")); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
     }
 
+    /**
+     * @deprecated in favor of the statement configuration functions
+     * @param callback a callback function - typically throws an exception to block the statement from executing
+     * @return this condition
+     */
+    @Deprecated
     @Override
     public IsIn<T> withListEmptyCallback(Callback callback) {
         return new IsIn<>(values, callback);

src/main/java/org/mybatis/dynamic/sql/where/condition/IsInCaseInsensitive.java

@@ -34,6 +34,11 @@ public static IsInCaseInsensitive empty() {
         return EMPTY;
     }
 
+    /**
+     * @deprecated in favor of the statement configuration functions
+     * @return a new empty condition
+     */
+    @Deprecated
     private IsInCaseInsensitive emptyWithCallback() {
         return new IsInCaseInsensitive(Collections.emptyList(), emptyCallback);
     }
@@ -42,6 +47,12 @@ protected  IsInCaseInsensitive(Collection<String> values) {
         super(values);
     }
 
+    /**
+     * @deprecated in favor of the statement configuration functions
+     * @param values values
+     * @param emptyCallback empty callback
+     */
+    @Deprecated
     protected  IsInCaseInsensitive(Collection<String> values, Callback emptyCallback) {
         super(values, emptyCallback);
     }
@@ -53,6 +64,12 @@ public String renderCondition(String columnName, Stream<String> placeholders) {
                         Collectors.joining(",", "in (", ")")); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
     }
 
+    /**
+     * @deprecated in favor of the statement configuration functions
+     * @param callback a callback function - typically throws an exception to block the statement from executing
+     * @return this condition
+     */
+    @Deprecated
     @Override
     public IsInCaseInsensitive withListEmptyCallback(Callback callback) {
         return new IsInCaseInsensitive(values, callback);

src/main/java/org/mybatis/dynamic/sql/where/condition/IsNotIn.java

@@ -38,6 +38,11 @@ public static <T> IsNotIn<T> empty() {
         return t;
     }
 
+    /**
+     * @deprecated in favor of the statement configuration functions
+     * @return a new empty condition
+     */
+    @Deprecated
     private <S> IsNotIn<S> emptyWithCallback() {
         return new IsNotIn<>(Collections.emptyList(), emptyCallback);
     }
@@ -46,6 +51,12 @@ protected IsNotIn(Collection<T> values) {
         super(values);
     }
 
+    /**
+     * @deprecated in favor of the statement configuration functions
+     * @param values values
+     * @param emptyCallback empty callback
+     */
+    @Deprecated
     protected IsNotIn(Collection<T> values, Callback emptyCallback) {
         super(values, emptyCallback);
     }
@@ -57,6 +68,13 @@ public String renderCondition(String columnName, Stream<String> placeholders) {
                         Collectors.joining(",", "not in (", ")")); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
     }
 
+    /**
+     *
+     * @deprecated in favor of the statement configuration functions
+     * @param callback a callback function - typically throws an exception to block the statement from executing
+     * @return this condition
+     */
+    @Deprecated
     @Override
     public IsNotIn<T> withListEmptyCallback(Callback callback) {
         return new IsNotIn<>(values, callback);

src/main/java/org/mybatis/dynamic/sql/where/condition/IsNotInCaseInsensitive.java

@@ -34,6 +34,11 @@ public static IsNotInCaseInsensitive empty() {
         return EMPTY;
     }
 
+    /**
+     * @deprecated in favor of the statement configuration functions
+     * @return a new empty condition
+     */
+    @Deprecated
     private IsNotInCaseInsensitive emptyWithCallback() {
         return new IsNotInCaseInsensitive(Collections.emptyList(), emptyCallback);
     }
@@ -42,6 +47,12 @@ protected IsNotInCaseInsensitive(Collection<String> values) {
         super(values);
     }
 
+    /**
+     * @deprecated in favor of the statement configuration functions
+     * @param values values
+     * @param emptyCallback empty callback
+     */
+    @Deprecated
     protected IsNotInCaseInsensitive(Collection<String> values, Callback emptyCallback) {
         super(values, emptyCallback);
     }
@@ -53,6 +64,12 @@ public String renderCondition(String columnName, Stream<String> placeholders) {
                         Collectors.joining(",", "not in (", ")")); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
     }
 
+    /**
+     * @deprecated in favor of the statement configuration functions
+     * @param callback a callback function - typically throws an exception to block the statement from executing
+     * @return this condition
+     */
+    @Deprecated
     @Override
     public IsNotInCaseInsensitive withListEmptyCallback(Callback callback) {
         return new IsNotInCaseInsensitive(values, callback);