[jOOQ/jOOQ#10493] Add TemplatingException for errors that occur with plain SQL templating

This commit is contained in:
Lukas Eder 2020-08-17 09:15:24 +02:00
parent 061e425340
commit 18957d5bee
2 changed files with 72 additions and 3 deletions

View File

@ -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.
* <p>
* 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);
}
}

View File

@ -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)));
}