Index: openacs-4/contrib/packages/simulation/sql/postgresql/simulation-content-types-create.sql
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/sql/postgresql/Attic/simulation-content-types-create.sql,v
diff -u -r1.17 -r1.18
--- openacs-4/contrib/packages/simulation/sql/postgresql/simulation-content-types-create.sql	19 Dec 2003 09:24:27 -0000	1.17
+++ openacs-4/contrib/packages/simulation/sql/postgresql/simulation-content-types-create.sql	12 Jan 2004 16:07:37 -0000	1.18
@@ -261,7 +261,7 @@
 ----------------------------------------------------------------------
 -- sim_message
 ----------------------------------------------------------------------
--- # TODO: foreign constraints for to_role_id and from_role_id
+-- # TODO (.25h): foreign constraints for to_role_id and from_role_id in the 'column_spec'
 
 select content_type__create_type(
     'sim_message',                 -- content_type
Index: openacs-4/contrib/packages/simulation/test/crawl-links.test
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/test/Attic/crawl-links.test,v
diff -u -r1.3 -r1.4
--- openacs-4/contrib/packages/simulation/test/crawl-links.test	28 Nov 2003 16:55:52 -0000	1.3
+++ openacs-4/contrib/packages/simulation/test/crawl-links.test	12 Jan 2004 16:07:37 -0000	1.4
@@ -17,7 +17,7 @@
     ::twt::crawl_links $simulation_uri
 
     ::twt::log "Did not visit the following URLs because of exclude list: [join [::twt::get_excluded_urls] "\n"]"
-    ::twt::log "TODO: Should we visit the delete links for better coverage? It tears down demo data though."
+    ::twt::log "TODO B: Should we visit the delete links for better coverage? It tears down demo data though."
             
  } result] } {
      global errorInfo
Index: openacs-4/contrib/packages/simulation/test/simulation-test-procs.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/test/Attic/simulation-test-procs.tcl,v
diff -u -r1.8 -r1.9
--- openacs-4/contrib/packages/simulation/test/simulation-test-procs.tcl	8 Jan 2004 15:54:01 -0000	1.8
+++ openacs-4/contrib/packages/simulation/test/simulation-test-procs.tcl	12 Jan 2004 16:07:37 -0000	1.9
@@ -680,4 +680,4 @@
     if { [regexp "Permission Denied" [response body]] } {
         error "Got permission denied when deleting an object"
     }
-}
\ No newline at end of file
+}
Index: openacs-4/contrib/packages/simulation/www/simulation-master.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/www/Attic/simulation-master.tcl,v
diff -u -r1.9 -r1.10
--- openacs-4/contrib/packages/simulation/www/simulation-master.tcl	7 Jan 2004 16:16:06 -0000	1.9
+++ openacs-4/contrib/packages/simulation/www/simulation-master.tcl	12 Jan 2004 16:04:22 -0000	1.10
@@ -63,7 +63,7 @@
 #
 ######################################################################
 
