r11510: disconnect pooled session already removed from stack with

[clsql.git] / sql / operations.lisp

diff --git a/sql/operations.lisp b/sql/operations.lisp
index bdbb9294ca33500e8583c2904cc84d1aea8c3bcf..94ee20964b5c089ded361d10902eb30b890881c6 100644 (file)
--- a/sql/operations.lisp
+++ b/sql/operations.lisp
@@ -37,16 +37,16 @@
   (apply #'make-query args))
 
 (defsql sql-any (:symbol "any") (&rest rest)
-  (make-instance 'sql-value-exp
-                :modifier 'any :components rest))
+  (make-instance 'sql-function-exp
+                :name 'any :args rest))
 
 (defsql sql-some (:symbol "some") (&rest rest)
-  (make-instance 'sql-value-exp
-                :modifier 'some :components rest))
+  (make-instance 'sql-function-exp
+                :name 'some :args rest))
 
 (defsql sql-all (:symbol "all") (&rest rest)
-  (make-instance 'sql-value-exp
-                :modifier 'all :components rest))
+  (make-instance 'sql-function-exp
+                :name 'all :args rest))
 
 (defsql sql-not (:symbol "not") (&rest rest)
   (make-instance 'sql-value-exp
@@ -65,8 +65,8 @@
                 :operator 'except :sub-expressions rest))
 
 (defsql sql-minus (:symbol "minus") (&rest rest)
-  (make-instance 'sql-value-exp
-                :modifier 'minus :components rest))
+  (make-instance 'sql-set-exp 
+                :operator 'minus :sub-expressions rest))
 
 (defsql sql-limit (:symbol "limit") (&rest rest)
   (make-instance 'sql-query-modifier-exp 
@@ -95,8 +95,8 @@
                 :components '|NOT NULL|))
 
 (defsql sql-exists (:symbol "exists") (&rest rest)
-  (make-instance 'sql-value-exp
-                :modifier 'exists :components rest))
+  (make-instance 'sql-function-exp
+                :name 'exists :args rest))
 
 (defsql sql-* (:symbol "*") (&rest rest)
   (if (zerop (length rest))
@@ -137,17 +137,27 @@
 
 (defsql sql-in (:symbol "in") (&rest rest)
   (make-instance 'sql-relational-exp
-                :operator 'in :sub-expressions rest))
+                 :operator 'in :sub-expressions rest))
 
-(defsql sql-concat (:symbol "concat") (&rest rest)
+(defsql sql-concat-op (:symbol "concat-op") (&rest rest)
   (make-instance 'sql-relational-exp
                 :operator '\|\| :sub-expressions rest))
 
+(defsql sql-concat (:symbol "concat") (&rest rest)
+  (make-instance 'sql-function-exp
+                :name 'concat :args rest))
+
 (defsql sql-substr (:symbol "substr") (&rest rest)
+  (if (= (length rest) 3)
+      (make-instance 'sql-function-exp 
+                    :name 'substr :args rest)
+      (error 'sql-user-error :message "SUBSTR must have 3 arguments.")))
+
+(defsql sql-substring (:symbol "substring") (&rest rest)
   (if (= (length rest) 3)
       (make-instance 'sql-function-exp 
                     :name 'substring :args rest)
-      (error 'clsql-sql-syntax-error "SUBSTR must have 3 arguments.")))
+      (error 'sql-user-error :message "SUBSTRING must have 3 arguments.")))
 
 (defsql sql-is (:symbol "is") (&rest rest)
   (make-instance 'sql-relational-exp
@@ -213,9 +223,34 @@
 (defsql sql-between (:symbol "between") (&rest rest)
   (if (= (length rest) 3)
       (make-instance 'sql-between-exp :name 'between :args rest)
-      (error 'clsql-sql-syntax-error "BETWEEN must have 3 arguments.")))
+      (error 'sql-user-error :message "BETWEEN must have 3 arguments.")))
 
 (defsql sql-distinct (:symbol "distinct") (&rest rest)
   (make-instance 'sql-query-modifier-exp :modifier 'distinct 
                 :components rest))
 
+(defsql sql-coalesce (:symbol "coalesce") (&rest rest)
+  (make-instance 'sql-function-exp
+                :name 'coalesce :args rest))
+
+(defsql sql-nvl (:symbol "nvl") (&rest rest)
+  (if (= (length rest) 2)      
+      (make-instance 'sql-function-exp
+                     :name 'coalesce :args rest)
+      (error 'sql-user-error :message "NVL accepts exactly 2 arguments.")))
+
+(defsql sql-userenv (:symbol "userenv") (&rest rest)
+  (make-instance 'sql-function-exp
+                :name 'userenv :args rest))
+
+(defsql sql-lower  (:symbol "lower") (&rest rest)
+  (if (= (length rest) 1)
+      (make-instance 'sql-function-exp
+                     :name 'lower :args rest)
+    (error 'sql-user-error :message "LOWER must have 1 argument.")))
+
+(defsql sql-upper  (:symbol "upper") (&rest rest)
+  (if (= (length rest) 1)
+      (make-instance 'sql-function-exp
+                     :name 'upper :args rest)
+    (error 'sql-user-error :message "UPPER must have 1 argument.")))