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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to