r8263: rename entities.xml to entities.inc
[hyperobject.git] / mop.lisp
index f7f7ae02ed5d0c287197c21f80ae5ad15f686800..5307f3ed545f07ed60be74ca9ee88378f1ab2cce 100644 (file)
--- a/mop.lisp
+++ b/mop.lisp
 ;;;; in Text, HTML, and XML formats. This includes hyperlinking
 ;;;; capability and sub-objects.
 ;;;;
-;;;; $Id: mop.lisp,v 1.6 2002/12/05 19:15:02 kevin Exp $
-;;;;
-;;;; This file is Copyright (c) 2000-2002 by Kevin M. Rosenberg
+;;;; $Id$
 ;;;;
+;;;; 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)
 
 ;; Main class
 
 (defclass hyperobject-class (standard-class)
   ( ;; slots initialized in defclass
-   (title :initarg :title :type string :initform nil
-         :documentation "Print Title for class")
-   (print-slots :initarg :print-slots :type list :initform nil
-               :documentation "List of slots to print")
+   (user-name :initarg :user-name :type string :initform nil
+             :accessor user-name
+             :documentation "User name for class")
+   (user-name-plural :initarg :user-name-plural :type string :initform nil
+                    :accessor user-name-plural
+                    :documentation "Plural user name for class")
+   (default-print-slots :initarg :default-print-slots :type list :initform nil
+                       :accessor default-print-slots
+                       :documentation "Defaults slots for a view")
    (description :initarg :description :initform nil
+               :accessor description
                :documentation "Class description")
    (version :initarg :version :initform nil
-               :documentation "Version number for class")
-   (sql-name :initarg :table-name :initform nil :reader sql-name)
+           :accessor version
+           :documentation "Version number for class")
+   (closures :initarg :closures :initform nil
+            :accessor closures
+            :documentation "Closures to call on slot chnages")
+   (sql-name :initarg :sql-name :accessor sql-name :initform nil
+            :documentation "SQL Name for this class")
+   (guid :initarg :guid :accessor guid :initform nil
+        :documentation "ID string for this class")
 
    ;;; The remainder of these fields are calculated one time
    ;;; in finalize-inheritence.
    
-   (subobjects :initform nil :documentation
+   (subobjects :initform nil :accessor subobjects
+              :documentation
               "List of fields that contain a list of subobjects objects.")
-   (hyperlinks :type list :initform nil :documentation 
-              "List of fields that have hyperlinks")
-   (class-id :type integer :initform nil :documentation
-            "Unique ID for the class")
+   (hyperlinks :type list :initform nil :accessor hyperlinks
+              :documentation "List of fields that have hyperlinks")
+   (direct-rules :type list :initform nil :initarg :direct-rules
+                :accessor direct-rules
+                :documentation "List of rules to fire on slot changes.")
+   (direct-views :type list :initform nil :initarg :direct-views
+                :accessor direct-views
+                :documentation "List of views")
+   (class-id :type integer :initform nil
+            :accessor class-id
+            :documentation "Unique ID for the class")
+   (default-view :initform nil :initarg :default-view :accessor default-view
+                :documentation "The default view for a class")
+   (documementation :initform nil :initarg :documentation
+                   :documentation "Documentation string for hyperclass.")
    
+   ;; SQL commands
    (create-table-cmd :initform nil :reader create-table-cmd)
-   (create-index-cmds :initform nil :reader create-index-cmds)
+   (create-indices-cmds :initform nil :reader create-index-cmds)
    (drop-table-cmd :initform nil :reader drop-table-cmd)
 
-   (value-func :initform nil :type function)
-   (xmlvalue-func :initform nil :type function)
-   (fmtstr-text :initform nil :type string)
-   (fmtstr-html :initform nil :type string)
-   (fmtstr-xml :initform nil :type string)
-   (fmtstr-text-labels :initform nil :type string)
-   (fmtstr-html-labels :initform nil :type string)
-   (fmtstr-xml-labels :initform nil :type string)
-   (fmtstr-html-ref :initform nil :type string)
-   (fmtstr-xml-ref :initform nil :type string)
-   (fmtstr-html-ref-labels :initform nil :type string)
-   (fmtstr-xml-ref-labels :initform nil :type string)
+   (views :type list :initform nil :initarg :views :accessor views
+         :documentation "List of views")
+   (rules :type list :initform nil :initarg :rules :accessor rules
+         :documentation "List of rules")
    )
   (:documentation "Metaclass for Markup Language classes."))
 
 (defclass subobject ()
-  ((name :type symbol :initform nil :initarg :name :reader name)
-   (reader :type function :initform nil :initarg :reader :reader reader)))
+  ((name-class :type symbol :initarg :name-class :reader name-class)
+   (name-slot :type symbol :initarg :name-slot :reader name-slot)
+   (subobj-class :type symbol :initarg :subobj-class :reader subobj-class)
+   (lookup :type (or function symbol) :initarg :lookup :reader lookup)
+   (lookup-keys :type list :initarg :lookup-keys :reader lookup-keys))
+  (:documentation "subobject information")
+  (:default-initargs :name-class nil :name-slot nil :subobj-class nil
+                    :lookup nil :lookup-keys nil))
+
 
 (defmethod print-object ((obj subobject) (s stream))
-  (print-unreadable-object (obj s :type t :identity t)
-    (format s "~S" (name obj))))
+  (print-unreadable-object (obj s :type t)
+    (format s "~S" (name-slot obj))))
 
 (defclass hyperlink ()
   ((name :type symbol :initform nil :initarg :name :reader name)
-   (lookup :type function :initform nil :initarg :lookup :reader lookup)
+   (lookup
+    ;; The type specifier seems to break sbcl
+    :type (or function symbol)
+    ;;    :type t
+    :initform nil :initarg :lookup :reader lookup)
    (link-parameters :type list :initform nil :initarg :link-parameters
                    :reader link-parameters)))
 
   (print-unreadable-object (obj s :type t :identity t)
     (format s "~S" (name obj))))
 
