r9449: * db-odbc/odbc-sql.lisp, db-aodbc/aodbc-sql.lisp: Move common code to
[clsql.git] / db-odbc / odbc-sql.lisp
index bfc6d892b4faf0c838749cba697dbf202ca9fa5a..862c99188490b1f597f5fd1c31a23783c6e6165c 100644 (file)
@@ -17,7 +17,7 @@
 ;;;; *************************************************************************
 
 (defpackage #:clsql-odbc
-    (:use #:common-lisp #:clsql-base-sys)
+    (:use #:common-lisp #:clsql-sys)
     (:export #:odbc-database)
     (:documentation "This is the CLSQL interface to ODBC."))
 
@@ -25,9 +25,8 @@
 
 ;; ODBC interface
 
-(defclass odbc-database (database)
-  ((odbc-conn :accessor database-odbc-conn :initarg :odbc-conn)
-   (odbc-db-type :accessor database-odbc-db-type)))
+(defclass odbc-database (generic-odbc-database)
+  ((odbc-db-type :accessor database-odbc-db-type)))
 
 (defmethod database-name-from-spec (connection-spec
                                    (database-type (eql :odbc)))
   (check-connection-spec connection-spec database-type (dsn user password))
   (destructuring-bind (dsn user password) connection-spec
     (handler-case
-       (let ((db
-              (make-instance 'odbc-database
-                :name (database-name-from-spec connection-spec :odbc)
-                :database-type :odbc
-                :odbc-conn
-                (odbc-dbi:connect :user user
-                                  :password password
-                                  :data-source-name dsn))))
+       (let ((db (make-instance 'odbc-database
+                                :name (database-name-from-spec connection-spec :odbc)
+                                :database-type :odbc
+                                :dbi-package (find-package '#:odbc-dbi)
+                                :odbc-conn
+                                (odbc-dbi:connect :user user
+                                                  :password password
+                                                  :data-source-name dsn))))
          (store-type-of-connected-database db)
          db)
-    (clsql-error (e)
-      (error e))
-    #+ignore
-    (error ()  ;; Init or Connect failed
-      (error 'clsql-connect-error
-            :database-type database-type
-            :connection-spec connection-spec
-            :errno nil
-            :error "Connection failed")))))
+      (error ()        ;; Init or Connect failed
+       (error 'sql-connection-error
+              :database-type database-type
+              :connection-spec connection-spec
+              :message "Connection failed")))))
 
 (defmethod database-underlying-type ((database odbc-database))
   (database-odbc-db-type database))
 
 (defun store-type-of-connected-database (db)
-  (let* ((odbc-conn (database-odbc-conn db))
+  (let* ((odbc-conn (clsql-sys::odbc-conn db))
         (server-name (odbc-dbi::get-odbc-info odbc-conn odbc::$SQL_SERVER_NAME))
         (dbms-name (odbc-dbi::get-odbc-info odbc-conn odbc::$SQL_DBMS_NAME))
         (type
            :oracle))))
     (setf (database-odbc-db-type db) type)))
   
-(defmethod database-disconnect ((database odbc-database))
-  (odbc-dbi:disconnect (database-odbc-conn database))
-  (setf (database-odbc-conn database) nil)
-  t)
-
-(defmethod database-query (query-expression (database odbc-database) 
-                          result-types field-names) 
-  (handler-case
-      (odbc-dbi:sql query-expression :db (database-odbc-conn database)
-                   :result-types result-types
-                    :column-names field-names)
-    (clsql-error (e)
-      (error e))
-    #+ignore
-    (error ()
-      (error 'clsql-sql-error
-            :database database
-            :expression query-expression
-            :errno nil
-            :error "Query failed"))))
-
-(defmethod database-execute-command (sql-expression 
-                                    (database odbc-database))
-  (handler-case
-      (odbc-dbi:sql sql-expression :db (database-odbc-conn database))
-    (clsql-error (e)
-      (error e))
-    #+ignore
-    (error ()
-      (error 'clsql-sql-error
-            :database database
-            :expression sql-expression
-            :errno nil
-            :error "Execute command failed"))))
-
-(defstruct odbc-result-set
-  (query nil)
-  (types nil)
-  (full-set nil :type boolean))
 
-(defmethod database-query-result-set ((query-expression string)
-                                     (database odbc-database) 
-                                     &key full-set result-types)
-  (handler-case 
-      (multiple-value-bind (query column-names)
-         (odbc-dbi:sql query-expression 
-                  :db (database-odbc-conn database) 
-                  :row-count nil
-                  :column-names t
-                  :query t
-                  :result-types result-types)
-       (values
-        (make-odbc-result-set :query query :full-set full-set 
-                               :types result-types)
-        (length column-names)
-        nil ;; not able to return number of rows with odbc
-        ))
-    #+ignore
-    (error ()
-      (error 'clsql-sql-error
-            :database database
-            :expression query-expression
-            :errno nil
-            :error "Query result set failed"))))
 
-(defmethod database-dump-result-set (result-set (database odbc-database))
-  (odbc-dbi:close-query (odbc-result-set-query result-set))
-  t)
-
-(defmethod database-store-next-row (result-set
-                                   (database odbc-database)
-                                   list)
-  (let ((row (odbc-dbi:fetch-row (odbc-result-set-query result-set) nil 'eof)))
-    (if (eq row 'eof)
-       nil
-      (progn
-       (loop for elem in row
-           for rest on list
-           do
-             (setf (car rest) elem))
-       list))))
 
 ;;; Sequence functions
 
      (mapcan #'(lambda (s)
                 (let ((sn (%table-name-to-sequence-name (car s))))
                   (and sn (list sn))))
-            (database-query "SHOW TABLES" database nil)))
+            (database-query "SHOW TABLES" database nil nil)))
     ((:postgresql :postgresql-socket)
      (mapcar #'(lambda (s) (%table-name-to-sequence-name (car s)))
            (database-query "SELECT RELNAME FROM pg_class WHERE RELNAME LIKE '%clsql_seq%'" 
-                           database nil)))))
+                           database nil nil)))))
 
