r5152: *** empty log message ***
[hyperobject.git] / views.lisp
index 1f01b7f5c8e3849abe86a4473a5f463c801cae3f..6776e39e1bc57a8455aa4a7711f09e7a4de281e4 100644 (file)
@@ -7,16 +7,12 @@
 ;;;; Programmer:    Kevin M. Rosenberg
 ;;;; Date Started:  Apr 2000
 ;;;;
-;;;; $Id: views.lisp,v 1.31 2003/05/13 15:56:50 kevin Exp $
-;;;;
-;;;; This file is Copyright (c) 2000-2002 by Kevin M. Rosenberg
+;;;; $Id: views.lisp,v 1.56 2003/06/17 17:50:45 kevin Exp $
 ;;;;
+;;;; This file is Copyright (c) 2000-2003 by Kevin M. Rosenberg
 ;;;; *************************************************************************
  
-(in-package :hyperobject)
-
-(eval-when (:compile-toplevel :execute)
-  (declaim (optimize (speed 2) (safety 2) (compilation-speed 0) (debug 2))))
+(in-package #:hyperobject)
 
 
 (defclass object-view ()
                   :accessor file-start-str)
    (file-end-str :type (or string null) :initform nil :initarg :file-end-str
                 :accessor file-end-str)
-   (list-start-fmtstr :type (or string null) :initform nil :initarg :list-start-fmtstr
-                     :accessor list-start-fmtstr)
-   (list-start-value-func :type (or function symbol null) :initform nil
-                         :initarg :list-start-value-func
-                         :accessor list-start-value-func)
+   (list-start-printer :type (or string function null) :initform nil
+                          :initarg :list-start-printer
+                     :accessor list-start-printer)
    (list-start-indent :initform nil :initarg :list-start-indent
                      :accessor list-start-indent)
-   (list-end-fmtstr :type (or string null) :initform nil :initarg :list-end-fmtstr
-                   :accessor list-end-fmtstr)
-   (list-end-value-func :type (or function symbol null) :initform nil
-                       :initarg :list-end-value-func
-                       :accessor list-end-value-func)
+   (list-end-printer :type (or string function null) :initform nil
+                        :initarg :list-end-printer
+                   :accessor list-end-printer)
    (list-end-indent :initform nil :initarg :list-end-indent
                    :accessor list-end-indent)
-   (obj-start-fmtstr :type (or string symbol null) :initform nil :initarg :obj-start-fmtstr
-                    :accessor obj-start-fmtstr)
-   (obj-start-value-func :initform nil :initarg :obj-start-value-func
-                        :accessor obj-start-value-func)
+   (obj-start-printer :type (or string function null) :initform nil :initarg :obj-start-printer
+                    :accessor obj-start-printer)
    (obj-start-indent :initform nil :initarg :obj-start-indent
                     :accessor obj-start-indent)
-   (obj-end-fmtstr :type (or string null) :initform nil :initarg :obj-end-fmtstr
-                  :accessor obj-end-fmtstr)
-   (obj-end-value-func :type (or function symbol null) :initform nil
-                      :initarg :obj-end-value-func
-                      :accessor obj-end-value-func)
+   (obj-end-printer :type (or string function null) :initform nil :initarg :obj-end-printer
+                  :accessor obj-end-printer)
    (obj-end-indent :initform nil :initarg :obj-end-indent
                   :accessor obj-end-indent)
+   (subobj-start-printer :type (or string function null) :initform nil :initarg :subobj-start-printer
+                    :accessor subobj-start-printer)
+   (subobj-start-indent :initform nil :initarg :subobj-start-indent
+                    :accessor subobj-start-indent)
+   (subobj-end-printer :type (or string function null) :initform nil :initarg :subobj-end-printer
+                  :accessor subobj-end-printer)
+   (subobj-end-indent :initform nil :initarg :subobj-end-indent
+                  :accessor subobj-end-indent)
    (obj-data-indent :initform nil :initarg :obj-data-indent
                    :accessor obj-data-indent)
-   (obj-data-fmtstr :type (or string null) :initform nil :initarg :obj-data-fmtstr
-                   :accessor obj-data-fmtstr)
-   (obj-data-end-fmtstr :type (or string null) :initform nil
-                       :initarg :obj-data-end-fmtstr
-                       :accessor obj-data-end-fmtstr)
-   (obj-data-value-func :type (or function symbol null) :initform nil
-                       :initarg :obj-data-value-func
-                       :accessor obj-data-value-func)
+   (obj-data-printer :type (or function null) :initform nil
+                       :initarg :obj-data-printer
+                       :accessor obj-data-printer)
+   (obj-data-print-code :type (or function null) :initform nil
+                 :initarg :obj-data-print-code
+                 :accessor obj-data-print-code)
+   (obj-data-start-printer :type (or function string null) :initform nil
+                    :initarg :obj-data-start-printer
+                    :accessor obj-data-start-printer)
+   (obj-data-end-printer :type (or string null) :initform nil
+                       :initarg :obj-data-end-printer
+                       :accessor obj-data-end-printer)
+   (indenter :type (or function null) :initform nil
+            :accessor indenter
+            :documentation "Function that performs hierarchical indenting")
    (link-slots :type list :initform nil
               :documentation "List of slot names that have hyperlinks"
               :accessor link-slots)
-   (link-page-name :type (or string null) :initform nil :initarg :link-page-name
-                  :accessor link-page-name)
+   (link-page :type (or string null) :initform nil
+                     :initarg :link-page
+                     :accessor link-page)
    (link-href-start :type (or string null) :initform nil :initarg :link-href-start
                    :accessor link-href-start)
    (link-href-end :type (or string null) :initform nil :initarg :link-href-end
                  :accessor link-href-end)
    (link-ampersand :type (or string null) :initform nil :initarg :link-ampersand
                   :accessor link-ampersand))
