r4871: *** empty log message ***
[umlisp.git] / sql-classes.lisp
index 088aec8fb9b07ce2c2567a6130115e0b515eb6b5..9e71b0f7b09972c0dcc41a68fce99b222e31857a 100644 (file)
@@ -7,7 +7,7 @@
 ;;;; Author:        Kevin M. Rosenberg
 ;;;; Date Started:  Apr 2000
 ;;;;
-;;;; $Id: sql-classes.lisp,v 1.58 2003/05/04 00:30:36 kevin Exp $
+;;;; $Id: sql-classes.lisp,v 1.75 2003/05/07 22:53:36 kevin Exp $
 ;;;;
 ;;;; This file, part of UMLisp, is
 ;;;;    Copyright (c) 2000-2002 by Kevin M. Rosenberg, M.D.
 (defun current-srl! (srl)
   (setq *current-srl* srl))
 
-;; only WHERE-VALUE and SRL are evaluated
-(defmacro with-umlisp-query ((table fields srl where-name where-value
-                                   &key (lrl "KCUILRL") distinct single
-                                   order like)
-                            &body body)
-  (let ((value (gensym)))
-    (if single
-       `(let ((,value ,where-value))
-         (when ,value 
-           (let ((tuple (car (umlisp-query ,table ,fields ,srl ,where-name ,value
-                                           :lrl ,lrl :single ,single
-                                           :distinct ,distinct :order ,order
-                                           :like ,like))))
-             (when tuple
-               (destructuring-bind ,fields tuple
-                 ,@body)))))
-       `(let ((,value ,where-value))
-         (when ,value 
-           (loop for tuple in
-                 (umlisp-query ,table ,fields ,srl ,where-name ,value
-                               :lrl ,lrl :single ,single :distinct ,distinct
-                               :order ,order :like ,like)
-                 collect (destructuring-bind ,fields tuple
-                           ,@body)))))))
-
-(defmacro with-umlisp-query-eval ((table fields srl where-name where-value
-                                        &key (lrl "KCUILRL") distinct single
-                                        order like)
-                                 &body body)
-  (let ((value (gensym)))
-    (if single
-       `(let ((,value ,where-value))
-         (when ,value 
-           (let ((tuple (car (umlisp-query-eval ,table ,fields ,srl ,where-name ,value
-                                           :lrl ,lrl :single ,single
-                                           :distinct ,distinct :order ,order
-                                           :like ,like))))
-             (when tuple
-               (destructuring-bind ,fields tuple
-                 ,@body)))))
-       `(let ((,value ,where-value))
-         (when ,value 
-           (loop for tuple in
-                 (umlisp-query-eval ,table ,fields ,srl ,where-name ,value
-                                    :lrl ,lrl :single ,single :distinct ,distinct
-                                    :order ,order :like ,like)
-                 collect (destructuring-bind ,fields tuple
-                           ,@body)))))))
-
-
-
-(defmacro umlisp-query (table fields srl where-name where-value
-                    &key (lrl "KCUILRL") single distinct order like)
-  "Query the UMLisp database. Return a list of umlisp objects whose name
-is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
-  `(mutex-sql-query
-    (query-string-macro ,table ,fields ,srl ,where-name ,where-value 
-     :lrl ,lrl :single ,single :distinct ,distinct :order ,order :like ,like)))
-
-(defun umlisp-query-eval (table fields srl where-name where-value
-                    &key (lrl "KCUILRL") single distinct order like)
-  "Query the UMLisp database. Return a list of umlisp objects whose name
-is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
-  (mutex-sql-query
-   (query-string-macro ,table ,fields ,srl ,where-name ,where-value 
-     :lrl ,lrl :single ,single :distinct ,distinct :order ,order :like ,like)))
-
-  
-(defmacro query-string-macro (table fields &optional srl where-name where-value
+(defmacro query-string (table fields &optional srl where-name where-value
                        &key (lrl "KCUILRL") single distinct order like)
   (let* ((%%fields (format nil "select ~A~{~:@(~A~)~^,~} from ~:@(~A~)"
                           (if distinct "distinct " "") fields table))
-        (%%order (if order (format nil " order by ~{~:@(~A~) ~(~A~)~^,~}" order) ""))
+        (%%order (if order (format nil " order by ~{~:@(~A~) ~(~A~)~^,~}"
+                                   order)
+                     ""))
         (%%lrl (format nil " and ~:@(~A~)<=" lrl))
         (%%where (when where-name
                    (format nil " where ~:@(~A~)~A" where-name
-                         (if like " like " "=")))))
+                         (if like " like " "")))))
     `(concatenate
       'string
       ,%%fields
       ,@(when %%where (list %%where))
       ,@(when %%where
-             `((if (numberp ,where-value)
-                   (write-to-string ,where-value)
-                   (format nil ,(if like "'%~A%'" "'~A'") ,where-value))))
+             `((typecase ,where-value
+                 (number
+                  (concatenate 'string "=" (write-to-string ,where-value)))
+                 (null
+                  " is null")
+                 (t
+                  (format nil ,(if like "'%~A%'" "='~A'") ,where-value)))))
       (if ,srl (concatenate 'string ,%%lrl (write-to-string ,srl)) "")
       ,@(when %%order (list %%order))
       ,@(when single (list " limit 1")))))
 
-(defun query-string (table fields &optional srl where-name where-value
-                    &key (lrl "KCUILRL") single distinct order like)
+(defun query-string-eval (table fields &optional srl where-name where-value
+                         &key (lrl "KCUILRL") single distinct order like)
   (concatenate
    'string
    (format nil "select ~A~{~:@(~A~)~^,~} from ~:@(~A~)" 
           (if distinct "distinct " "") fields table)
+   (if where-name (format nil " where ~:@(~A~)" where-name) "")
    (if where-name
        (format nil
-              (if (stringp where-value)
-                  (if like
-                      " where ~A like '%~A%'"
-                      " where ~A='~A'")
-                  " where ~A=~A")
-              where-name  where-value)
+              (typecase where-value
+                (number "=~D")
+                (null " is null")
+                (t
+                 (if like " like '%~A%""='~A'")))
+              where-value)
        "")
-   (if srl (format nil " and ~:@(~A~) <= ~D" lrl srl) "")
+   (if srl (format nil " and ~:@(~A~)<=~D" lrl srl) "")
    (if order (format nil " order by ~{~:@(~A~) ~(~A~)~^,~}" order) "")
    (if single " limit 1" "")))
 
+
+(defmacro umlisp-query (table fields srl where-name where-value
+                    &key (lrl "KCUILRL") single distinct order like)
+  "Query the UMLisp database. Return a list of umlisp objects whose name
+is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
+  `(mutex-sql-query
+    (query-string ,table ,fields ,srl ,where-name ,where-value 
+     :lrl ,lrl :single ,single :distinct ,distinct :order ,order :like ,like)))
+
+(defmacro umlisp-query-eval (table fields srl where-name where-value
+                    &key (lrl "KCUILRL") single distinct order like)
+  "Query the UMLisp database. Return a list of umlisp objects whose name
+is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
+  `(mutex-sql-query
+    (query-string-eval ,table ,fields ,srl ,where-name ,where-value 
+     :lrl ,lrl :single ,single :distinct ,distinct :order ,order :like ,like)))
+
+;; only WHERE-VALUE and SRL are evaluated
+(defmacro with-umlisp-query ((table fields srl where-name where-value
+                                   &key (lrl "KCUILRL") distinct single
+                                   order like)
+                            &body body)
+  (let ((value (gensym))
+       (r (gensym))) 
+    (if single
+       `(let* ((,value ,where-value)
+               (tuple (car (umlisp-query ,table ,fields ,srl ,where-name ,value
+                                         :lrl ,lrl :single ,single
+                                         :distinct ,distinct :order ,order
+                                         :like ,like))))
+         ,@(unless where-name `((declare (ignore ,value))))
+         (when tuple
+               (destructuring-bind ,fields tuple
+                 ,@body)))
+       `(let ((,value ,where-value))
+          ,@(unless where-name `((declare (ignore ,value))))
+          (let ((,r '()))
+            (dolist (tuple (umlisp-query ,table ,fields ,srl ,where-name ,value
+                                         :lrl ,lrl :single ,single :distinct ,distinct
+                                         :order ,order :like ,like))
+              (push (destructuring-bind ,fields tuple ,@body) ,r))
+            (nreverse ,r))
+          #+ignore
+          (loop for tuple in
+                (umlisp-query ,table ,fields ,srl ,where-name ,value
+                              :lrl ,lrl :single ,single :distinct ,distinct
+                              :order ,order :like ,like)
+              collect (destructuring-bind ,fields tuple ,@body))))))
+
+(defmacro with-umlisp-query-eval ((table fields srl where-name where-value
+                                        &key (lrl "KCUILRL") distinct single
+                                        order like)
+                                 &body body)
+  (let ((value (gensym))
+       (r (gensym))
+       (eval-fields (cadr fields)))
+    (if single
+       `(let* ((,value ,where-value)
+               (tuple (car (umlisp-query-eval ,table ,fields ,srl ,where-name ,value
+                                              :lrl ,lrl :single ,single
+                                              :distinct ,distinct :order ,order
+                                              :like ,like))))
+         (when tuple
+           (destructuring-bind ,eval-fields tuple
+             ,@body)))
+       `(let ((,value ,where-value)
+              (,r '()))
+          (dolist (tuple (umlisp-query-eval ,table ,fields ,srl ,where-name ,value
+                                            :lrl ,lrl :single ,single :distinct ,distinct
+                                            :order ,order :like ,like))
+            (push (destructuring-bind ,eval-fields tuple ,@body) ,r))
+          (nreverse ,r)
+          #+ignore
+          (loop for tuple in
+                (umlisp-query-eval ,table ,fields ,srl ,where-name ,value
+                                   :lrl ,lrl :single ,single :distinct ,distinct
+                                   :order ,order :like ,like)
+              collect (destructuring-bind ,eval-fields tuple ,@body))))))
+
+;;;
+;;; Read from SQL database
+
 (defun find-ucon-cui (cui &key (srl *current-srl*))
   "Find ucon for a cui"
   (with-umlisp-query (mrcon (kpfstr kcuilrl) srl cui (parse-cui cui) :single t)
-    (make-instance 'ucon :cui (parse-cui cui)
-                  :pfstr kpfstr
+    (make-instance 'ucon :cui (parse-cui cui) :pfstr kpfstr
                   :lrl (ensure-integer kcuilrl))))
 
-(defun find-ucon-cui-old (cui &key (srl *current-srl*))
-  "Find ucon for a cui"
-  (when (stringp cui) (setq cui (parse-cui cui)))
-  (when cui
-    (let ((ls (format nil "select KPFSTR,KCUILRL from MRCON where CUI=~d" cui)))
-      (when srl
-       (string-append ls (format nil " and KCUILRL <= ~d" srl)))
-      (string-append ls " limit 1")
-      (let ((tuple (car (mutex-sql-query ls))))
-       (destructuring-bind (kpfstr kcuilrl) tuple
-         (make-instance 'ucon :cui cui :pfstr kpfstr
-                        :lrl (ensure-integer kcuilrl)))))))
-
 (defun find-ucon-cui-sans-pfstr (cui &key (srl *current-srl*))
   "Find ucon for a cui"
   (with-umlisp-query (mrcon (kcuilrl) srl cui (parse-cui cui) :single t)
-    (make-instance 'ucon :cui (parse-cui cui)
-                  :lrl (ensure-integer kcuilrl)
+    (make-instance 'ucon :cui (parse-cui cui) :lrl (ensure-integer kcuilrl)
                   :pfstr nil)))
 
 (defun find-pfstr-cui (cui &key (srl *current-srl*))
@@ -168,16 +174,16 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
 
 (defun find-ucon-lui (lui &key (srl *current-srl*))
   "Find list of ucon for lui"
-  (with-umlisp-query (mrcon (cui kpfstr kcuilrl) srl lui (parse-lui lui) :distinct t)
-    (make-instance 'ucon :cui (ensure-integer cui)
-                  :pfstr kpfstr
+  (with-umlisp-query (mrcon (cui kpfstr kcuilrl) srl lui (parse-lui lui)
+                           :distinct t)
+    (make-instance 'ucon :cui (ensure-integer cui) :pfstr kpfstr
                   :lrl (ensure-integer kcuilrl))))
 
 (defun find-ucon-sui (sui &key (srl *current-srl*))
   "Find list of ucon for sui"
-  (with-umlisp-query (mrcon (cui kpfstr kcuilrl) srl sui (parse-sui sui) :distinct t)
-    (make-instance 'ucon :cui (ensure-integer cui)
-                  :pfstr kpfstr
+  (with-umlisp-query (mrcon (cui kpfstr kcuilrl) srl sui (parse-sui sui)
+                           :distinct t)
+    (make-instance 'ucon :cui (ensure-integer cui) :pfstr kpfstr
                   :lrl (ensure-integer kcuilrl))))
 
 (defun find-ucon-cuisui (cui sui &key (srl *current-srl*))
@@ -204,10 +210,18 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
         (make-instance 'ucon :cui (ensure-integer cui)
                        :pfstr pfstr
                        :lrl (ensure-integer cuilrl)))
-     (query-string-macro mrcon (cui kpfstr kcuilrl) srl nil nil
+     (query-string mrcon (cui kpfstr kcuilrl) srl nil nil
                   :order (cui asc) :distinct t)
      :database db)))
 
+(defun find-ucon-all2 (&key (srl *current-srl*))
+  "Return list of all ucon's"
+  (with-umlisp-query (mrcon (cui kpfstr kcuilrl) srl nil nil :order (cui asc)
+                           :distinct t)
+    (make-instance 'ucon :cui (ensure-integer cui)
+                  :pfstr kpfstr
+                  :lrl (ensure-integer kcuilrl))))
+
 (defun map-ucon-all (fn &key (srl *current-srl*))
   "Map a function over all ucon's"
   (with-sql-connection (db)
@@ -218,7 +232,8 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
                  (make-instance 'ucon :cui (ensure-integer cui)
                                 :pfstr pfstr
                                 :lrl (ensure-integer cuilrl))))
-     (query-string-macro mrcon (cui kpfstr kcuilrl) srl nil nil :order (cui asc) :distinct t)
+     (query-string mrcon (cui kpfstr kcuilrl) srl nil nil :order (cui asc)
+                  :distinct t)
      :database db)))
 
 
@@ -234,43 +249,50 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
 
 (defun find-usty-word (word &key (srl *current-srl*))
   "Return a list of usty that match word"
-  (with-umlisp-query (mrsty (tui sty) srl sty word :lrl klrl :like t :distinct t)
+  (with-umlisp-query (mrsty (tui sty) srl sty word :lrl klrl :like t
+                           :distinct t)
     (make-instance 'usty :tui (ensure-integer tui) :sty sty)))
 
 (defun find-urel-cui (cui &key (srl *current-srl*))
   "Return a list of urel for cui"
-  (with-umlisp-query (mrrel (rel cui2 rela sab sl mg kpfstr2) srl cui1 (parse-cui cui) :lrl "KSRL")
-    (make-instance 'urel :cui1 (parse-cui cui) :rel rel :cui2 (ensure-integer cui2) :rela rela
-                  :sab sab :sl sl :mg mg :pfstr2 kpfstr2)))
+  (with-umlisp-query (mrrel (rel cui2 rela sab sl mg kpfstr2) srl cui1
+                           (parse-cui cui) :lrl "KSRL")
+    (make-instance 'urel :cui1 (parse-cui cui) :rel rel
+                  :cui2 (ensure-integer cui2) :rela rela :sab sab :sl sl
+                  :mg mg :pfstr2 kpfstr2)))
 
 (defun find-urel-cui2 (cui2 &key (srl *current-srl*))
   "Return a list of urel for cui2"
-  (with-umlisp-query (mrrel (rel cui1 rela sab sl mg kpfstr2) srl cui2 (parse-cui cui2) :lrl "KSRL")
-    (make-instance 'urel :cui2 (parse-cui cui2) :rel rel :cui1 (ensure-integer cui1) :rela rela
-                  :sab sab :sl sl :mg mg :pfstr2 kpfstr2)))
+  (with-umlisp-query (mrrel (rel cui1 rela sab sl mg kpfstr2) srl cui2
+                           (parse-cui cui2) :lrl "KSRL")
+    (make-instance 'urel :cui2 (parse-cui cui2) :rel rel
+                  :cui1 (ensure-integer cui1) :rela rela :sab sab :sl sl
+                  :mg mg :pfstr2 kpfstr2)))
 
 (defun find-ucon-rel-cui2 (cui2 &key (srl *current-srl*))
-  (mapcar 
-   #'(lambda (cui) (find-ucon-cui cui :srl srl))
-   (remove-duplicates (mapcar #'cui1 (find-urel-cui2 cui2 :srl srl)))))
+  (loop for cui in (remove-duplicates
+                   (mapcar #'cui1 (find-urel-cui2 cui2 :srl srl)))
+       collect (find-ucon-cui cui :srl srl)))
 
 (defun find-ucoc-cui (cui &key (srl *current-srl*))
   "Return a list of ucoc for cui"
-  (with-umlisp-query (mrcoc (cui2 soc cot cof coa kpfstr2) srl cui1 (parse-cui cui) 
-                            :lrl "KSRL" :order (cof asc))
+  (with-umlisp-query (mrcoc (cui2 soc cot cof coa kpfstr2) srl cui1
+                           (parse-cui cui) :lrl klrl :order (cof asc))
     (setq cui2 (ensure-integer cui2))
     (when (zerop cui2) (setq cui2 nil))
-    (make-instance 'ucoc :cui1 (parse-cui cui) :cui2 (ensure-integer cui2) :soc soc :cot cot
-                  :cof (ensure-integer cof) :coa coa :pfstr2 kpfstr2)))
+    (make-instance 'ucoc :cui1 (parse-cui cui) :cui2 (ensure-integer cui2)
+                  :soc soc :cot cot :cof (ensure-integer cof) :coa coa
+                  :pfstr2 kpfstr2)))
 
 (defun find-ucoc-cui2 (cui2 &key (srl *current-srl*))
   "Return a list of ucoc for cui2"
-  (with-umlisp-query (mrcoc (cui1 soc cot cof coa kpfstr2) srl cui2 (parse-cui cui2) 
-                            :lrl "KSRL" :order (cof asc))
+  (with-umlisp-query (mrcoc (cui1 soc cot cof coa kpfstr2) srl cui2
+                           (parse-cui cui2) :lrl klrl :order (cof asc))
     (setq cui2 (ensure-integer cui2))
     (when (zerop cui2) (setq cui2 nil))
-    (make-instance 'ucoc :cui1 (ensure-integer cui1) :cui2 (parse-cui cui2) :soc soc :cot cot
-                  :cof (ensure-integer cof) :coa coa :pfstr2 kpfstr2)))
+    (make-instance 'ucoc :cui1 (ensure-integer cui1) :cui2 (parse-cui cui2)
+                  :soc soc :cot cot :cof (ensure-integer cof) :coa coa
+                  :pfstr2 kpfstr2)))
 
 (defun find-ucon-coc-cui2 (cui2 &key (srl *current-srl*))
   "List of ucon with co-occurance cui2"
@@ -280,14 +302,10 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
 
 (defun find-ulo-cui (cui &key (srl *current-srl*))
   "Return a list of ulo for cui"
-  (with-umlisp-query (mrlo (isn fr un sui sna soui) srl cui (parse-cui cui) :lrl "KLRL")
-    (make-instance 'ulo :isn isn :fr (ensure-integer fr) :un un :sui (ensure-integer sui) :sna sna
-                  :soui soui)))
-
-(defgeneric suistr (lo))
-(defmethod suistr ((lo ulo))
-  "Return the string for a ulo object"
-  (find-string-sui (sui lo)))
+  (with-umlisp-query (mrlo (isn fr un sui sna soui) srl cui (parse-cui cui)
+                          :lrl "KLRL")
+    (make-instance 'ulo :isn isn :fr (ensure-integer fr) :un un
+                  :sui (ensure-integer sui) :sna sna :soui soui)))
 
 (defun find-uatx-cui (cui &key (srl *current-srl*))
   "Return a list of uatx for cui"
@@ -297,8 +315,8 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
 
 (defun find-uterm-cui (cui &key (srl *current-srl*))
   "Return a list of uterm for cui"
-  (with-umlisp-query (mrcon (lui lat ts kluilrl) srl cui (parse-cui cui) :lrl kluilrl
-                            :distinct t)
+  (with-umlisp-query (mrcon (lui lat ts kluilrl) srl cui (parse-cui cui)
+                           :lrl kluilrl :distinct t)
     (make-instance 'uterm :lui (ensure-integer lui) :cui (parse-cui cui)
                   :lat lat :ts ts :lrl (ensure-integer kluilrl))))
 
@@ -314,28 +332,32 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
   (with-umlisp-query (mrcon (lat ts kluilrl) srl kcuilui
                             (make-cuilui (parse-cui cui) (parse-lui lui))
                             :lrl kluilrl :single t)
-    (make-instance 'uterm :cui cui :lui lui :lat lat :ts ts :lrl (ensure-integer kluilrl))))
+    (make-instance 'uterm :cui cui :lui lui :lat lat :ts ts
+                  :lrl (ensure-integer kluilrl))))
 
 (defun find-ustr-cuilui (cui lui &key (srl *current-srl*))
   "Return a list of ustr for cui/lui"
-  (with-umlisp-query (mrcon (sui stt str lrl) srl kcuilui (make-cuilui cui lui) :lrl lrl)
+  (with-umlisp-query (mrcon (sui stt str lrl) srl kcuilui
+                           (make-cuilui cui lui) :lrl lrl)
     (make-instance 'ustr :sui (ensure-integer sui) :cui cui :lui lui
                   :cuisui (make-cuisui cui sui) :stt stt :str str
                   :lrl (ensure-integer lrl))))
 
 (defun find-ustr-cuisui (cui sui &key (srl *current-srl*))
   "Return the single ustr for cuisui"
-  (with-umlisp-query (mrcon (lui stt str lrl) srl kcuisui (make-cuisui cui sui) :lrl lrl :single t)
+  (with-umlisp-query (mrcon (lui stt str lrl) srl kcuisui
+                           (make-cuisui cui sui) :lrl lrl :single t)
     (make-instance 'ustr :sui sui :cui cui :cuisui (make-cuisui cui sui)
-                  :lui (ensure-integer lui) :stt stt :str str :lrl (ensure-integer lrl))))
+                  :lui (ensure-integer lui) :stt stt :str str
+                  :lrl (ensure-integer lrl))))
 
 (defun find-ustr-sui (sui &key (srl *current-srl*))
   "Return the list of ustr for sui"
-  (with-umlisp-query (mrcon (cui lui stt str lrl) srl sui (parse-sui sui) :lrl lrl)
+  (with-umlisp-query (mrcon (cui lui stt str lrl) srl sui (parse-sui sui)
+                           :lrl lrl)
     (make-instance 'ustr :sui sui :cui cui :stt stt :str str
                   :cuisui (make-cuisui (ensure-integer cui) (parse-sui sui))
-                  :lui (ensure-integer lui)
-                  :lrl (ensure-integer lrl))))
+                  :lui (ensure-integer lui) :lrl (ensure-integer lrl))))
       
 (defun find-ustr-sab (sab &key (srl *current-srl*))
   "Return the list of ustr for sab"
@@ -343,7 +365,8 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
     (let ((cuisui (ensure-integer kcuisui)))
       (apply #'find-ustr-cuisui 
             (append
-             (multiple-value-list (decompose-cuisui cuisui)) (list :srl srl))))))
+             (multiple-value-list (decompose-cuisui cuisui))
+             (list :srl srl))))))
 
 (defun find-ustr-all (&key (srl *current-srl*))
   "Return list of all ustr's"
@@ -351,18 +374,14 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
       (clsql:map-query 
        'list
        #'(lambda (cui lui sui stt lrl pfstr)
-          (setq cui (ensure-integer cui))
-          (setq lui (ensure-integer lui))
-          (setq sui (ensure-integer sui))      
-          (setq lrl (ensure-integer lrl))
-          (make-instance 'ustr :cui cui
-                         :lui lui
-                         :sui sui
-                         :cuisui (make-cuisui cui sui)
-                         :stt stt
-                         :lrl lrl
-                         :str pfstr))
-       (query-string-macro mrcon (cui lui sui stt lrl kpfstr) srl nil nil :lrl lrl :distinct t
+          (make-instance 'ustr :cui (ensure-integer cui)
+                         :lui (ensure-integer lui) :sui (ensure-integer sui)
+                         :stt stt :str pfstr
+                         :cuisui (make-cuisui (ensure-integer cui)
+                                              (ensure-integer sui))
+                         :lrl (ensure-integer lrl)))
+       (query-string mrcon (cui lui sui stt lrl kpfstr) srl nil nil :lrl lrl
+                    :distinct t
                     :order (sui asc))
        :database db)))
 
@@ -372,26 +391,27 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
     str))
 
 (defun find-uso-cuisui (cui sui &key (srl *current-srl*))
-  (with-umlisp-query (mrso (sab code srl tty) srl kcuisui (make-cuisui cui sui) :lrl srl)
+  (with-umlisp-query (mrso (sab code srl tty) srl kcuisui
+                          (make-cuisui cui sui) :lrl srl)
       (make-instance 'uso :sab sab :code code :srl srl :tty tty)))
 
 (defun find-ucxt-cuisui (cui sui &key (srl *current-srl*))
-  (with-umlisp-query (mrcxt (sab code cxn cxl rnk cxs cui2 hcd rela xc) srl kcuisui
-                            (make-cuisui cui sui) :lrl ksrl)
+  (with-umlisp-query (mrcxt (sab code cxn cxl rnk cxs cui2 hcd rela xc)
+                           srl kcuisui (make-cuisui cui sui) :lrl ksrl)
     (make-instance 'ucxt :sab sab :code code
-                  :cxn (ensure-integer cxn)
-                  :cxl cxl :cxs cxs :hcd hcd :rela rela :xc xc
-                  :rnk (ensure-integer rnk)
+                  :cxn (ensure-integer cxn) :cxl cxl :cxs cxs :hcd hcd
+                  :rela rela :xc xc :rnk (ensure-integer rnk)
                   :cui2 (ensure-integer cui2))))
 
 (defun find-usat-ui (cui &optional (lui nil) (sui nil) &key (srl *current-srl*))
   (let ((ls (format nil "select CODE,ATN,SAB,ATV from MRSAT where ")))
     (cond
-      (sui (string-append ls (format nil "KCUISUI=~d" (make-cuisui cui sui))))
-      (lui (string-append ls (format nil "KCUILUI=~d and sui=0" (make-cuilui cui lui))))
-      (t (string-append ls (format nil "cui=~d and lui=0 and sui=0" cui))))
+      (sui (string-append ls (format nil "KCUISUI=~D" (make-cuisui cui sui))))
+      (lui (string-append ls (format nil "KCUILUI=~D and sui=0"
+                                    (make-cuilui cui lui))))
+      (t (string-append ls (format nil "cui=~D and lui=0 and sui=0" cui))))
     (when srl
-      (string-append ls (format nil " and KSRL <= ~d" srl)))
+      (string-append ls (format nil " and KSRL <= ~D" srl)))
     (loop for tuple in (mutex-sql-query ls) collect 
          (destructuring-bind (code atn sab atv) tuple
            (make-instance 'usat :code code :atn atn :sab sab :atv atv)))))
@@ -413,24 +433,27 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
 
 (defun find-usab-all ()
   "Find usab for a key"
-  (with-umlisp-query (mrsab (vcui rcui vsab rsab son sf sver mstart mend imeta rmeta slc scc srl tfr cfr cxty ttyl atnl lat cenc curver sabin) nil nil nil)
+  (with-umlisp-query (mrsab (vcui rcui vsab rsab son sf sver mstart mend imeta
+                                 rmeta slc scc srl tfr cfr cxty ttyl atnl lat
+                                 cenc curver sabin) nil nil nil)
     (make-instance 'usab :vcui (ensure-integer vcui) 
-                  :rcui (ensure-integer rcui)
-                  :vsab vsab :rsab rsab :son son :sf sf :sver sver :mstart mstart
-                  :mend mend :imeta imeta :rmeta rmeta :slc slc :scc scc
-                  :srl (ensure-integer srl) 
+                  :rcui (ensure-integer rcui) :vsab vsab :rsab rsab :son son
+                  :sf sf :sver sver :mstart mstart :mend mend :imeta imeta
+                  :rmeta rmeta :slc slc :scc scc  :srl (ensure-integer srl)
                   :tfr (ensure-integer tfr) :cfr (ensure-integer cfr)
                   :cxty cxty :ttyl ttyl :atnl atnl :lat lat :cenc cenc
                   :curver curver :sabin sabin)))
 
 (defun find-usab-by-key (key-name key)
   "Find usab for a key"
-  (with-umlisp-query-eval ('mrsab '(vcui rcui vsab rsab son sf sver mstart mend imeta rmeta slc scc srl tfr cfr cxty ttyl atnl lat cenc curver sabin) nil key-name key :single t)
+  (with-umlisp-query-eval ('mrsab '(vcui rcui vsab rsab son sf sver mstart
+                                   mend imeta rmeta slc scc srl tfr cfr cxty
+                                   ttyl atnl lat cenc curver sabin)
+                                 nil key-name key :single t)
     (make-instance 'usab :vcui (ensure-integer vcui) 
-                  :rcui (ensure-integer rcui)
-                  :vsab vsab :rsab rsab :son son :sf sf :sver sver :mstart mstart
-                  :mend mend :imeta imeta :rmeta rmeta :slc slc :scc scc
-                  :srl (ensure-integer srl) 
+                  :rcui (ensure-integer rcui) :vsab vsab :rsab rsab :son son
+                  :sf sf :sver sver :mstart mstart :mend mend :imeta imeta
+                  :rmeta rmeta :slc slc :scc scc :srl (ensure-integer srl)
                   :tfr (ensure-integer tfr) :cfr (ensure-integer cfr)
                   :cxty cxty :ttyl ttyl :atnl atnl :lat lat :cenc cenc
                   :curver curver :sabin sabin)))
@@ -450,27 +473,25 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
 
 (defun find-ucon-tui (tui &key (srl *current-srl*))
   "Find list of ucon for tui"
-  (with-umlisp-query (mrsty (cui) srl tui (parse-tui tui) :lrl klrl
-                            :order (cui asc))
+  (with-umlisp-query (mrsty (cui) srl tui (parse-tui tui) :lrl klrl :order (cui asc))
     (find-ucon-cui (ensure-integer cui) :srl srl)))
   
 (defun find-ucon-word (word &key (srl *current-srl*) (like nil))
   "Return list of ucons that match word. Optionally, use SQL's LIKE syntax"
   (with-umlisp-query-eval ('mrxw_eng '(cui) srl 'wd word :like like :distinct t
-                                    :lrl klrl :order '(cui asc))
+                                    :lrl 'klrl :order '(cui asc))
     (find-ucon-cui cui :srl srl)))
 
 (defun find-ucon-normalized-word (word &key (srl *current-srl*) (like nil))
   "Return list of ucons that match word, optionally use SQL's LIKE syntax"
   (with-umlisp-query-eval ('mrxnw_eng '(cui) srl 'nwd word :like like :distinct t
-                                     :lrl klrl :order '(cui asc))
+                                     :lrl 'klrl :order '(cui asc))
     (find-ucon-cui cui :srl srl)))
 
 (defun find-ustr-word (word &key (srl *current-srl*))
   "Return list of ustrs that match word"
-  (with-umlisp-query (mrxw_eng (cui sui) srl wd word
-                               :lrl klrl
-                               :order (cui asc sui asc))
+  (with-umlisp-query (mrxw_eng (cui sui) srl wd word :lrl klrl
+                              :order (cui asc sui asc))
     (find-ustr-cuisui (ensure-integer cui) (ensure-integer sui) :srl srl)))
 
 (defun find-ustr-normalized-word (word &key (srl *current-srl*))
@@ -479,6 +500,18 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
                                 :order (cui asc sui asc))
     (find-ustr-cuisui (ensure-integer cui) (ensure-integer sui) :srl srl)))
 
+(defun find-ucon-noneng-word (word &key (srl *current-srl*) (like nil))
+  "Return list of ucons that match non-english word"
+  (with-umlisp-query-eval ('mrxw_noneng '(cui) srl 'wd word :like like
+                                       :distinct t :lrl 'klrl :order '(cui asc))
+    (find-ucon-cui cui :srl srl)))
+
+(defun find-ustr-noneng-word (word &key (srl *current-srl*))
+  "Return list of ustrs that match non-english word"
+  (with-umlisp-query (mrxw_noneng (cui sui) srl wd word :lrl klrl
+                                 :order (cui asc sui asc))
+    (find-ustr-cuisui (ensure-integer cui) (ensure-integer sui) :srl srl)))
+
 ;; Special tables
 
 (defun find-usrl-all ()
@@ -515,9 +548,7 @@ is OBJNAME from TABLE where WHERE-NAME field = WHERE-VALUE with FIELDS"
   "Sort a list of objects based on scoring to a string"
   (let ((scored '()))
     (dolist (obj objs)
-      (push 
-       (list obj 
-            (score-multiword-match str (funcall lookup-func obj))) 
+      (push (list obj (score-multiword-match str (funcall lookup-func obj))) 
        scored))
     (mapcar #'car (sort scored #'> :key #'cadr))))