-#+(or cmu scl sbcl)
 (defmethod validate-superclass ((class hyperobject-class) (superclass standard-class))
   t)
 
 (defmethod finalize-inheritance :after ((cl hyperobject-class))
-  (init-hyperobject-class cl))
+  ;; Work-around needed for OpenMCL
+  #+ignore
+  (unless (find-class (class-name cl))
+    (setf (find-class (class-name cl)) cl))
+  
+  (init-hyperobject-class cl)
+  )
+
+(eval-when (:compile-toplevel :load-toplevel :execute)
+  (when (>= (length (generic-function-lambda-list
+                    (ensure-generic-function
+                     'compute-effective-slot-definition)))
+           3)
+    (pushnew :ho-normal-cesd cl:*features*))
+  
+    (when (>= (length (generic-function-lambda-list
+                      (ensure-generic-function
+                       'direct-slot-definition-class)))
+           3)
+      (pushnew :ho-normal-dsdc cl:*features*))
+    
+    (when (>= (length (generic-function-lambda-list
+                      (ensure-generic-function
+                       'effective-slot-definition-class)))
+             3)
+      (pushnew :ho-normal-esdc cl:*features*)))
 
 ;; Slot definitions
-(defmethod direct-slot-definition-class ((cl hyperobject-class) 
-                                        #+allegro &rest
-                                                  iargs)
+(defmethod direct-slot-definition-class ((cl hyperobject-class)
+                                        #+ho-normal-dsdc &rest iargs)
   (find-class 'hyperobject-dsd))
 
+(defmethod effective-slot-definition-class ((cl hyperobject-class) 
+                                           #+ho-normal-esdc &rest iargs)
+  (find-class 'hyperobject-esd))
+
 
-; Slot definitions
+;;; Slot definitions
 
 (eval-when (:compile-toplevel :load-toplevel :execute)
   (defmacro process-class-option (slot-name &optional required)
     `(defmethod clos:process-a-class-option ((class hyperobject-class)
                                             (name (eql ,slot-name))
                                             value)
-    (when (and ,required (null value))
-      (error "hyperobject class slot ~A must have a value" name))
-    (list name `',value))
-    #+(or allegro sbcl cmu scl)
+      (when (and ,required (null value))
+       (error "hyperobject class slot ~A must have a value" name))
+      (list name `',value))
+    #+(or allegro sbcl cmu scl openmcl)
     (declare (ignore slot-name required))
     )
 
     `(defmethod clos:process-a-slot-option ((class hyperobject-class)
                                            (option (eql ,slot-name))
                                            value
-                                         already-processed-other-options
+                                           already-processed-options
                                            slot)
-       (list option `',value))
+      (list* option `',value already-processed-options))
     #-lispworks
     (declare (ignore slot-name))
     )
 
   (eval
    `(defclass hyperobject-dsd (standard-direct-slot-definition)
-      (,@(mapcar #'(lambda (x)
-                    `(,(intern (symbol-name x))
-                       :initform nil))
-                *slot-options-no-initarg*)
-        ,@(mapcar #'(lambda (x)
-                      `(,(intern (symbol-name x))
-                         :initarg
-                         ,(intern (symbol-name x) (symbol-name :keyword))
-                         :initform nil))
-                  *slot-options*))))
+     (,@(mapcar #'(lambda (x)
+                   `(,(intern (symbol-name x))
+                     :initform nil))
+               *slot-options-no-initarg*)
+      ,@(mapcar #'(lambda (x)
+                   `(,(intern (symbol-name x))
+                     :initarg
+                     ,(intern (symbol-name x) (symbol-name :keyword))
+                     :initform nil
+                     :accessor
+                     ,(intern (concatenate 'string
+                                           (symbol-name :dsd-)
+                                           (symbol-name x)))))
+               *slot-options*))))
   (eval
    `(defclass hyperobject-esd (standard-effective-slot-definition)
-      (,@(mapcar #'(lambda (x)
-                    `(,(intern (symbol-name x))
-                       :initarg
-                       ,(intern (symbol-name x) (symbol-name :keyword))
-                       :initform nil))
-                 (append *slot-options* *slot-options-no-initarg*)))))
+     (,@(mapcar #'(lambda (x)
+                   `(,(intern (symbol-name x))
+                     :initarg
+                     ,(intern (symbol-name x) (symbol-name :keyword))
+                     :initform nil
+                     :accessor
+                     ,(intern (concatenate 'string
+                                           (symbol-name :esd-)
+                                           (symbol-name x)))))
+               (append *slot-options* *slot-options-no-initarg*)))))
   ) ;; eval-when
-  
-(defmethod compute-effective-slot-definition :around
-    ((cl hyperobject-class) #+(or allegro lispworks) name dsds)
-  #+allergo (declare (ignore name))
+
+(defun intern-in-keyword (obj)
+  (cond
+    ((null obj)
+     nil)
+    ((eq t obj)
+     t)
+    ((atom obj)
+     (intern (symbol-name obj) (find-package 'keyword)))
+    ((consp obj)
+     (cons (intern-in-keyword (car obj) ) (intern-in-keyword (cdr obj))))
+    (t
+     obj)))
+
+(defun canonicalize-value-type (vt)
+  (typecase vt
+    (atom
+     (ensure-keyword vt))
+    (cons
+     (list (ensure-keyword (car vt)) (cadr vt)))
+    (t
+     t)))
+
+(defmethod compute-effective-slot-definition :around ((cl hyperobject-class)
+                                                     #+ho-normal-cesd name
+                                                     dsds)
+  (declare (ignore #+ho-normal-cesd name))
+  (let ((esd (call-next-method)))
+    (if (typep esd 'hyperobject-esd)
+       (compute-hyperobject-esd esd dsds)
+       esd)))
+
+(defun compute-hyperobject-esd (esd dsds)
   (let* ((dsd (car dsds))
-        (ho-type (slot-value dsd 'type))
-        (sql-type (ho-type-to-sql-type ho-type)))
-    (setf (slot-value dsd 'ho-type) ho-type)
-    (setf (slot-value dsd 'sql-type) sql-type)
-    (setf (slot-value dsd 'type) (ho-type-to-lisp-type ho-type))
-    (let ((ia (compute-effective-slot-definition-initargs
-              cl #+lispworks name dsds)))
-      (apply
-       #'make-instance 'hyperobject-esd 
-       :ho-type ho-type
-       :sql-type sql-type
-       :print-formatter (slot-value dsd 'print-formatter)
-       :subobject (slot-value dsd 'subobject)
-       :hyperlink (slot-value dsd 'hyperlink)
-       :hyperlink-parameters (slot-value dsd 'hyperlink-parameters)
-       :description (slot-value dsd 'description)
-       ia))))
-
-(defun ho-type-to-lisp-type (ho-type)
-  (check-type ho-type symbol)
-  (case (intern (symbol-name ho-type) (symbol-name :keyword))
-    (:string
-     'string)
+        (value-type (canonicalize-value-type (slot-value dsd 'value-type))))
+    (multiple-value-bind (sql-type sql-length) 
+       (value-type-to-sql-type value-type)
+      (setf (esd-sql-type esd) sql-type)
+      (setf (esd-sql-length esd) sql-length))
+    (setf (slot-value esd 'type) (value-type-to-lisp-type value-type))
+    (setf (esd-value-type esd) value-type)
+    (setf (esd-user-name esd)
+         (aif (dsd-user-name dsd)
+              it
+              (string-downcase (symbol-name (slot-definition-name dsd)))))
+    (setf (esd-sql-name esd)
+         (aif (dsd-sql-name dsd)
+              it
+              (lisp-name-to-sql-name (slot-definition-name dsd))))
+    (setf (esd-sql-name esd)
+         (aif (dsd-sql-name dsd)
+              it
+              (lisp-name-to-sql-name (slot-definition-name dsd))))
+    (dolist (name '(print-formatter subobject hyperlink hyperlink-parameters
+                   description value-constraint indexed null-allowed
+                   unique short-description void-text read-only-groups
+                   hidden-groups unit disable-predicate view-type
+                   list-of-values stored))
+      (setf (slot-value esd name) (slot-value dsd name)))
+    esd))
+
+(defun lisp-name-to-sql-name (lisp)
+  "Convert a lisp name (atom or list, string or symbol) into a canonical
+SQL name"
+  (unless (stringp lisp)
+    (setq lisp
+         (typecase lisp
+           (symbol (symbol-name lisp))
+           (t (write-to-string lisp)))))
+  (do* ((len (length lisp))
+       (sql (make-string len))
+       (i 0 (1+ i)))
+      ((= i len) (string-upcase sql))
+    (declare (fixnum i)
+            (simple-string sql))
+    (setf (schar sql i)
+         (let ((c (char lisp i)))
+           (case c
+             ((#\- #\$ #\+ #\#) #\_)
+             (otherwise c))))))
+
+#+ho-normal-cesd
+(setq cl:*features* (delete :ho-normal-cesd cl:*features*))
+#+ho-normal-dsdc
+(setq cl:*features* (delete :ho-normal-dsdc cl:*features*))
+#+ho-normal-esdc
+(setq cl:*features* (delete :ho-normal-esdc cl:*features*))
+
+(defun lisp-type-is-a-string (type)
+  (or (eq type 'string)
+      (and (listp type) (some #'(lambda (x) (eq x 'string)) type))))
+
+(defun base-value-type (value-type)
+  (if (atom value-type)
+      value-type
+    (car value-type)))
+
+(defun value-type-to-lisp-type (value-type)
+  (case (base-value-type value-type)
+    ((:string :cdata :varchar :char)
+     '(or null string))
+    (:datetime
+     '(or null integer))
+    (:character
+     '(or null character))
     (:fixnum
-     'fixnum)
+     '(or null fixnum))
     (:boolean
-     'boolean)
-    (:integer
-     'integer)
-    (:cdata
-     'string)
-    (:float
-     'float)
-    (:nil
-     t)
+     '(or null boolean))
+    ((:integer :long-integer)
+     '(or null integer))
+    ((:float :single-float)
+     '(or null single-float))
+    (:double-float
+     '(or null double-float))
     (otherwise
-     ho-type)))
+     t)))
+
+(defun value-type-to-sql-type (value-type)
+  "Return two values, the sql type and field length."
+  (let ((type (base-value-type value-type))
+       (length (when (consp value-type)
+                 (cadr value-type))))
+    (values
+     (case type
+       ((:char :character)
+       :char)
+       (:varchar
+       :varchar)
+       ((:fixnum :integer)
+       :integer)
+       (:long-integer
+       :long-integer)
+       (:boolean
+       :boolean)
+       ((:float :single-float)
+       :single-float)
+       (:double-float
+       :double-float)
+       (:datetime
+        :long-integer)
+       (otherwise
+       :text))
+     length)))
 
-(defun ho-type-to-sql-type (ho-type)
-  (check-type ho-type symbol)
-  (case (intern (symbol-name ho-type) (symbol-name :keyword))
-    (:string
-     'string)
-    (:fixnum
-     'integer)
-    (:boolean
-     'boolean)
-    (:integer
-     'integer)
-    (:cdata
-     'string)
-    (:float
-     'float)
-    (:nil
-     t)
-    (otherwise
-     ho-type)))
+;;;; Class initialization function
 
+;; One entry for each class with lazy readers defined.  The value is a plist mapping
+;; slot-name to a lazy reader, each of which is a list of a function and slot-names.
+(defvar *lazy-readers* (make-hash-table))
+
+(defmethod slot-unbound ((class hyperobject-class) instance slot-name)
+  (let ((lazy-reader
+        (loop for super in (class-precedence-list class)
+              as lazy-reader = (getf (gethash super *lazy-readers*) slot-name)
+              when lazy-reader return it)))
+    (if lazy-reader
+       (setf (slot-value instance slot-name)
+             (if (atom lazy-reader)
+                 (make-instance lazy-reader)
+                 (apply (car lazy-reader)
+                        (loop for arg-slot-name in (cdr lazy-reader)
+                              collect (slot-value instance arg-slot-name)))))
+       ;; No lazy reader -- defer to regular slot-unbound handling.
+       (call-next-method))))
+
+;; The reader is a function and the reader-keys are slot names.  The slot is lazily set to
+;; the result of applying the function to the slot-values of those slots, and that value
+;; is also returned.
+(defun ensure-lazy-reader (cl class-name slot-name subobj-class reader 
+                          &rest reader-keys)
+  (setf (getf (gethash cl *lazy-readers*) slot-name)
+    (aif subobj-class
+        it
+        (list* reader (copy-list reader-keys)))))
+
+(defun remove-lazy-reader (class-name slot-name)
+  (setf (getf (gethash (find-class class-name) *lazy-readers*) slot-name)
+    nil))
 
-;;;; Class initialization function
 
 (defun finalize-subobjects (cl)
   "Process class subobjects slot"
-  (setf (slot-value cl 'subobjects)
-    (let ((subobjects '()))
-      (dolist (slot (class-slots cl))
-       (when (slot-value slot 'subobject)
-         (push (make-instance 'subobject :name (slot-definition-name slot)
-                              :reader (if (eq t (slot-value slot 'subobject))
-                                          (slot-definition-name slot)
-                                        (slot-value slot 'subobject)))
-               subobjects)))
-      subobjects)))
+  (setf (subobjects cl)
+       (let ((subobjects '()))
+         (dolist (slot (class-slots cl))
+           (let-when
+            (subobj-def (esd-subobject slot))
+            (let ((subobject
+                   (make-instance 'subobject
+                                  :name-class (class-name cl)
+                                  :name-slot (slot-definition-name slot)
+                                  :subobj-class (when (atom subobj-def)
+                                                  subobj-def)
+                                  :lookup (when (listp subobj-def)
+                                            (car subobj-def))
+                                  :lookup-keys (when (listp subobj-def)
+                                                 (cdr subobj-def)))))
+              (unless (eq (lookup subobject) t)
+                (apply #'ensure-lazy-reader
+                       cl
+                       (name-class subobject) (name-slot subobject)
+                       (subobj-class subobject)
+                       (lookup subobject) (lookup-keys subobject))
+                (push subobject subobjects)))))
+         ;; sbcl/cmu reverse class-slots compared to the defclass form
+         ;; so re-reverse on cmu/sbcl
+         #+(or cmu sbcl) subobjects
+         #-(or cmu sbcl) (nreverse subobjects)
+         )))
+
+(defun finalize-class-slots (cl)
+  "Make sure all class slots have an expected value"
+  (unless (user-name cl)
+    (setf (user-name cl) (format nil "~:(~A~)" (class-name cl))))
+  
+  (setf (user-name-plural cl)
+       (if (and (consp (user-name cl)) (cadr (user-name cl)))
+           (cadr (user-name cl))
+           (format nil "~A~P" (if (consp (user-name cl))
+                                  (car (user-name cl))
+                                  (user-name cl))
+                   2)))
+
+  (dolist (name '(user-name description version guid sql-name))
+    (awhen (slot-value cl name)
+          (setf (slot-value cl name)
+            (if (listp it)
+                (car it)
+              it))))
+  
+  (unless (sql-name cl)
+    (setf (sql-name cl) (lisp-name-to-sql-name (class-name cl))))
+  )
 
 (defun finalize-documentation (cl)
   "Calculate class documentation slot"
-  (awhen (slot-value cl 'title)
-        (setf (slot-value cl 'title)
-              (etypecase (slot-value cl 'title)
-                  (cons (car it))
-                  ((or string symbol) it))))
-  (awhen (slot-value cl 'description)
-        (setf (slot-value cl 'description)
-              (etypecase (slot-value cl 'description)
-                  (cons (car it))
-                  ((or string symbol) it))))
-
   (let ((*print-circle* nil))
-    (setf (documentation (class-name cl) 'class)
-      (format nil "Hyperobject~A~A~A~A"
-             (aif (slot-value cl 'title)
-                  (format nil ": ~A" it ""))
-             (aif (slot-value cl 'description)
-                  (format nil "~%Class description: ~A" it) "")
-             (aif (slot-value cl 'subobjects)
-                  (format nil "~%Subobjects:~{ ~A~}" (mapcar #'name it)) "")
-             (aif (slot-value cl 'print-slots)
-                  (format nil "~%Print-slots:~{ ~A~}" it) "")
-             ))))
+    (setf (documentation cl 'type)
+         (format nil "Hyperobject~A~A~A~A"
+                 (aif (user-name cl)
+                      (format nil ": ~A" it ""))
+                 (aif (description cl)
+                      (format nil "~%Class description: ~A" it) "")
+                 (aif (subobjects cl)
+                      (format nil "~%Subobjects:~{ ~A~}" (mapcar #'name-slot it)) "")
+                 (aif (default-print-slots cl)
+                      (format nil "~%Default print slots:~{ ~A~}" it) "")
+                 ))))
+
+(defun finalize-hyperlinks (cl)
+  (let ((hyperlinks '()))
+    (dolist (esd (class-slots cl))
+      (awhen (slot-value esd 'hyperlink)
+            (push
+             (make-instance 'hyperlink
+                            :name (slot-definition-name esd)
+                            :lookup it
+                            :link-parameters (slot-value esd 'hyperlink-parameters))
+             hyperlinks)))
+    ;; cmu/sbcl reverse class-slots compared to the defclass form
+    ;; hyperlinks is already reversed from the dolist/push loop, so re-reverse on sbcl/cmu
+    #-(or cmu sbcl) (setq hyperlinks (nreverse hyperlinks))
+    (setf (slot-value cl 'hyperlinks) hyperlinks)))
 
 (defun init-hyperobject-class (cl)
   "Initialize a hyperobject class. Calculates all class slots"
   (finalize-views cl)
   (finalize-hyperlinks cl)
   (finalize-sql cl)
+  (finalize-rules cl)
+  (finalize-class-slots cl)
   (finalize-documentation cl))
 
 
 (defun find-slot-by-name (cl name)
   (find name (class-slots cl) :key #'slot-definition-name))
 
-(defun hyperobject-class-fmtstr-text (obj)
-  (slot-value (class-of obj) 'fmtstr-text))
-
-(defun hyperobject-class-fmtstr-html (obj)
-  (slot-value (class-of obj) 'fmtstr-html))
-
-(defun hyperobject-class-fmtstr-xml (obj)
-  (slot-value (class-of obj) 'fmtstr-xml))
+(defun hyperobject-class-user-name (obj)
+  (user-name (class-of obj)))
 
-(defun hyperobject-class-fmtstr-text-labels (obj)
-  (slot-value (class-of obj) 'fmtstr-text-labels))
-
-(defun hyperobject-class-fmtstr-html-labels (obj)
-  (slot-value (class-of obj) 'fmtstr-html-labels))
-
-(defun hyperobject-class-fmtstr-xml-labels (obj)
-  (slot-value (class-of obj) 'fmtstr-xml-labels))
-
-(defun hyperobject-class-value-func (obj)
-  (slot-value (class-of obj) 'value-func))
-
-(defun hyperobject-class-xmlvalue-func (obj)
-  (slot-value (class-of obj) 'xmlvalue-func))
-
-(eval-when (:compile-toplevel :load-toplevel :execute)
-  
-  (defun hyperobject-class-title (obj)
-    (awhen (slot-value (class-of obj) 'title)
-          (if (consp it)
-              (car it)
-              it))))
+(defun hyperobject-class-user-name-plural (obj)
+  (user-name-plural (class-of obj)))
 
 (defun hyperobject-class-subobjects (obj)
-  (slot-value (class-of obj) 'subobjects))
+  (subobjects (class-of obj)))
 
 (defun hyperobject-class-hyperlinks (obj)
-  (slot-value (class-of obj) 'hyperlinks))
-
-(defun hyperobject-class-fields (obj)
-  (class-slots (class-of obj)))
-
-(defun hyperobject-class-print-slots (obj)
-  (slot-value (class-of obj) 'print-slots))
-
-(defun hyperobject-class-fmtstr-html-ref (obj)
-  (slot-value (class-of obj) 'fmtstr-html-ref))
-
-(defun hyperobject-class-fmtstr-xml-ref (obj)
-  (slot-value (class-of obj) 'fmtstr-xml-ref))
-
-(defun hyperobject-class-fmtstr-html-ref-labels (obj)
-  (slot-value (class-of obj) 'fmtstr-html-ref-labels))
-
-(defun hyperobject-class-fmtstr-xml-ref-labels (obj)
-  (slot-value (class-of obj) 'fmtstr-xml-ref-labels))
-
+  (hyperlinks (class-of obj)))
+
+(defun hyperobject-class-slots (obj)
+  ;; cmucl/sbcl reverse class-slots
+  #+(or cmu sbcl) (reverse (class-slots (class-of obj)))
+  #-(or cmu sbcl) (class-slots (class-of obj)))
+
+(defun all-subobjects (obj)
+  "Returns a list of all subobjects in an object"
+  (let ((so-list '()))
+    (dolist (subobj-obj (subobjects (class-of obj)) (nreverse so-list))
+      (dolist (so (funcall (name-slot subobj-obj) obj))
+        (push so so-list)))))