-  (:default-initargs :link-page-name "disp-func1")
+  (:default-initargs :link-page "disp-func1")
   (:documentation "View class for a hyperobject"))
 
 
                                 :category :compact-text)))
        view))
     ((consp view-def)
-     (eval `(make-instance ,view-def)))
+     (apply #'make-instance 'object-view view-def))
     (t
      (error "Invalid parameter to make-object-view: ~S" view-def))))
 
     )
   )
 
+(defmacro write-simple (v s)
+  `(typecase ,v
+    (string
+     (write-string ,v ,s))
+    #+allegro
+    (fixnum
+     (excl::print-fixnum ,s 10 ,v)) 
+    (symbol
+     (write-string (symbol-name ,v) ,s))
+    (t
+     (write-string (write-to-string ,v) ,s))))
+
+(defun write-ho-value (obj name type formatter cdata strm)
+  (declare (ignorable type))
+  (let* ((slot-data (slot-value obj name))
+        (fmt-data (if formatter
+                      (funcall formatter slot-data)
+                      slot-data)))
+    (if cdata
+       (write-xml-cdata fmt-data strm)
+       (write-simple fmt-data strm))))
+
+(defun ppfc-html (title name type formatter cdata print-func)
+  (vector-push-extend '(write-string "<span class=\"" s) print-func)
+  (vector-push-extend `(write-string ,title s) print-func)
+  (vector-push-extend '(write-string "\">" s) print-func)
+  (vector-push-extend `(write-ho-value x ',name ',type ',formatter ,cdata s) print-func)
+  (vector-push-extend '(write-string "</span>" s) print-func))
+
+(defun ppfc-xml (tag name type formatter cdata print-func)
+  (vector-push-extend '(write-char #\< s) print-func)
+  (vector-push-extend `(write-string ,tag s) print-func)
+  (vector-push-extend '(write-char #\> s) print-func)
+  (vector-push-extend `(write-ho-value x ',name ',type ',formatter ,cdata s) print-func)
+  (vector-push-extend '(write-string "</" s) print-func)
+  (vector-push-extend `(write-string ,tag s) print-func)
+  (vector-push-extend '(write-char #\> s) print-func))
+
+(defun ppfc-html-labels (label name type formatter cdata print-func)
+  (vector-push-extend '(write-string "<span class=\"label\">" s) print-func)
+  (vector-push-extend `(write-string ,label s) print-func)
+  (vector-push-extend '(write-string "</span> " s) print-func)
+  (ppfc-html label name type formatter cdata print-func))
+
+(defun ppfc-xhtml-labels (label tag name type formatter cdata print-func)
+  (vector-push-extend '(write-string "<span class=\"label\">" s) print-func)
+  (vector-push-extend `(write-string ,label s) print-func)
+  (vector-push-extend '(write-string "</span> " s) print-func)
+  (ppfc-html tag name type formatter cdata print-func))
+
+(defun ppfc-xml-labels (label tag name type formatter cdata print-func)
+  (vector-push-extend '(write-string "<label>" s) print-func)
+  (vector-push-extend `(write-string ,label s) print-func)
+  (vector-push-extend '(write-string "</label> " s) print-func)
+  (ppfc-xml tag name type formatter cdata print-func))
+
+(defun ppfc-html-link (name type formatter cdata nlink print-func)
+  (declare (fixnum nlink))
+  (vector-push-extend '(write-char #\< s) print-func)
+  (vector-push-extend `(write-string (nth ,(+ nlink nlink) links) s) print-func) 
+  (vector-push-extend '(write-char #\> s) print-func)
+  (vector-push-extend `(write-ho-value x ',name ',type ',formatter ,cdata s) print-func)
+  (vector-push-extend '(write-string "</" s) print-func)
+  (vector-push-extend `(write-string (nth ,(+ nlink nlink 1) links) s) print-func) 
+  (vector-push-extend '(write-char #\> s) print-func))
+
+(defun ppfc-html-link-labels (label name type formatter cdata nlink print-func)
+  (vector-push-extend '(write-string "<span class=\"label\">" s) print-func)
+  (vector-push-extend `(write-string ,label s) print-func)
+  (vector-push-extend '(write-string "</span> " s) print-func)
+  (ppfc-html-link name type formatter cdata nlink print-func))
+
+(defun push-print-fun-code (category slot nlink print-func)
+  (let* ((formatter (esd-print-formatter slot))
+        (name (slot-definition-name slot))
+        (user-name (esd-user-name slot))
+        (xml-user-name (escape-xml-string user-name))
+        (xml-tag (escape-xml-string user-name))
+        (type (slot-value slot 'type))
+        (cdata (not (null
+                     (and (in category :xml :xhtml :xml-link :xhtml-link
+                              :xml-labels :ie-xml-labels
+                              :xhtml-link-labels :xml-link-labels :ie-xml-link
+                              :ie-xml-link-labels)
+                          (or formatter
+                              (lisp-type-is-a-string type))))))
+        (hyperlink (esd-hyperlink slot)))
+    
+    (case category
+      (:compact-text
+       (vector-push-extend
+       `(write-ho-value x ',name ',type ',formatter ,cdata s) print-func))
+      (:compact-text-labels
+       (vector-push-extend `(write-string ,user-name s) print-func)
+       (vector-push-extend '(write-char #\space s) print-func)
+       (vector-push-extend
+       `(write-ho-value x ',name ',type ',formatter ,cdata s) print-func))
+      ((or :html :xhtml)
+       (ppfc-html user-name name type formatter cdata print-func))
+      (:xml
+       (ppfc-xml xml-tag name type formatter cdata print-func))
+      (:html-labels
+       (ppfc-html-labels user-name name type formatter cdata print-func))
+      (:xhtml-labels
+       (ppfc-xhtml-labels xml-user-name user-name name type formatter cdata print-func))
+      (:xml-labels
+       (ppfc-xml-labels xml-user-name xml-tag name type formatter cdata print-func))
+      ((or :html-link :xhtml-link)
+       (if hyperlink
+          (ppfc-html-link name type formatter cdata nlink print-func)
+          (ppfc-html user-name name type formatter cdata print-func)))
+      ((or :xml-link :ie-xml-link)
+       (if hyperlink
+          (ppfc-html-link name type formatter cdata nlink print-func)
+          (ppfc-xml xml-tag name type formatter cdata print-func)))
+      (:html-link-labels
+       (if hyperlink
+          (ppfc-html-link-labels user-name name type formatter cdata nlink
+                                 print-func)
+          (ppfc-html-labels user-name name type formatter cdata print-func)))
+      (:xhtml-link-labels
+       (if hyperlink
+          (ppfc-html-link-labels xml-user-name name type formatter cdata nlink
+                                 print-func)
+          (ppfc-xhtml-labels xml-tag user-name name type formatter cdata
+                             print-func)))
+      ((or :xml-link-labels :ie-xml-link-labels)
+       (if hyperlink
+          (ppfc-html-link-labels xml-user-name name type formatter cdata nlink
+                                 print-func)
+          (ppfc-xml-labels xml-tag user-name name type formatter cdata
+                           print-func))))))
+
+
+(defun view-has-links-p (view)
+  (in (category view) :html-link :xhtml-link :xml-link :ie-xml-link
+      :html-link-labels :xhtml-link-labels :xml-link-labels
+      :ie-xml-link-labels))
 
 (defun initialize-view-by-category (obj-cl view)
   "Initialize a view based upon a preset category"
-  (let ((fmtstr nil)
-       (first-field t)
-       (value-func '())
-       (links '())
-       (category (category view)))
-
-    (unless (in category :compact-text :compact-text-labels
-               :html :html-labels :html-link-labels
-               :xhtml :xhtml-labels :xhtml-link-labels
-               :xml :xml-labels :xml-link :ie-xml-link
-               :xml-link-labels :ie-xml-link-labels)
-      (error "Unknown view category ~A" category))
-    
-    (unless (slots view)
-      (setf (slots view) (default-print-slots obj-cl)))
-    (dolist (slot-name (slots view))
-      (let ((slot (find-slot-by-name obj-cl slot-name)))
-       (unless slot
-         (error "Slot ~A is not found in class ~S" slot-name obj-cl))
-       (let* ((name (slot-definition-name slot))
-              (namestr-lower (string-downcase (symbol-name name)))
-              (xml-namestr (escape-xml-string namestr-lower))
-              (xml-tag (escape-xml-string namestr-lower))
-              (type (slot-value slot 'type))
-              (print-formatter (esd-print-formatter slot)))
-
-         (cond
-           (first-field
-            (setq fmtstr "")
-            (setq first-field nil))
-           (t
-            (string-append fmtstr " ")))
-
-         (let ((value-fmt
-                (case type
-                  ((or :integer :fixnum)
-                   "~d")
-                  (:boolean
-                   "~a")
-                  (otherwise
-                   "~a"))))
-           (case category
-             (:compact-text
-              (string-append fmtstr value-fmt))
-             (:compact-text-labels
-              (string-append fmtstr namestr-lower " " value-fmt))
-             ((or :html :xhtml)
-              (string-append fmtstr (concatenate 'string "<span class=\"" namestr-lower "\">" value-fmt "</span>")))
-             (:xml
-              (string-append fmtstr (concatenate 'string "<" namestr-lower ">" value-fmt "</" namestr-lower ">")))
-             (:html-labels
-              (string-append fmtstr (concatenate 'string "<span class=\"label\">" namestr-lower "</span> <span class=\"" namestr-lower "\">" value-fmt "</span>")))
-             (:xhtml-labels
-              (string-append fmtstr (concatenate 'string "<span class=\"label\">" xml-namestr "</span> <span class=\"" namestr-lower "\">" value-fmt "</span>")))
-             (:xml-labels
-              (string-append fmtstr (concatenate 'string "<label>" xml-namestr "</label> <" xml-tag ">" value-fmt "</" xml-tag ">")))
-             ((or :html-link :xhtml-link)
-              (push name links)
-              (if (esd-hyperlink slot)
-                  (string-append fmtstr "<~~a>" value-fmt "</~~a>")
-                  (string-append fmtstr (concatenate 'string "<span class=\"" namestr-lower "\">" value-fmt "</span>"))))
-             ((or :xml-link :ie-xml-link)
-              (push name links)
-              (if (esd-hyperlink slot)
-                  (string-append fmtstr "<~~a>" value-fmt "</~~a>")
-                  (string-append fmtstr (concatenate 'string "<" xml-tag ">" value-fmt "</" xml-tag ">"))))
-             (:html-link-labels
-              (push name links)
-              (if (esd-hyperlink slot)
-                  (string-append fmtstr "<span class=\"label\">" namestr-lower "</span> <~~a>" value-fmt "</~~a>")
-                  (string-append fmtstr (concatenate 'string "<span class=\"label\">" namestr-lower "</span> <span class=\"" namestr-lower "\">" value-fmt "</span>"))))
-             (:xhtml-link-labels
-              (push name links)
-              (if (esd-hyperlink slot)
-                  (string-append fmtstr "<span class=\"label\">" xml-namestr "</span> <~~a>" value-fmt "</~~a>")
-                  (string-append fmtstr (concatenate 'string "<span class=\"label\">" xml-namestr "</span> <span class=\"" namestr-lower "\">" value-fmt "</span>"))))
-             ((or :xml-link-labels :ie-xml-link-labels)
-              (push name links)
-              (if (esd-hyperlink slot)
-                  (string-append fmtstr "<label>" xml-namestr "</label> <~~a>" value-fmt "</~~a>")
-                  (string-append fmtstr (concatenate 'string "<label>" xml-namestr "</label> <" xml-tag ">" value-fmt "</" xml-tag ">")))))
-           ) ;; let value-fmt
-           
-         (let ((func (if print-formatter
-                 `(,print-formatter (slot-value x (quote ,name)))
-                 `(slot-value x (quote ,name)))))
-           (when (and (in category :xml :xhtml :xml-link :xhtml-link
-                          :xml-labels :ie-xml-labels
-                          :xhtml-link-labels :xml-link-labels :ie-xml-link
-                          :ie-xml-link-labels)
-                      (or print-formatter
-                          (lisp-type-is-a-string type)))
-             (setq func `(kmrcl:xml-cdata ,func)))
-           (push func value-func))
-         )))
-         
-    (when value-func
-      (setq value-func
-           (compile nil (eval `(lambda (x) (values ,@(nreverse value-func)))))))
-
-    (setf (obj-data-fmtstr view) fmtstr)
-    (setf (obj-data-value-func view) value-func)
-    (setf (link-slots view) (nreverse links))
+  (unless (in (category view) :compact-text :compact-text-labels
+             :html :html-labels :html-link-labels
+             :xhtml :xhtml-labels :xhtml-link-labels
+             :xhtml-link :html-link
+             :xml :xml-labels :xml-link :ie-xml-link
+             :xml-link-labels :ie-xml-link-labels)
+    (error "Unknown view category ~A" (category view)))
+  
+  (unless (slots view) (setf (slots view) (default-print-slots obj-cl)))
+
+  (let ((links '())
+       (print-func (make-array 10 :fill-pointer 0 :adjustable t)))
+
+    (do* ((slots (slots view) (cdr slots))
+         (slot-name (car slots) (car slots))
+         (slot (find-slot-by-name obj-cl slot-name)
+               (find-slot-by-name obj-cl slot-name)))
+        ((null slots))
+      (unless slot
+       (error "Slot ~A is not found in class ~S" slot-name obj-cl))
+      
+      (push-print-fun-code (category view) slot (length links) print-func)
+      (when (> (length slots) 1)
+       (vector-push-extend '(write-char #\space s) print-func))
+
+      (when (and (view-has-links-p view) (esd-hyperlink slot))
+       (push (slot-definition-name slot) links)))
+
+    (vector-push-extend 'x print-func) ;; return object
+    (setf (obj-data-print-code view) `(lambda (x s links)
+                                      (declare (ignorable s links))
+                                      ,@(map 'list #'identity print-func)))
+    (setf (obj-data-printer view)
+         (compile nil (eval (obj-data-print-code view))))
     
-    (case category
-      ((or :compact-text :compact-text-labels)
-       (initialize-text-view view))
-      ((or :html :xhtml :html-labels :xhtml-labels)
-       (initialize-html-view view))
-      ((or :xml :xml-labels)
-       (initialize-xml-view view))
-      ((or :html-link :html-link-labels)
-       (initialize-html-view view)
-       (setf (link-href-start view) "a href=")
-       (setf (link-href-end view) "a")
-       (setf (link-ampersand view) "&"))
-      ((or :xhtml-link :xhtml-link-labels)
-       (initialize-html-view view)
-       (setf (link-href-start view) "a href=")
-       (setf (link-href-end view) "a")
-       (setf (link-ampersand view) "&amp;"))
-      ((or :xml-link :xml-link-labels)
-       (initialize-xml-view view)
-       (setf (link-href-start view)
-            "xmllink xlink:type=\"simple\" xlink:href=")
-       (setf (link-href-end view) "xmllink")
-       (setf (link-ampersand view) "&amp;"))
-      ((or :ie-xml-link :ie-xml-link-labels)
-       (initialize-xml-view view)
-       (setf (link-href-start view) "html:a href=")
-       (setf (link-href-end view) "html:a")
-       (setf (link-ampersand view) "&amp;"))))
+    (setf (link-slots view) (nreverse links)))
+
+  (finalize-view-by-category view)
   view)
 
+(defun finalize-view-by-category (view)
+  (case (category view)
+    ((or :compact-text :compact-text-labels)
+     (initialize-text-view view))
+    ((or :html :html-labels)
+     (initialize-html-view view))
+    ((or :xhtml :xhtml-labels)
+     (initialize-xhtml-view view))
+    ((or :xml :xml-labels)
+     (initialize-xml-view view))
+    ((or :html-link :html-link-labels)
+     (initialize-html-view view)
+     (setf (link-href-start view) "a href=")
+     (setf (link-href-end view) "a")
+     (setf (link-ampersand view) "&"))
+    ((or :xhtml-link :xhtml-link-labels)
+     (initialize-xhtml-view view)
+     (setf (link-href-start view) "a href=")
+     (setf (link-href-end view) "a")
+     (setf (link-ampersand view) "&amp;"))
+    ((or :xml-link :xml-link-labels)
+     (initialize-xml-view view)
+     (setf (link-href-start view)
+          "xmllink xlink:type=\"simple\" xlink:href=")
+     (setf (link-href-end view) "xmllink")
+     (setf (link-ampersand view) "&amp;"))
+    ((or :ie-xml-link :ie-xml-link-labels)
+     (initialize-xml-view view)
+     (setf (link-href-start view) "html:a href=")
+     (setf (link-href-end view) "html:a")
+     (setf (link-ampersand view) "&amp;"))))
+
 
 ;;;; *************************************************************************
 ;;;;  View Data Format Section
 (defun class-name-of (obj)
   (string-downcase (class-name (class-of obj))))
 
-(defun text-list-start-value-func (obj nitems)
-  (values (hyperobject-class-user-name obj) nitems))
+(defvar +newline-string+ (format nil "~%"))
 
-(defun htmlformat-list-start-value-func (x nitems) 
-  (values (hyperobject-class-user-name x) nitems (class-name-of x)))
+(defun write-user-name-maybe-plural (obj nitems strm)
+  (write-string
+   (if (> nitems 1)
+       (hyperobject-class-user-name-plural obj)
+       (hyperobject-class-user-name obj))
+   strm))
 
 (defun initialize-text-view (view)
-  (setf (list-start-fmtstr view) "~a~P:~%")
-  (setf (list-start-value-func view) #'text-list-start-value-func)
+  (setf (list-start-printer view)
+       (compile nil
+                (eval '(lambda (obj nitems indent strm)
+                        (write-user-name-maybe-plural obj nitems strm)
+                        (write-char #\: strm)
+                        (write-char #\Newline strm)))))
   (setf (list-start-indent view) t)
   (setf (obj-data-indent view) t)
-  (setf (obj-data-end-fmtstr view) (format nil "~%"))
-  )
+  (setf (obj-data-end-printer view) +newline-string+)
+  (setf (indenter view) #'indent-spaces))
+
+(defun html-list-start-func (obj nitems indent strm)
+  (write-string "<div class=\"ho-username\">" strm)
+  (write-user-name-maybe-plural obj nitems strm)
+  (write-string "</div>" strm)
+  (write-char #\newline strm)
+  (write-string "<ul>" strm)
+  (write-char #\newline strm))
 
 (defun initialize-html-view (view)
   (initialize-text-view view)
+  (setf (indenter view) #'indent-html-spaces)
   (setf (file-start-str view) (format nil "<html><body>~%"))
   (setf (file-end-str view) (format nil "</body><html>~%"))
   (setf (list-start-indent view) t)
-  (setf (list-start-fmtstr view)
-       "<p><b>~a~p:</b></p><div class=\"~A\"><ul>~%")
-  (setf (list-start-value-func view)
-       #'htmlformat-list-start-value-func)
-  (setf (list-end-fmtstr view) (format nil "</ul></div>~%"))
+  (setf (list-start-printer view) #'html-list-start-func)
+  (setf (list-end-printer view) (format nil "</ul>~%"))
   (setf (list-end-indent view) t)
-  (setf (list-end-value-func view) nil)
-  (setf (obj-start-indent view) t)
-  (setf (obj-start-fmtstr view) "<li>")
-  (setf (obj-start-value-func view) nil)
-  (setf (obj-end-indent view)  t)
-  (setf (obj-end-fmtstr view)  (format nil "</li>~%"))
-  (setf (obj-end-value-func view) nil)
-  (setf (obj-data-indent view) t))
+  (setf (obj-start-indent view) nil)
+  (setf (obj-start-printer view) "<li>")
+  (setf (obj-end-indent view)  nil)
+  (setf (obj-end-printer view)  (format nil "</li>~%"))
+  (setf (obj-data-end-printer view) nil)
+  (setf (obj-data-indent view) nil))
+
+(defun xhtml-list-start-func (obj nitems indent strm)
+  (write-string "<div class=\"ho-username\">" strm)
+  (indent-html-spaces indent strm)
+  (write-user-name-maybe-plural obj nitems strm)
+  (write-string "</div>" strm)
+  (write-char #\newline strm))
 
 (defun initialize-xhtml-view (view)
   (initialize-text-view view)
+  (setf (indenter view) #'indent-html-spaces)
   (setf (file-start-str view) (format nil "<html><body>~%"))
   (setf (file-end-str view) (format nil "</body><html>~%"))
-  (setf (list-start-indent view) t)
-  (setf (list-start-fmtstr view)
-       "<p><b>~a~p:</b></p><div class=\"~A\"><ul>~%")
-  (setf (list-start-value-func view)
-                   #'htmlformat-list-start-value-func)
-  (setf (list-end-fmtstr view) (format nil "</ul></div>~%"))
-  (setf (list-end-indent view) t)
-  (setf (list-end-value-func view) nil)
-  (setf (obj-start-indent view) t)
-  (setf (obj-start-fmtstr view) "<li>")
-  (setf (obj-start-value-func view) nil)
-  (setf (obj-end-indent view)  t)
-  (setf (obj-end-fmtstr view) (format nil "</li>~%"))
-  (setf (obj-end-value-func view) nil)
+  (setf (list-start-indent view) nil)
+  (setf (list-start-printer view) #'xhtml-list-start-func)
+  (setf (list-end-printer view) (format nil "~%"))
+  (setf (list-end-indent view) nil)
+  (setf (obj-start-indent view) nil)
+  (setf (obj-start-printer view) nil)
+  (setf (obj-end-printer view) (format nil "</div>~%"))
+  (setf (obj-data-start-printer view) "<div>")
+  (setf (obj-data-end-printer view) nil)
+  (setf (obj-end-indent view)  nil)
   (setf (obj-data-indent view) t))
 
-(defun xmlformat-list-end-value-func (x)
-  (format nil "~alist" (class-name-of x)))
-
-(defun xmlformat-list-start-value-func (x nitems) 
-  (values (format nil "~alist" (class-name-of x)) (hyperobject-class-user-name x) nitems))
+(defun xmlformat-list-end-func (x strm)
+  (write-string "</" strm)
+  (write-string (class-name-of x) strm)
+  (write-string "list" strm)
+  (write-string ">" strm)
+  (write-char #\newline strm))
+
+(defun xmlformat-list-start-func (x nitems indent strm)
+  (write-char #\< strm)
+  (write-string (class-name-of x) strm)
+  (write-string "list><title>" strm)
+  (write-user-name-maybe-plural x nitems strm)
+  (write-string ":</title>" strm)
+  (write-char #\newline strm))
 
 (defun initialize-xml-view (view)
   (initialize-text-view view)
   (setf (file-start-str view) "") ; (std-xml-header)
   (setf (list-start-indent view)  t)
-  (setf (list-start-fmtstr view) "<~a><title>~a~p:</title> ~%")
-  (setf (list-start-value-func view)
-       #'xmlformat-list-start-value-func)
+  (setf (list-start-printer view) #'xmlformat-list-start-func)
   (setf (list-end-indent view) t)
-  (setf (list-end-fmtstr view) "</~a>~%")
-  (setf (list-end-value-func view) #'xmlformat-list-end-value-func)
-  (setf (obj-start-fmtstr view) (format nil "<~(~a~)>" (object-class-name view)))
+  (setf (list-end-printer view) #'xmlformat-list-end-func)
+  (setf (obj-start-printer view) (format nil "<~(~a~)>" (object-class-name view)))
   (setf (obj-start-indent view) t)
-  (setf (obj-end-fmtstr view) (format nil "</~(~a~)>~%" (object-class-name view)))
-  (setf (obj-end-indent view) nil)
+  (setf (subobj-end-printer view) (format nil "</~(~a~)>~%" (object-class-name view)))
+  (setf (subobj-end-indent view) nil)
   (setf (obj-data-indent view) nil))
 
 
 
 (defun fmt-list-start (obj view strm indent num-items)
   (when (list-start-indent view)
-    (indent-spaces indent strm))
-  (let-when (fmtstr (list-start-fmtstr view))
-           (let-if (value-func (list-start-value-func view))
-                   (multiple-value-call #'format strm fmtstr
-                                        (funcall value-func obj num-items))
-                   (write-string fmtstr strm))))
+    (awhen (indenter view)
+          (funcall it indent strm)))
+  (awhen (list-start-printer view)
+        (if (stringp it)
+            (write-string it strm)
+            (funcall it obj num-items indent strm))))
 
 (defun fmt-list-end (obj view strm indent num-items)
   (declare (ignore num-items))
   (when (list-end-indent view)
-      (indent-spaces indent strm))
-  (let-when (fmtstr (list-end-fmtstr view))
-           (let-if (value-func (list-end-value-func view))
-                   (multiple-value-call #'format strm fmtstr
-                                        (funcall value-func obj))
-                   (write-string fmtstr strm))))
+    (awhen (indenter view)
+          (funcall it indent strm)))
+  (awhen (list-end-printer view)
+        (if (stringp it)
+            (write-string it strm)
+            (funcall it obj strm))))
 
 ;;; Object Start and Ends
 
+
 (defun fmt-obj-start (obj view strm indent)
   (when (obj-start-indent view)
-    (indent-spaces indent strm))
-  (let-when (fmtstr (obj-start-fmtstr view))
-           (let-if (value-func (obj-start-value-func view))
-                   (multiple-value-call #'format strm fmtstr
-                                        (funcall value-func obj))
-                   (write-string fmtstr strm))))
+    (awhen (indenter view)
+          (funcall it indent strm)))
+  (awhen (obj-start-printer view)
+        (if (stringp it)
+            (write-string it strm)
+            (funcall it obj strm))))
 
 (defun fmt-obj-end (obj view strm indent)
   (when (obj-end-indent view)
-    (indent-spaces indent strm))
-  (let-when (fmtstr (obj-end-fmtstr view))
-           (let-if (value-func (obj-end-value-func view))
-                   (multiple-value-call #'format strm fmtstr 
-                                        (funcall value-func obj))
-                   (write-string fmtstr strm))))
+    (awhen (indenter view)
+          (funcall it indent strm))) 
+  (awhen (obj-end-printer view)
+        (if (stringp it)
+            (write-string it strm)
+            (funcall it obj strm))))
+
+(defun fmt-subobj-start (obj view strm indent)
+  (when (subobj-start-indent view)
+    (awhen (indenter view)
+          (funcall it indent strm)))
+  (awhen (subobj-start-printer view)
+        (if (stringp it)
+            (write-string it strm)
+            (funcall it obj strm))))
+
+(defun fmt-subobj-end (obj view strm indent)
+  (when (subobj-end-indent view)
+    (awhen (indenter view)
+          (funcall it indent strm))) 
+  (awhen (subobj-end-printer view)
+        (if (stringp it)
+            (write-string it strm)
+            (funcall it obj strm))))
   
 ;;; Object Data 
 
-(defun make-link-start (view fieldfunc fieldvalue refvars)
-  (format nil "~a\"~a?func=~a~akey=~a~a\"" 
-         (link-href-start view)
-         (make-url (link-page-name view))
-         fieldfunc 
-         (link-ampersand view) fieldvalue
-         (if refvars
-             (let ((varstr ""))
-               (dolist (var refvars)
-                 (string-append
-                  varstr (link-ampersand view)
-                  (format nil "~a=~a" (car var) (cadr var))))
-               varstr)
-             "")))
 
+(defun make-link-start (view fieldfunc fieldvalue refvars link-printer)
+  (with-output-to-string (s)
+    (write-string (link-href-start view) s)
+    (write-char #\" s)
+    (let ((link-page (link-page view)))
+      (cond
+       ((null link-printer)
+        (write-string (make-url link-page) s)
+        (write-string "?func=" s)
+        (write-simple fieldfunc s)
+        (write-string (link-ampersand view) s)
+        (write-string "key=" s)
+        (write-simple fieldvalue s)
+        (dolist (var refvars)
+          (write-string (link-ampersand view) s)
+          (write-simple (car var) s)
+          (write-char #\= s)
+          (write-simple (cdr var) s)))
+       (link-printer
+        (funcall link-printer link-page fieldfunc fieldvalue refvars s))))
+    (write-char #\" s)))
+  
 (defun make-link-end (obj view fieldname)
   (declare (ignore obj fieldname))
   (link-href-end view))
 
-(defun fmt-obj-data (obj view strm indent refvars)
+(defun fmt-obj-data (obj view strm indent refvars link-printer)
+  (awhen (obj-data-start-printer view)
+        (if (stringp it)
+            (write-string it strm)
+            (funcall it obj strm)))
   (when (obj-data-indent view)
-    (indent-spaces indent strm))
+    (awhen (indenter view)
+          (funcall it indent strm)))
   (if (link-slots view)
-      (fmt-obj-data-with-link obj view strm refvars)
+      (fmt-obj-data-with-link obj view strm refvars link-printer)
       (fmt-obj-data-plain obj view strm))
-  (awhen (obj-data-end-fmtstr view)
-        (write-string it strm)))
+  (awhen (obj-data-end-printer view)
+        (if (stringp it)
+            (write-string it strm)
+            (funcall it obj strm))))
 
 (defun fmt-obj-data-plain (obj view strm)
-  (awhen (obj-data-value-func view)
-        (multiple-value-call #'format strm (obj-data-fmtstr view)
-                             (funcall it obj))))
+  (awhen (obj-data-printer view)
+        (funcall it obj strm nil)))
 
-(defun fmt-obj-data-with-link (obj view strm refvars)
+(defun fmt-obj-data-with-link (obj view strm refvars link-printer)
   (let ((refvalues '()))
+    (declare (dynamic-extent refvalues))
     ;; make list of hyperlink link fields for printing to refstr template
     (dolist (name (link-slots view))
-      (let-when (hyperlink
-                (find name (hyperobject-class-hyperlinks obj) :key #'name))
-               (push  (make-link-start view (lookup hyperlink) (slot-value obj name)
-                                       (append (link-parameters hyperlink) refvars))
-                      refvalues)
-               (push (make-link-end obj view name) refvalues)))
-    (setq refvalues (nreverse refvalues))
-    (apply #'format strm (make-link-data-str obj view) refvalues)))
+      (awhen (find name (hyperobject-class-hyperlinks obj) :key #'name)
+            (push (make-link-start view (lookup it) (slot-value obj name)
+                                   (append (link-parameters it) refvars)
+                                   link-printer)
+                  refvalues)
+            (push (make-link-end obj view name) refvalues)))
+    (funcall (obj-data-printer view) obj strm (nreverse refvalues))))
 
 (defun obj-data (obj view)
   "Returns the objects data as a string. Used by common-graphics outline function"
-  (awhen (obj-data-value-func view)
-        (multiple-value-call #'format nil (funcall (obj-data-fmtstr view))
-                             (funcall it obj))))
-
-(defun make-link-data-str (obj view)
-  "Return fmt string for that contains ~a slots for hyperlink link start and end"
-  (awhen (obj-data-value-func view)
-        (multiple-value-call #'format nil (obj-data-fmtstr view)
-                             (funcall it obj))))
+  (with-output-to-string (s) (fmt-obj-data-plain obj view s)))
 
 ;;; Display method for objects
 
   objs)
 
 (defun view-hyperobject (objs view category strm &optional (indent 0) filter
-                        subobjects refvars)
+                        subobjects refvars link-printer)
   "Display a single or list of hyperobject-class instances and their subobjects"
   (let-when (objlist (mklist objs))
     (let ((nobjs (length objlist))
       (dolist (obj objlist)
         (unless (and filter (not (funcall filter obj)))
           (fmt-obj-start obj view strm indent)
-          (fmt-obj-data obj view strm (1+ indent) refvars)
+          (fmt-obj-data obj view strm (1+ indent) refvars link-printer)
+          (fmt-obj-end obj view strm indent)
+          (fmt-subobj-start obj view strm indent)
           (when (and subobjects (hyperobject-class-subobjects obj))
            (dolist (subobj (hyperobject-class-subobjects obj))
              (aif (slot-value obj (name-slot subobj))
-                  (view-hyperobject it (get-category-view (car (mklist it)) category)
+                  (view-hyperobject it
+                                    (get-category-view (car (mklist it))
+                                                       category)
                                     category strm (1+ indent) filter
-                                    subobjects refvars))))
-         (fmt-obj-end obj view strm indent)))
+                                    subobjects refvars link-printer))))
+         (fmt-subobj-end obj view strm indent)))
       (fmt-list-end (car objlist) view strm indent nobjs)))
   objs)
 
 
 (defun view (objs &key (stream *standard-output*) category view
-            filter subobjects refvars file-wrapper)
-  "EXPORTED Function: prints hyperobject-class objects. Simplies call to view-hyperobject"
+            filter subobjects refvars file-wrapper link-printer)
+  "EXPORTED Function: prints hyperobject-class objects. Calls view-hyperobject"
   (let-when (objlist (mklist objs))
     (when category
       (setq view (get-category-view (car objlist) category)))
       (setq view (default-view (class-of (car objlist)))))
     (when file-wrapper
       (fmt-file-start view stream))
-    (view-hyperobject objlist view category stream 0 filter subobjects refvars)
+    (view-hyperobject objlist view category stream 0 filter subobjects refvars
+                     link-printer)
     (when file-wrapper
       (fmt-file-end view stream)))
   objs)