diff --git a/jOOQ/src/main/java/org/jooq/exception/TemplatingException.java b/jOOQ/src/main/java/org/jooq/exception/TemplatingException.java new file mode 100644 index 0000000000..98616dc3ce --- /dev/null +++ b/jOOQ/src/main/java/org/jooq/exception/TemplatingException.java @@ -0,0 +1,65 @@ +/* + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + * + * Other licenses: + * ----------------------------------------------------------------------------- + * Commercial licenses for this work are available. These replace the above + * ASL 2.0 and offer limited warranties, support, maintenance, and commercial + * database integrations. + * + * For more information, please visit: http://www.jooq.org/licenses + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + */ +package org.jooq.exception; + +import org.jooq.QueryPart; +import org.jooq.SQL; + +/** + * An exception that occurs with plain {@link SQL} templating. + *
+ * Possible exceptions occur when the number of placeholders and the number of + * provided {@link QueryPart} objects mismatch. + * + * @author Lukas Eder + */ +public class TemplatingException extends DataAccessException { + + /** + * Generated UID + */ + private static final long serialVersionUID = -3984254040787669866L; + + public TemplatingException(String message) { + super(message); + } + + public TemplatingException(String message, Throwable throwable) { + super(message, throwable); + } +} diff --git a/jOOQ/src/main/java/org/jooq/impl/Tools.java b/jOOQ/src/main/java/org/jooq/impl/Tools.java index 530962f7f9..6e347c9234 100644 --- a/jOOQ/src/main/java/org/jooq/impl/Tools.java +++ b/jOOQ/src/main/java/org/jooq/impl/Tools.java @@ -274,6 +274,7 @@ import org.jooq.exception.DataAccessException; import org.jooq.exception.DataTypeException; import org.jooq.exception.MappingException; import org.jooq.exception.NoDataFoundException; +import org.jooq.exception.TemplatingException; import org.jooq.exception.TooManyRowsException; import org.jooq.impl.ResultsImpl.ResultOrRowsImpl; import org.jooq.tools.Ints; @@ -2592,13 +2593,16 @@ final class Tools { // Try getting the {numbered placeholder} Integer index = Ints.tryParse(sql, start, end); if (index != null) { + if (index < 0 || index >= substitutes.size()) + throw new TemplatingException("No substitute QueryPart provided for placeholder {" + index + "} in plain SQL template: " + sql); + QueryPart substitute = substitutes.get(index); render.visit(substitute); - if (bind != null) { + if (bind != null) bind.visit(substitute); - } - } else { + } + else { // Then we're dealing with a {keyword} render.visit(DSL.keyword(sql.substring(start, end))); }