X-Git-Url: http://git.kpe.io/?a=blobdiff_plain;f=db-oracle%2Foracle-sql.lisp;h=3598dc93f628953edad15e9c753d83d6eaa5dd16;hb=981c14d37d3c6b4037df4a2e43a4a2be5f8b358a;hp=a86a5615796ace3b3fbbf07949dcba949c41639a;hpb=63158b2ef9d21af6454966d78b7e3ea81a9c73aa;p=clsql.git

diff --git a/db-oracle/oracle-sql.lisp b/db-oracle/oracle-sql.lisp
index a86a561..3598dc9 100644
--- a/db-oracle/oracle-sql.lisp
+++ b/db-oracle/oracle-sql.lisp
@@ -15,8 +15,7 @@
 
 (in-package #:clsql-oracle)
 
-(defmethod database-initialize-database-type
-    ((database-type (eql :oracle)))
+(defmethod database-initialize-database-type ((database-type (eql :oracle)))
   t)
 
 ;;;; arbitrary parameters, tunable for performance or other reasons
@@ -205,7 +204,7 @@ the length of that format.")
 (uffi:def-type string-array (:array :unsigned-char))
 
 (defun deref-oci-string (arrayptr string-index size)
-  (declare (type string-array arrayptr))
+;;  (declare (type string-array arrayptr))
   (declare (type (mod #.+n-buf-rows+) string-index))
   (declare (type (and unsigned-byte fixnum) size))
   (let* ((raw (uffi:convert-from-foreign-string 
@@ -415,7 +414,7 @@ the length of that format.")
 		    (value
 		     (let* ((arb (foreign-resource-buffer (cd-indicators cd)))
 			    (indicator (uffi:deref-array arb '(:array :short) irow)))
-		       (declare (type short-array arb))
+		       ;; b(declare (type short-array arb))
 		       (unless (= indicator -1)
 			 (ecase (cd-oci-data-type cd)
 			   (#.SQLT-STR  
@@ -792,16 +791,26 @@ the length of that format.")
 	)
       ;; Actually, oci-server-version returns the client version, not the server versions
       ;; will use "SELECT VERSION FROM V$INSTANCE" to get actual server version.
-      (let (db client-version)
+      (let (db server-version client-version)
+	(declare (ignorable server-version))
 	(uffi:with-foreign-object (buf '(:array :unsigned-char #.+errbuf-len+))
 	  (oci-server-version (deref-vp svchp)
 			      (deref-vp errhp)
 			      (uffi:char-array-to-pointer buf)
 			      +errbuf-len+ +oci-htype-svcctx+)
-	  (setf client-version (uffi:convert-from-foreign-string buf)))
+	  (setf client-version (uffi:convert-from-foreign-string buf))
+	  ;; This returns the client version, not the server version, so diable it
+	  #+ignore
+	  (oci-server-version (deref-vp srvhp)
+			      (deref-vp errhp)
+			      (uffi:char-array-to-pointer buf)
+			      +errbuf-len+ +oci-htype-server+)
+	  #+ignore
+	  (setf server-version (uffi:convert-from-foreign-string buf)))
 	(setq db (make-instance 'oracle-database
 				:name (database-name-from-spec connection-spec
 							       database-type)
+				:connection-spec connection-spec
 				:envhp envhp
 				:errhp errhp
 				:database-type :oracle
@@ -809,8 +818,11 @@ the length of that format.")
 				:dsn data-source-name
 				:user user
 				:client-version client-version
+				:server-version server-version
 				:major-client-version (major-client-version-from-string
-						       client-version)))
+						       client-version)
+				:major-server-version (major-client-version-from-string
+						       server-version)))
 	(oci-logon (deref-vp envhp)
 		   (deref-vp errhp) 
 		   svchp
@@ -822,9 +834,9 @@ the length of that format.")
 	(setf (slot-value db 'clsql-sys::state) :open)
         (database-execute-command
 	 (format nil "ALTER SESSION SET NLS_DATE_FORMAT='~A'" (date-format db)) db)
-	(let ((server-version (caar (database-query "SELECT VERSION FROM V$INSTANCE" db nil nil))))
+	(let ((server-version (caar (database-query "SELECT BANNER FROM V$VERSION WHERE BANNER LIKE '%Oracle%'" db nil nil))))
 	  (setf (slot-value db 'server-version) server-version
-		(slot-value db 'major-server-version) (major-server-version-from-string 
+		(slot-value db 'major-server-version) (major-client-version-from-string
 						       server-version)))
         db))))