X-Git-Url: http://git.kpe.io/?a=blobdiff_plain;f=sql%2Fexpressions.lisp;h=5e75b01ccdfaaae1842dabe6dea7179cb77674e0;hb=008ef476827e2b4ec81829f1e204e9f954017678;hp=b54029e4b727121c4d299ab888f00cd4f5950eb6;hpb=709d415122d04713da58a7c6d7ee10162c386a56;p=clsql.git

diff --git a/sql/expressions.lisp b/sql/expressions.lisp
index b54029e..5e75b01 100644
--- a/sql/expressions.lisp
+++ b/sql/expressions.lisp
@@ -1,8 +1,6 @@
 ;;;; -*- Mode: LISP; Syntax: ANSI-Common-Lisp; Base: 10 -*-
 ;;;; *************************************************************************
 ;;;;
-;;;; $Id$
-;;;;
 ;;;; Classes defining SQL expressions and methods for formatting the
 ;;;; appropriate SQL commands.
 ;;;;
@@ -194,15 +192,25 @@
     sql
     `(make-instance 'sql-ident-table :name ',name :table-alias ',alias)))
 
+(defun special-char-p (s)
+  "Check if a string has any special characters"
+  (loop for char across s
+       thereis (find char '(#\space #\, #\. #\! #\@ #\# #\$ #\%
+                                 #\^ #\& #\* #\| #\( #\) #\- #\+))))
+
 (defmethod output-sql ((expr sql-ident-table) database)
   (with-slots (name alias) expr
-    (etypecase name
-      (string
-       (format *sql-stream* "~s" (sql-escape name)))
-      (symbol
-       (write-string (sql-escape name) *sql-stream*)))
-    (when alias
-      (format *sql-stream* " ~s" alias)))
+    (flet ((p (s) ;; the etypecase is in sql-escape too
+	     (let ((sym? (symbolp s))
+		   (s (sql-escape s)))
+	       (format *sql-stream*
+		       (if (and sym? (not (special-char-p s)))
+			   "~a" "~s")
+		       s))))
+      (p name)
+      (when alias
+	(princ #\space *sql-stream*)
+	(p alias))))
   t)
 
 (defmethod output-sql-hash-key ((expr sql-ident-table) database)