You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@openjpa.apache.org by mp...@apache.org on 2007/01/24 23:48:24 UTC

svn commit: r499601 - /incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/QueryImpl.java

Author: mprudhom
Date: Wed Jan 24 14:48:23 2007
New Revision: 499601

URL: http://svn.apache.org/viewvc?view=rev&rev=499601
Log:
Throw an exception if the user tries to perform an bulk update with a parameter other than a primitive.

Modified:
    incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/QueryImpl.java

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/QueryImpl.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/QueryImpl.java?view=diff&rev=499601&r1=499600&r2=499601
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/QueryImpl.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/QueryImpl.java Wed Jan 24 14:48:23 2007
@@ -1070,7 +1070,7 @@
             } else if (value instanceof Constant) {
                 val = ((Constant) value).getValue(params);
             } else {
-                val = null;
+                throw new UserException(_loc.get("only-update-primitives"));
             }
 
             OpenJPAStateManager sm = _broker.getStateManager(ob);