You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@labs.apache.org by si...@apache.org on 2010/01/05 14:41:03 UTC
svn commit: r896036 -
/labs/magma/trunk/foundation-database/src/main/java/org/apache/magma/database/QueryByExampleBuilder.java
Author: simoneg
Date: Tue Jan 5 13:41:02 2010
New Revision: 896036
URL: http://svn.apache.org/viewvc?rev=896036&view=rev
Log:
Fix qbe builder to better support enums and avoid missing clauses correctly
Modified:
labs/magma/trunk/foundation-database/src/main/java/org/apache/magma/database/QueryByExampleBuilder.java
Modified: labs/magma/trunk/foundation-database/src/main/java/org/apache/magma/database/QueryByExampleBuilder.java
URL: http://svn.apache.org/viewvc/labs/magma/trunk/foundation-database/src/main/java/org/apache/magma/database/QueryByExampleBuilder.java?rev=896036&r1=896035&r2=896036&view=diff
==============================================================================
--- labs/magma/trunk/foundation-database/src/main/java/org/apache/magma/database/QueryByExampleBuilder.java (original)
+++ labs/magma/trunk/foundation-database/src/main/java/org/apache/magma/database/QueryByExampleBuilder.java Tue Jan 5 13:41:02 2010
@@ -140,6 +140,7 @@
} else if
(Boolean.class.isAssignableFrom(type) ||
Boolean.TYPE.isAssignableFrom(type) ||
+ type.isEnum() ||
DatabasePersisted.class.isAssignableFrom(type)) {
Object val = rm.invoke(qbe);
if (val == null) continue;
@@ -147,7 +148,7 @@
clause += "=?" + (params.size() + 1);
params.add(val);
}
- if (clause != null) {
+ if (clause != null && clause.length() > 0) {
if (wheresent) {
sb.append(this.useOr ? " OR " : " AND ");
} else {
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@labs.apache.org
For additional commands, e-mail: commits-help@labs.apache.org