X-Git-Url: http://git.kpe.io/?a=blobdiff_plain;f=db-oracle%2Foracle-sql.lisp;h=222e030c9b41881c05edfb5276d0327044a8a3f8;hb=a9f4dd155f9802d1444ab03339879232c6337f57;hp=b5876d44cbff9d96a6358d544af5e1728d65a43c;hpb=1df8353cc15bcbf98078605cb6955aafa622ecea;p=clsql.git

diff --git a/db-oracle/oracle-sql.lisp b/db-oracle/oracle-sql.lisp
index b5876d4..222e030 100644
--- a/db-oracle/oracle-sql.lisp
+++ b/db-oracle/oracle-sql.lisp
@@ -139,8 +139,7 @@ the length of that format.")
 
 (defun handle-oci-error (&key database nulls-ok)
   (cond (database
-         (with-slots (errhp)
-	     database
+         (with-slots (errhp) database
            (uffi:with-foreign-objects ((errbuf '(:array :unsigned-char
 						 #.+errbuf-len+))
 				       (errcode :long))
@@ -396,7 +395,9 @@ the length of that format.")
 			   (#.SQLT-FLT  
 			    (uffi:deref-array b '(:array :double) irow))
 			   (#.SQLT-INT  
-			    (uffi:deref-array b '(:array :int) irow))
+			    (ecase (cd-sizeof cd)
+			      (4
+			       (uffi:deref-array b '(:array :int) irow))))
 			   (#.SQLT-DATE 
 			    (deref-oci-string b irow (cd-sizeof cd))))))))
 	       (when (and (eq :string (cd-result-type cd))
@@ -626,11 +627,11 @@ the length of that format.")
 			     (deref-vp errhp))
 	       (let ((*scale (uffi:deref-pointer scale :byte))
 		     (*precision (uffi:deref-pointer precision :byte)))
-		 
-		 ;; (format t "scale=~d, precision=~d~%" *scale *precision)
+
+		 ;;(format t "scale=~d, precision=~d~%" *scale *precision)
 		 (cond
-		  ((or (and (zerop *scale) (not (zerop *precision)))
-		       (and (minusp *scale) (< *precision 10)))
+		  ((or (and (minusp *scale) (zerop *precision))
+		       (and (zerop *scale) (plusp *precision)))
 		   (setf buffer (acquire-foreign-resource :int +n-buf-rows+)
 			 sizeof 4			;; sizeof(int)
 			 dtype #.SQLT-INT))
@@ -924,8 +925,8 @@ the length of that format.")
 
 (defmethod database-execute-command (sql-expression (database oracle-database))
   (database-query sql-expression database nil nil)
-  ;; HACK HACK HACK
-  (database-query "commit" database nil nil)
+  (when (database-autocommit database)
+    (oracle-commit database))
   t)
 
 
@@ -993,27 +994,30 @@ the length of that format.")
 	  do (setf (nth i list) (nth i row)))
       list)))
 
-(defmethod clsql-sys:database-start-transaction ((database oracle-database))
+(defmethod database-start-transaction ((database oracle-database))
   (call-next-method)
-  )
-
-;;(with-slots (svchp errhp) database
-;;    (osucc (oci-trans-start (uffi:deref-pointer svchp)
-;;			    (uffi:deref-pointer errhp)
-;;			    60
-;;			    +oci-trans-new+)))
-;;  t)
-  
+  ;; Not needed with simple transaction
+  #+ignore
+  (with-slots (svchp errhp) database
+    (oci-trans-start (deref-vp svchp)
+		     (deref-vp errhp)
+		     60
+		     +oci-trans-new+))
+  t)
 
-(defmethod clsql-sys:database-commit-transaction ((database oracle-database))
-  (call-next-method)
+
+(defun oracle-commit (database)
   (with-slots (svchp errhp) database
-	      (osucc (oci-trans-commit (deref-vp svchp)
-				       (deref-vp errhp)
-				       0)))
+    (osucc (oci-trans-commit (deref-vp svchp)
+			     (deref-vp errhp)
+			     0))))
+
+(defmethod database-commit-transaction ((database oracle-database))
+  (call-next-method)
+  (oracle-commit database)
   t)
 
-(defmethod clsql-sys:database-abort-transaction ((database oracle-database))
+(defmethod database-abort-transaction ((database oracle-database))
   (call-next-method)
   (osucc (oci-trans-rollback (deref-vp (svchp database))
 			     (deref-vp (errhp database))