You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by fa...@apache.org on 2008/02/14 22:11:01 UTC

svn commit: r627873 - /openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/SQLStoreQuery.java

Author: fancy
Date: Thu Feb 14 13:10:50 2008
New Revision: 627873

URL: http://svn.apache.org/viewvc?rev=627873&view=rev
Log:
OPENJPA-477 fix  setting up parameter list for PreparedStatement

Modified:
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/SQLStoreQuery.java

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/SQLStoreQuery.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/SQLStoreQuery.java?rev=627873&r1=627872&r2=627873&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/SQLStoreQuery.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/SQLStoreQuery.java Thu Feb 14 13:10:50 2008
@@ -226,11 +226,10 @@
             try {
                 stmnt = prepareCall(conn, buf);
 
-                int index = 0;
-                for (Iterator i = paramList.iterator(); i.hasNext() && 
-                    stmnt != null;)
-                    dict.setUnknown(stmnt, ++index, i.next(), null);
-                
+                buf.setParameters(paramList);
+                if (stmnt != null)
+                    buf.setParameters(stmnt);
+
                 int count = executeUpdate(store, conn, stmnt, buf);                
                 return Numbers.valueOf(count);
             } catch (SQLException se) {