-(defmethod database-list-tables ((database odbc-database)
-                                &key (owner nil))
-  (declare (ignore owner))
-    (multiple-value-bind (rows col-names)
-       (odbc-dbi:list-all-database-tables :db (database-odbc-conn database))
-      (declare (ignore col-names))
-      ;; TABLE_SCHEM is hard-coded in second column by ODBC Driver Manager
-      ;; TABLE_NAME in third column, TABLE_TYPE in fourth column
-      (loop for row in rows
-         when (and (not (string-equal "information_schema" (nth 1 row)))
-                   (string-equal "TABLE" (nth 3 row)))
-         collect (nth 2 row))))
-
-(defmethod database-list-views ((database odbc-database)
-                                &key (owner nil))
-  (declare (ignore owner))
-    (multiple-value-bind (rows col-names)
-       (odbc-dbi:list-all-database-tables :db (database-odbc-conn database))
-      (declare (ignore col-names))
-      ;; TABLE_SCHEM is hard-coded in second column by ODBC Driver Manager
-      ;; TABLE_NAME in third column, TABLE_TYPE in fourth column
-      (loop for row in rows
-         when (and (not (string-equal "information_schema" (nth 1 row)))
-                   (string-equal "VIEW" (nth 3 row)))
-         collect (nth 2 row))))
-
-(defmethod database-list-attributes ((table string) (database odbc-database)
-                                     &key (owner nil))
-  (declare (ignore owner))
-  (multiple-value-bind (rows col-names)
-      (odbc-dbi:list-all-table-columns table :db (database-odbc-conn database))
-    (let ((pos (position "COLUMN_NAME" col-names :test #'string-equal)))
-      (when pos
-       (loop for row in rows
-           collect (nth pos row))))))
-
-(defmethod database-attribute-type ((attribute string) (table string) (database odbc-database)
-                                     &key (owner nil))
-  (declare (ignore owner))
-  (multiple-value-bind (rows col-names)
-      (odbc-dbi:list-all-table-columns table :db (database-odbc-conn database))
-    (let ((pos (position "TYPE_NAME" col-names :test #'string-equal)))
-      (when pos
-       (loop for row in rows
-           collect (nth pos row))))))
 
 (defmethod database-set-sequence-position (sequence-name
                                            (position integer)
           (car (database-query 
                 (concatenate 'string "SELECT last_value,is_called FROM " 
                              table-name)
-                database
-                :auto))))
+                database :auto nil))))
      (cond
        ((char-equal (schar (second tuple) 0) #\f)
        (database-execute-command
    (caar (database-query 
          (concatenate 'string "SELECT last_value FROM " 
                       (%sequence-name-to-table sequence-name))
-         database
-         :auto))))
+         database :auto nil))))
 
 (defmethod database-create (connection-spec (type (eql :odbc)))
+  (declare (ignore connection-spec))
   (warn "Not implemented."))
 
 (defmethod database-destroy (connection-spec (type (eql :odbc)))
+  (declare (ignore connection-spec))
   (warn "Not implemented."))
 
 (defmethod database-probe (connection-spec (type (eql :odbc)))
 (defmethod database-list-table-indexes (table (database odbc-database)
                                        &key (owner nil))
   (declare (ignore owner))
-  (odbc-list-table-indexes table database))
-
-(defun odbc-list-table-indexes (table database)
   (multiple-value-bind (rows col-names)
       (odbc-dbi:list-table-indexes 
        table
-       :db (database-odbc-conn database))
+       :db (clsql-sys::odbc-conn database))
     (declare (ignore col-names))
     ;; INDEX_NAME is hard-coded in sixth position by ODBC driver
     ;; FIXME: ??? is hard-coded in the fourth position
   ;; nothing to do
   t)
 
-(when (clsql-base-sys:database-type-library-loaded :odbc)
-  (clsql-base-sys:initialize-database-type :database-type :odbc))
+(when (clsql-sys:database-type-library-loaded :odbc)
+  (clsql-sys:initialize-database-type :database-type :odbc))