-#  TODO: kill link bar for players
+#  TODO (.25h): kill link bar for players
 
 if { $citybuild_p } {
     lappend subnavbar_list [list "${base_url}citybuild" "CityBuild"]
@@ -90,7 +90,7 @@
 
 lappend subnavbar_list [list "/doc/simulation" "Doc"]
 
-# TODO: should use ad_navbar
+# TODO B: should use ad_navbar
 # couldn't figure out how to pass the input to ad_narbar so hacking it in here
 # and also added context checking
 set link_list ""
Index: openacs-4/contrib/packages/simulation/www/citybuild/object-edit.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/www/citybuild/Attic/object-edit.tcl,v
diff -u -r1.12 -r1.13
--- openacs-4/contrib/packages/simulation/www/citybuild/object-edit.tcl	12 Jan 2004 11:35:33 -0000	1.12
+++ openacs-4/contrib/packages/simulation/www/citybuild/object-edit.tcl	12 Jan 2004 16:04:22 -0000	1.13
@@ -27,7 +27,7 @@
 set context [list [list "." "CityBuild"] $page_title]
 
 
-# TODO: LARS: Related images/order_n ... not sure what the issue was, if there's not a problem, let's drop it.
+# TODO (1h): Related images/order_n ... not sure what the issue was, if there's not a problem, let's drop it.
 
 
 ######################################################################
@@ -515,7 +515,7 @@
 
 } -new_data {
     
-    # TODO: For now, assume we are always using this to create global objects
+    # TODO B: For now, assume we are always using this to create global objects -- LARS: Not sure what to do on this
     # When we use this form to create case objects, we should simply check for create permission on the case
     # we could alternately switch to using the cr folder system, in which case:
     #    sim_object_create on package_id is replaced by 'create' on a root folder, and
Index: openacs-4/packages/simulation/sql/postgresql/simulation-content-types-create.sql
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/sql/postgresql/simulation-content-types-create.sql,v
diff -u -r1.17 -r1.18
--- openacs-4/packages/simulation/sql/postgresql/simulation-content-types-create.sql	19 Dec 2003 09:24:27 -0000	1.17
+++ openacs-4/packages/simulation/sql/postgresql/simulation-content-types-create.sql	12 Jan 2004 16:07:37 -0000	1.18
@@ -261,7 +261,7 @@
 ----------------------------------------------------------------------
 -- sim_message
 ----------------------------------------------------------------------
--- # TODO: foreign constraints for to_role_id and from_role_id
+-- # TODO (.25h): foreign constraints for to_role_id and from_role_id in the 'column_spec'
 
 select content_type__create_type(
     'sim_message',                 -- content_type
Index: openacs-4/packages/simulation/test/crawl-links.test
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/test/Attic/crawl-links.test,v
diff -u -r1.3 -r1.4
--- openacs-4/packages/simulation/test/crawl-links.test	28 Nov 2003 16:55:52 -0000	1.3
+++ openacs-4/packages/simulation/test/crawl-links.test	12 Jan 2004 16:07:37 -0000	1.4
@@ -17,7 +17,7 @@
     ::twt::crawl_links $simulation_uri
 
     ::twt::log "Did not visit the following URLs because of exclude list: [join [::twt::get_excluded_urls] "\n"]"
-    ::twt::log "TODO: Should we visit the delete links for better coverage? It tears down demo data though."
+    ::twt::log "TODO B: Should we visit the delete links for better coverage? It tears down demo data though."
             
  } result] } {
      global errorInfo
Index: openacs-4/packages/simulation/test/simulation-test-procs.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/test/simulation-test-procs.tcl,v
diff -u -r1.8 -r1.9
--- openacs-4/packages/simulation/test/simulation-test-procs.tcl	8 Jan 2004 15:54:01 -0000	1.8
+++ openacs-4/packages/simulation/test/simulation-test-procs.tcl	12 Jan 2004 16:07:37 -0000	1.9
@@ -680,4 +680,4 @@
     if { [regexp "Permission Denied" [response body]] } {
         error "Got permission denied when deleting an object"
     }
-}
\ No newline at end of file
+}
Index: openacs-4/packages/simulation/www/simulation-master.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/www/simulation-master.tcl,v
diff -u -r1.9 -r1.10
--- openacs-4/packages/simulation/www/simulation-master.tcl	7 Jan 2004 16:16:06 -0000	1.9
+++ openacs-4/packages/simulation/www/simulation-master.tcl	12 Jan 2004 16:04:22 -0000	1.10
@@ -63,7 +63,7 @@
 #
 ######################################################################
 
-#  TODO: kill link bar for players
+#  TODO (.25h): kill link bar for players
 
 if { $citybuild_p } {
     lappend subnavbar_list [list "${base_url}citybuild" "CityBuild"]
@@ -90,7 +90,7 @@
 
 lappend subnavbar_list [list "/doc/simulation" "Doc"]
 
-# TODO: should use ad_navbar
+# TODO B: should use ad_navbar
 # couldn't figure out how to pass the input to ad_narbar so hacking it in here
 # and also added context checking
 set link_list ""
Index: openacs-4/packages/simulation/www/citybuild/object-edit.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/www/citybuild/object-edit.tcl,v
diff -u -r1.12 -r1.13
--- openacs-4/packages/simulation/www/citybuild/object-edit.tcl	12 Jan 2004 11:35:33 -0000	1.12
+++ openacs-4/packages/simulation/www/citybuild/object-edit.tcl	12 Jan 2004 16:04:22 -0000	1.13
@@ -27,7 +27,7 @@
 set context [list [list "." "CityBuild"] $page_title]
 
 
-# TODO: LARS: Related images/order_n ... not sure what the issue was, if there's not a problem, let's drop it.
+# TODO (1h): Related images/order_n ... not sure what the issue was, if there's not a problem, let's drop it.
 
 
 ######################################################################
@@ -515,7 +515,7 @@
 
 } -new_data {
     
-    # TODO: For now, assume we are always using this to create global objects
+    # TODO B: For now, assume we are always using this to create global objects -- LARS: Not sure what to do on this
     # When we use this form to create case objects, we should simply check for create permission on the case
     # we could alternately switch to using the cr folder system, in which case:
     #    sim_object_create on package_id is replaced by 'create' on a root folder, and