From 8e9bfeaea58004903345d929b8721d713d763d0a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timo=20Westk=C3=A4mper?= Date: Sun, 18 Mar 2018 19:17:17 +0200 Subject: [PATCH 1/2] Fixes for release --- pom.xml | 16 ++++++++++++++++ .../querydsl/core/NonUniqueResultException.java | 4 ++++ 2 files changed, 20 insertions(+) diff --git a/pom.xml b/pom.xml index 61f3a192e..e7867b6a4 100644 --- a/pom.xml +++ b/pom.xml @@ -284,7 +284,23 @@ + com/querydsl/core/Fetchable com/querydsl/core/types/dsl/PathBuilderValidator + com/querydsl/collections/AbstractCollQuery + com/querydsl/hibernate/search/AbstractSearchQuery + com/querydsl/jdo/AbstractJDOQuery + com/querydsl/jdo/sql/AbstractSQLQuery + com/querydsl/jpa/hibernate/AbstractHibernateQuery + com/querydsl/jpa/JPASubQuery + com/querydsl/jpa/hibernate/sql/AbstractHibernateSQLQuery + com/querydsl/jpa/impl/AbstractJPAQuery + com/querydsl/jpa/sql/AbstractJPASQLQuery + com/querydsl/lucene3/AbstractLuceneQuery + com/querydsl/lucene4/AbstractLuceneQuery + com/querydsl/lucene5/AbstractLuceneQuery + com/querydsl/mongodb/AbstractMongodbQuery + com/querydsl/sql/ProjectableSQLQuery + com/querydsl/sql/UnionImpl com/querydsl/sql/Union com/querydsl/sql/SQLListenerContextImpl com/querydsl/sql/dml/SQLDeleteClause diff --git a/querydsl-core/src/main/java/com/querydsl/core/NonUniqueResultException.java b/querydsl-core/src/main/java/com/querydsl/core/NonUniqueResultException.java index 232109f59..39156f714 100644 --- a/querydsl-core/src/main/java/com/querydsl/core/NonUniqueResultException.java +++ b/querydsl-core/src/main/java/com/querydsl/core/NonUniqueResultException.java @@ -32,4 +32,8 @@ public class NonUniqueResultException extends QueryException { super(message); } + public NonUniqueResultException(Exception e) { + super(e); + } + } From 5f675400352c3100370046181a5646a5e0de1e1c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timo=20Westk=C3=A4mper?= Date: Sun, 18 Mar 2018 21:41:40 +0200 Subject: [PATCH 2/2] Add missing casts --- .../sql/dml/AbstractSQLDeleteClause.java | 12 +++---- .../sql/dml/AbstractSQLInsertClause.java | 22 ++++++------- .../sql/dml/AbstractSQLUpdateClause.java | 24 +++++++------- .../sql/mssql/AbstractSQLServerQuery.java | 2 +- .../sql/mysql/AbstractMySQLQuery.java | 32 +++++++++---------- .../sql/oracle/AbstractOracleQuery.java | 10 +++--- .../postgresql/AbstractPostgreSQLQuery.java | 8 ++--- 7 files changed, 55 insertions(+), 55 deletions(-) diff --git a/querydsl-sql/src/main/java/com/querydsl/sql/dml/AbstractSQLDeleteClause.java b/querydsl-sql/src/main/java/com/querydsl/sql/dml/AbstractSQLDeleteClause.java index 68bec72eb..e68a9aee0 100644 --- a/querydsl-sql/src/main/java/com/querydsl/sql/dml/AbstractSQLDeleteClause.java +++ b/querydsl-sql/src/main/java/com/querydsl/sql/dml/AbstractSQLDeleteClause.java @@ -84,7 +84,7 @@ public abstract class AbstractSQLDeleteClause flag) { metadata.addFlag(new QueryFlag(position, flag)); return (C) this; @@ -108,7 +108,7 @@ public abstract class AbstractSQLDeleteClause flag) { metadata.addFlag(new QueryFlag(position, flag)); return (C) this; @@ -122,7 +122,7 @@ public abstract class AbstractSQLInsertClause... columns) { this.columns.addAll(Arrays.asList(columns)); return (C) this; @@ -468,7 +468,7 @@ public abstract class AbstractSQLInsertClause sq) { subQuery = sq; for (Map.Entry, Object> entry : sq.getMetadata().getParams().entrySet()) { @@ -478,7 +478,7 @@ public abstract class AbstractSQLInsertClause C set(Path path, T value) { columns.add(path); if (value instanceof Expression) { @@ -492,7 +492,7 @@ public abstract class AbstractSQLInsertClause C set(Path path, Expression expression) { columns.add(path); values.add(expression); @@ -500,7 +500,7 @@ public abstract class AbstractSQLInsertClause C setNull(Path path) { columns.add(path); values.add(Null.CONSTANT); @@ -508,7 +508,7 @@ public abstract class AbstractSQLInsertClause) { @@ -540,7 +540,7 @@ public abstract class AbstractSQLInsertClause C populate(T obj, Mapper mapper) { Map, Object> values = mapper.createMap(entity, obj); for (Map.Entry, Object> entry : values.entrySet()) { diff --git a/querydsl-sql/src/main/java/com/querydsl/sql/dml/AbstractSQLUpdateClause.java b/querydsl-sql/src/main/java/com/querydsl/sql/dml/AbstractSQLUpdateClause.java index d24d98ecf..b4a3659ec 100644 --- a/querydsl-sql/src/main/java/com/querydsl/sql/dml/AbstractSQLUpdateClause.java +++ b/querydsl-sql/src/main/java/com/querydsl/sql/dml/AbstractSQLUpdateClause.java @@ -82,7 +82,7 @@ public abstract class AbstractSQLUpdateClause flag) { metadata.addFlag(new QueryFlag(position, flag)); return (C) this; @@ -106,7 +106,7 @@ public abstract class AbstractSQLUpdateClause C set(Path path, T value) { if (value instanceof Expression) { updates.put(path, (Expression) value); @@ -262,7 +262,7 @@ public abstract class AbstractSQLUpdateClause C set(Path path, Expression expression) { if (expression != null) { updates.put(path, expression); @@ -273,14 +273,14 @@ public abstract class AbstractSQLUpdateClause C setNull(Path path) { updates.put(path, Null.CONSTANT); return (C) this; } @Override - @WithBridgeMethods(SQLUpdateClause.class) + @WithBridgeMethods(value = SQLUpdateClause.class, castRequired = true) public C set(List> paths, List values) { for (int i = 0; i < paths.size(); i++) { if (values.get(i) instanceof Expression) { @@ -294,14 +294,14 @@ public abstract class AbstractSQLUpdateClause C populate(T obj, Mapper mapper) { Collection> primaryKeyColumns = entity.getPrimaryKey() != null ? entity.getPrimaryKey().getLocalColumns() diff --git a/querydsl-sql/src/main/java/com/querydsl/sql/mssql/AbstractSQLServerQuery.java b/querydsl-sql/src/main/java/com/querydsl/sql/mssql/AbstractSQLServerQuery.java index 6abe4aaad..d90b3f388 100644 --- a/querydsl-sql/src/main/java/com/querydsl/sql/mssql/AbstractSQLServerQuery.java +++ b/querydsl-sql/src/main/java/com/querydsl/sql/mssql/AbstractSQLServerQuery.java @@ -46,7 +46,7 @@ public abstract class AbstractSQLServerQuery 0) { String hints = SQLServerGrammar.tableHints(tableHints); diff --git a/querydsl-sql/src/main/java/com/querydsl/sql/mysql/AbstractMySQLQuery.java b/querydsl-sql/src/main/java/com/querydsl/sql/mysql/AbstractMySQLQuery.java index f22b18b68..bca612470 100644 --- a/querydsl-sql/src/main/java/com/querydsl/sql/mysql/AbstractMySQLQuery.java +++ b/querydsl-sql/src/main/java/com/querydsl/sql/mysql/AbstractMySQLQuery.java @@ -73,7 +73,7 @@ public abstract class AbstractMySQLQuery> * * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C bigResult() { return addFlag(Position.AFTER_SELECT, SQL_BIG_RESULT); } @@ -86,7 +86,7 @@ public abstract class AbstractMySQLQuery> * * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C bufferResult() { return addFlag(Position.AFTER_SELECT, SQL_BUFFER_RESULT); } @@ -97,7 +97,7 @@ public abstract class AbstractMySQLQuery> * * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C cache() { return addFlag(Position.AFTER_SELECT, SQL_CACHE); } @@ -108,7 +108,7 @@ public abstract class AbstractMySQLQuery> * * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C calcFoundRows() { return addFlag(Position.AFTER_SELECT, SQL_CALC_FOUND_ROWS); } @@ -119,7 +119,7 @@ public abstract class AbstractMySQLQuery> * * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C highPriority() { return addFlag(Position.AFTER_SELECT, HIGH_PRIORITY); } @@ -130,7 +130,7 @@ public abstract class AbstractMySQLQuery> * @param var variable name * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C into(String var) { return addFlag(Position.END, "\ninto " + var); } @@ -141,7 +141,7 @@ public abstract class AbstractMySQLQuery> * @param file file to write to * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C intoDumpfile(File file) { return addFlag(Position.END, "\ninto dumpfile '" + file.getPath() + "'"); } @@ -153,7 +153,7 @@ public abstract class AbstractMySQLQuery> * @param file file to write to * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C intoOutfile(File file) { return addFlag(Position.END, "\ninto outfile '" + file.getPath() + "'"); } @@ -164,7 +164,7 @@ public abstract class AbstractMySQLQuery> * * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C lockInShareMode() { return addFlag(Position.END, LOCK_IN_SHARE_MODE); } @@ -175,7 +175,7 @@ public abstract class AbstractMySQLQuery> * * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C noCache() { return addFlag(Position.AFTER_SELECT, SQL_NO_CACHE); } @@ -186,7 +186,7 @@ public abstract class AbstractMySQLQuery> * * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C smallResult() { return addFlag(Position.AFTER_SELECT, SQL_SMALL_RESULT); } @@ -198,7 +198,7 @@ public abstract class AbstractMySQLQuery> * * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C straightJoin() { return addFlag(Position.AFTER_SELECT, STRAIGHT_JOIN); } @@ -211,7 +211,7 @@ public abstract class AbstractMySQLQuery> * @param indexes index names * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C forceIndex(String... indexes) { return addJoinFlag(" force index (" + JOINER.join(indexes) + ")", JoinFlag.Position.END); } @@ -223,7 +223,7 @@ public abstract class AbstractMySQLQuery> * @param indexes index names * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C ignoreIndex(String... indexes) { return addJoinFlag(" ignore index (" + JOINER.join(indexes) + ")", JoinFlag.Position.END); } @@ -235,7 +235,7 @@ public abstract class AbstractMySQLQuery> * @param indexes index names * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C useIndex(String... indexes) { return addJoinFlag(" use index (" + JOINER.join(indexes) + ")", JoinFlag.Position.END); } @@ -248,7 +248,7 @@ public abstract class AbstractMySQLQuery> * * @return the current object */ - @WithBridgeMethods(MySQLQuery.class) + @WithBridgeMethods(value = MySQLQuery.class, castRequired = true) public C withRollup() { return addFlag(Position.AFTER_GROUP_BY, WITH_ROLLUP); } diff --git a/querydsl-sql/src/main/java/com/querydsl/sql/oracle/AbstractOracleQuery.java b/querydsl-sql/src/main/java/com/querydsl/sql/oracle/AbstractOracleQuery.java index 782724083..d137a82f0 100644 --- a/querydsl-sql/src/main/java/com/querydsl/sql/oracle/AbstractOracleQuery.java +++ b/querydsl-sql/src/main/java/com/querydsl/sql/oracle/AbstractOracleQuery.java @@ -58,7 +58,7 @@ public abstract class AbstractOracleQuery * @param cond condition * @return the current object */ - @WithBridgeMethods(OracleQuery.class) + @WithBridgeMethods(value = OracleQuery.class, castRequired = true) public C connectByPrior(Predicate cond) { return addFlag(Position.BEFORE_ORDER, CONNECT_BY_PRIOR, cond); } @@ -69,7 +69,7 @@ public abstract class AbstractOracleQuery * @param cond condition * @return the current object */ - @WithBridgeMethods(OracleQuery.class) + @WithBridgeMethods(value = OracleQuery.class, castRequired = true) public C connectBy(Predicate cond) { return addFlag(Position.BEFORE_ORDER, CONNECT_BY, cond); } @@ -80,7 +80,7 @@ public abstract class AbstractOracleQuery * @param cond condition * @return the current object */ - @WithBridgeMethods(OracleQuery.class) + @WithBridgeMethods(value = OracleQuery.class, castRequired = true) public C connectByNocyclePrior(Predicate cond) { return addFlag(Position.BEFORE_ORDER, CONNECT_BY_NOCYCLE_PRIOR, cond); } @@ -91,7 +91,7 @@ public abstract class AbstractOracleQuery * @param cond condition * @return the current object */ - @WithBridgeMethods(OracleQuery.class) + @WithBridgeMethods(value = OracleQuery.class, castRequired = true) public C startWith(Predicate cond) { return addFlag(Position.BEFORE_ORDER, START_WITH, cond); } @@ -103,7 +103,7 @@ public abstract class AbstractOracleQuery * @param path path * @return the current object */ - @WithBridgeMethods(OracleQuery.class) + @WithBridgeMethods(value = OracleQuery.class, castRequired = true) public C orderSiblingsBy(Expression path) { return addFlag(Position.BEFORE_ORDER, ORDER_SIBLINGS_BY, path); } diff --git a/querydsl-sql/src/main/java/com/querydsl/sql/postgresql/AbstractPostgreSQLQuery.java b/querydsl-sql/src/main/java/com/querydsl/sql/postgresql/AbstractPostgreSQLQuery.java index cfbb5785a..7fb9b69f2 100644 --- a/querydsl-sql/src/main/java/com/querydsl/sql/postgresql/AbstractPostgreSQLQuery.java +++ b/querydsl-sql/src/main/java/com/querydsl/sql/postgresql/AbstractPostgreSQLQuery.java @@ -52,7 +52,7 @@ public abstract class AbstractPostgreSQLQuery... paths) { StringBuilder builder = new StringBuilder(" of "); for (RelationalPath path : paths) { @@ -94,7 +94,7 @@ public abstract class AbstractPostgreSQLQuery... exprs) { return addFlag(Position.AFTER_SELECT, Expressions.template(Object.class, "distinct on({0}) ",