Index: openacs-4/contrib/packages/simulation/lib/cases.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/lib/Attic/cases.tcl,v
diff -u -r1.5 -r1.6
--- openacs-4/contrib/packages/simulation/lib/cases.tcl	6 Jan 2004 11:48:49 -0000	1.5
+++ openacs-4/contrib/packages/simulation/lib/cases.tcl	6 Jan 2004 13:05:47 -0000	1.6
@@ -10,6 +10,7 @@
     }
 }
 
+# TODO: add link to simplay/case for each case
 set package_id [ad_conn package_id]
 
 set elements {
Index: openacs-4/contrib/packages/simulation/lib/messages.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/lib/Attic/messages.tcl,v
diff -u -r1.10 -r1.11
--- openacs-4/contrib/packages/simulation/lib/messages.tcl	6 Jan 2004 09:51:47 -0000	1.10
+++ openacs-4/contrib/packages/simulation/lib/messages.tcl	6 Jan 2004 13:05:47 -0000	1.11
@@ -13,7 +13,11 @@
     }
 }
 
+# TODO: finish.  if case id is nil, check that adminplayer_p is true.  if not, fail.
+# if admin is true, ...
+
 set package_id [ad_conn package_id]
+set adminplayer_p [permission::permission_p -object_id $package_id -privilege sim_adminplayer]
 
 if { [exists_and_not_null case_id] } {
     set user_roles [workflow::case::get_user_roles -case_id $case_id]
Index: openacs-4/contrib/packages/simulation/lib/object-display.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/lib/Attic/object-display.adp,v
diff -u -r1.5 -r1.6
--- openacs-4/contrib/packages/simulation/lib/object-display.adp	29 Oct 2003 18:48:09 -0000	1.5
+++ openacs-4/contrib/packages/simulation/lib/object-display.adp	6 Jan 2004 13:05:47 -0000	1.6
@@ -12,3 +12,5 @@
     <a href="@edit_url@" class="button">Edit</a>
   </p>
 </if>
+<p>TODO: edit_url shouldn't show up if user doesn't have permission to
+  edit object
\ No newline at end of file
Index: openacs-4/contrib/packages/simulation/lib/sim-template-tasks.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/lib/Attic/sim-template-tasks.adp,v
diff -u -r1.4 -r1.5
--- openacs-4/contrib/packages/simulation/lib/sim-template-tasks.adp	5 Jan 2004 10:31:40 -0000	1.4
+++ openacs-4/contrib/packages/simulation/lib/sim-template-tasks.adp	6 Jan 2004 13:05:47 -0000	1.5
@@ -1,2 +1,2 @@
 <listtemplate name="tasks"></listtemplate>
-
+TODO: clearly identify state columns
Index: openacs-4/contrib/packages/simulation/lib/sim-template-tasks.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/lib/Attic/sim-template-tasks.tcl,v
diff -u -r1.16 -r1.17
--- openacs-4/contrib/packages/simulation/lib/sim-template-tasks.tcl	5 Jan 2004 10:31:40 -0000	1.16
+++ openacs-4/contrib/packages/simulation/lib/sim-template-tasks.tcl	6 Jan 2004 13:05:47 -0000	1.17
@@ -37,8 +37,8 @@
 # how is type going to work?  open question pending prototyping
 
 if { $display_mode == "edit"} {
-    set actions [list "Add a Task" [export_vars -base task-edit {workflow_id} ] {}]
-    lappend actions "Add a State" [export_vars -base state-edit { workflow_id}] {}
+    set actions [list "Add a State" [export_vars -base state-edit { workflow_id}] {}]
+    lappend actions "Add a Task" [export_vars -base task-edit {workflow_id} ] {}
 } else {
     set actions ""
 }
Index: openacs-4/contrib/packages/simulation/www/citybuild/object-edit.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/www/citybuild/Attic/object-edit.adp,v
diff -u -r1.3 -r1.4
--- openacs-4/contrib/packages/simulation/www/citybuild/object-edit.adp	11 Dec 2003 13:21:52 -0000	1.3
+++ openacs-4/contrib/packages/simulation/www/citybuild/object-edit.adp	6 Jan 2004 13:05:47 -0000	1.4
@@ -51,4 +51,5 @@
 
 <formtemplate id="object"></formtemplate>
 TODO: Show on map should be read-only unless user has "sim_set_map_p"
-priv on package_id
\ No newline at end of file
+priv on package_id
+<p>TODO: suppress HTML cut-paste text for stylesheets
\ No newline at end of file
Index: openacs-4/contrib/packages/simulation/www/simbuild/task-edit.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/www/simbuild/Attic/task-edit.adp,v
diff -u -r1.7 -r1.8
--- openacs-4/contrib/packages/simulation/www/simbuild/task-edit.adp	11 Dec 2003 13:21:53 -0000	1.7
+++ openacs-4/contrib/packages/simulation/www/simbuild/task-edit.adp	6 Jan 2004 13:05:48 -0000	1.8
@@ -4,4 +4,5 @@
   <property name="focus">task.pretty_name</property>
 
 <formtemplate id="task"></formtemplate>
-
+TODO: If there are no states, show a message "This task cannot be
+fully developed because its workflow has no states" and link to add states
Index: openacs-4/contrib/packages/simulation/www/simbuild/template-edit.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/www/simbuild/Attic/template-edit.adp,v
diff -u -r1.14 -r1.15
--- openacs-4/contrib/packages/simulation/www/simbuild/template-edit.adp	5 Jan 2004 11:56:59 -0000	1.14
+++ openacs-4/contrib/packages/simulation/www/simbuild/template-edit.adp	6 Jan 2004 13:05:48 -0000	1.15
@@ -7,8 +7,6 @@
 
 <if @workflow_id@ not nil>
 
-  <a href="@delete_url@" class="action" onclick="return confirm('Are you sure you want to delete the template?');">Delete this template</a>
-
   <a name="roles"><h4>Roles</h4></a>
 
   <include src="/packages/simulation/lib/sim-template-roles" workflow_id="@workflow_id@" display_mode="edit">
@@ -18,15 +16,20 @@
 
   <include src="/packages/simulation/lib/sim-template-tasks" workflow_id="@workflow_id@" display_mode="edit" >
   
-  <p>TODO: Allow reordering of roles, states, tasks.</p>
+<p>
+TODO: Hide task table if there are no states, and show only "add a
+  state" link instead
+  <p>TODO: B: Allow reordering of roles, states, tasks.</p>
   <p>TODO: Instead of initialize column and special task, UI should
   show an "initial state" radio button.  The special initialize task should not
   appear in this list.
-  <p>TODO: Better widget for suggested duration.</p>
-
+  <p>TODO: B: Better widget for suggested duration.</p>
+  <p>TODO: B: When Ready for Use Checkbox is in display (not edit)
+  mode, show only the text "yes" or "no" instead of showing a disabled checkbox </p>
+  <a class="action">TODO: If !ready_p, display "Mark this template ready for use" link</a>
   <a href="@spec_url@" class="action">Download a specification for this template</a>
   <if @inst_url@ not nil>
     <a href="@inst_url@" class="action">Start a simulation with this template</a>
   </if>
-
+  <a href="@delete_url@" class="action" onclick="return confirm('Are you sure you want to delete the template?');">Delete this template</a>
 </if>
Index: openacs-4/contrib/packages/simulation/www/siminst/map-tasks.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/www/siminst/Attic/map-tasks.adp,v
diff -u -r1.6 -r1.7
--- openacs-4/contrib/packages/simulation/www/siminst/map-tasks.adp	5 Jan 2004 10:31:41 -0000	1.6
+++ openacs-4/contrib/packages/simulation/www/siminst/map-tasks.adp	6 Jan 2004 13:05:48 -0000	1.7
@@ -7,5 +7,5 @@
 <p></p>
 
 <a href="." class="action">Back to simulation list</a>
+<p>TODO: don't show initial task on this list
 
-
Index: openacs-4/contrib/packages/simulation/www/simplay/control-bar.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/www/simplay/Attic/control-bar.adp,v
diff -u -r1.6 -r1.7
--- openacs-4/contrib/packages/simulation/www/simplay/control-bar.adp	5 Jan 2004 12:33:41 -0000	1.6
+++ openacs-4/contrib/packages/simulation/www/simplay/control-bar.adp	6 Jan 2004 13:05:48 -0000	1.7
@@ -6,7 +6,9 @@
   <li>Archive
   <li>About this simulation
 </ul>
-
+<p>TODO: link Archive to "portfolio" of all objects this actor owns
+  (since the actor is specific to role and case, these are objects in the same case)
+<p>TODO: Link "about this simulation" to something
 <h4>Your Role<if @roles:rowcount@ gt 1>s</if></h4>
 
 <ul>
Index: openacs-4/contrib/packages/simulation/www/simplay/index.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/www/simplay/Attic/index.adp,v
diff -u -r1.14 -r1.15
--- openacs-4/contrib/packages/simulation/www/simplay/index.adp	5 Jan 2004 14:09:33 -0000	1.14
+++ openacs-4/contrib/packages/simulation/www/simplay/index.adp	6 Jan 2004 13:05:48 -0000	1.15
@@ -19,3 +19,4 @@
 </if>
 
 <p>TODO: how can a player access completed cases?</p>
+<P>TODO: in admin mode, show all adminned cases
\ No newline at end of file
Index: openacs-4/contrib/packages/simulation/www/simplay/message.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/www/simplay/Attic/message.adp,v
diff -u -r1.4 -r1.5
--- openacs-4/contrib/packages/simulation/www/simplay/message.adp	5 Jan 2004 17:05:07 -0000	1.4
+++ openacs-4/contrib/packages/simulation/www/simplay/message.adp	6 Jan 2004 13:05:48 -0000	1.5
@@ -16,3 +16,12 @@
   </property>
   
 <formtemplate id="message"></formtemplate>
+<p>TODO: Task should be part of heading
+<p>TODO: description and "document"
+link should be inline before the message form instead of being part of
+the form
+<p>TODO: B: If there's only one possible recipient, don't show the
+checkbox 
+<p>TODO: fix problem that To: is blank in view mode
+<p>TODO: make attachments be read-only links in display mode, instead
+of showing checkboxes
Index: openacs-4/contrib/packages/simulation/www/simplay/messages.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/contrib/packages/simulation/www/simplay/Attic/messages.adp,v
diff -u -r1.7 -r1.8
--- openacs-4/contrib/packages/simulation/www/simplay/messages.adp	5 Jan 2004 10:49:59 -0000	1.7
+++ openacs-4/contrib/packages/simulation/www/simplay/messages.adp	6 Jan 2004 13:05:48 -0000	1.8
@@ -5,3 +5,4 @@
 <include src="/packages/simulation/lib/messages" user_id="@user_id@" case_id="@case_id@"></include>
 
 
+<p>TODO: show sent messages as well
\ No newline at end of file
Index: openacs-4/packages/simulation/lib/cases.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/lib/cases.tcl,v
diff -u -r1.5 -r1.6
--- openacs-4/packages/simulation/lib/cases.tcl	6 Jan 2004 11:48:49 -0000	1.5
+++ openacs-4/packages/simulation/lib/cases.tcl	6 Jan 2004 13:05:47 -0000	1.6
@@ -10,6 +10,7 @@
     }
 }
 
+# TODO: add link to simplay/case for each case
 set package_id [ad_conn package_id]
 
 set elements {
Index: openacs-4/packages/simulation/lib/messages.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/lib/messages.tcl,v
diff -u -r1.10 -r1.11
--- openacs-4/packages/simulation/lib/messages.tcl	6 Jan 2004 09:51:47 -0000	1.10
+++ openacs-4/packages/simulation/lib/messages.tcl	6 Jan 2004 13:05:47 -0000	1.11
@@ -13,7 +13,11 @@
     }
 }
 
+# TODO: finish.  if case id is nil, check that adminplayer_p is true.  if not, fail.
+# if admin is true, ...
+
 set package_id [ad_conn package_id]
+set adminplayer_p [permission::permission_p -object_id $package_id -privilege sim_adminplayer]
 
 if { [exists_and_not_null case_id] } {
     set user_roles [workflow::case::get_user_roles -case_id $case_id]
Index: openacs-4/packages/simulation/lib/object-display.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/lib/object-display.adp,v
diff -u -r1.5 -r1.6
--- openacs-4/packages/simulation/lib/object-display.adp	29 Oct 2003 18:48:09 -0000	1.5
+++ openacs-4/packages/simulation/lib/object-display.adp	6 Jan 2004 13:05:47 -0000	1.6
@@ -12,3 +12,5 @@
     <a href="@edit_url@" class="button">Edit</a>
   </p>
 </if>
+<p>TODO: edit_url shouldn't show up if user doesn't have permission to
+  edit object
\ No newline at end of file
Index: openacs-4/packages/simulation/lib/sim-template-tasks.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/lib/sim-template-tasks.adp,v
diff -u -r1.4 -r1.5
--- openacs-4/packages/simulation/lib/sim-template-tasks.adp	5 Jan 2004 10:31:40 -0000	1.4
+++ openacs-4/packages/simulation/lib/sim-template-tasks.adp	6 Jan 2004 13:05:47 -0000	1.5
@@ -1,2 +1,2 @@
 <listtemplate name="tasks"></listtemplate>
-
+TODO: clearly identify state columns
Index: openacs-4/packages/simulation/lib/sim-template-tasks.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/lib/sim-template-tasks.tcl,v
diff -u -r1.16 -r1.17
--- openacs-4/packages/simulation/lib/sim-template-tasks.tcl	5 Jan 2004 10:31:40 -0000	1.16
+++ openacs-4/packages/simulation/lib/sim-template-tasks.tcl	6 Jan 2004 13:05:47 -0000	1.17
@@ -37,8 +37,8 @@
 # how is type going to work?  open question pending prototyping
 
 if { $display_mode == "edit"} {
-    set actions [list "Add a Task" [export_vars -base task-edit {workflow_id} ] {}]
-    lappend actions "Add a State" [export_vars -base state-edit { workflow_id}] {}
+    set actions [list "Add a State" [export_vars -base state-edit { workflow_id}] {}]
+    lappend actions "Add a Task" [export_vars -base task-edit {workflow_id} ] {}
 } else {
     set actions ""
 }
Index: openacs-4/packages/simulation/www/citybuild/object-edit.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/www/citybuild/object-edit.adp,v
diff -u -r1.3 -r1.4
--- openacs-4/packages/simulation/www/citybuild/object-edit.adp	11 Dec 2003 13:21:52 -0000	1.3
+++ openacs-4/packages/simulation/www/citybuild/object-edit.adp	6 Jan 2004 13:05:47 -0000	1.4
@@ -51,4 +51,5 @@
 
 <formtemplate id="object"></formtemplate>
 TODO: Show on map should be read-only unless user has "sim_set_map_p"
-priv on package_id
\ No newline at end of file
+priv on package_id
+<p>TODO: suppress HTML cut-paste text for stylesheets
\ No newline at end of file
Index: openacs-4/packages/simulation/www/simbuild/task-edit.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/www/simbuild/task-edit.adp,v
diff -u -r1.7 -r1.8
--- openacs-4/packages/simulation/www/simbuild/task-edit.adp	11 Dec 2003 13:21:53 -0000	1.7
+++ openacs-4/packages/simulation/www/simbuild/task-edit.adp	6 Jan 2004 13:05:48 -0000	1.8
@@ -4,4 +4,5 @@
   <property name="focus">task.pretty_name</property>
 
 <formtemplate id="task"></formtemplate>
-
+TODO: If there are no states, show a message "This task cannot be
+fully developed because its workflow has no states" and link to add states
Index: openacs-4/packages/simulation/www/simbuild/template-edit.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/www/simbuild/template-edit.adp,v
diff -u -r1.14 -r1.15
--- openacs-4/packages/simulation/www/simbuild/template-edit.adp	5 Jan 2004 11:56:59 -0000	1.14
+++ openacs-4/packages/simulation/www/simbuild/template-edit.adp	6 Jan 2004 13:05:48 -0000	1.15
@@ -7,8 +7,6 @@
 
 <if @workflow_id@ not nil>
 
-  <a href="@delete_url@" class="action" onclick="return confirm('Are you sure you want to delete the template?');">Delete this template</a>
-
   <a name="roles"><h4>Roles</h4></a>
 
   <include src="/packages/simulation/lib/sim-template-roles" workflow_id="@workflow_id@" display_mode="edit">
@@ -18,15 +16,20 @@
 
   <include src="/packages/simulation/lib/sim-template-tasks" workflow_id="@workflow_id@" display_mode="edit" >
   
-  <p>TODO: Allow reordering of roles, states, tasks.</p>
+<p>
+TODO: Hide task table if there are no states, and show only "add a
+  state" link instead
+  <p>TODO: B: Allow reordering of roles, states, tasks.</p>
   <p>TODO: Instead of initialize column and special task, UI should
   show an "initial state" radio button.  The special initialize task should not
   appear in this list.
-  <p>TODO: Better widget for suggested duration.</p>
-
+  <p>TODO: B: Better widget for suggested duration.</p>
+  <p>TODO: B: When Ready for Use Checkbox is in display (not edit)
+  mode, show only the text "yes" or "no" instead of showing a disabled checkbox </p>
+  <a class="action">TODO: If !ready_p, display "Mark this template ready for use" link</a>
   <a href="@spec_url@" class="action">Download a specification for this template</a>
   <if @inst_url@ not nil>
     <a href="@inst_url@" class="action">Start a simulation with this template</a>
   </if>
-
+  <a href="@delete_url@" class="action" onclick="return confirm('Are you sure you want to delete the template?');">Delete this template</a>
 </if>
Index: openacs-4/packages/simulation/www/siminst/map-tasks.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/www/siminst/map-tasks.adp,v
diff -u -r1.6 -r1.7
--- openacs-4/packages/simulation/www/siminst/map-tasks.adp	5 Jan 2004 10:31:41 -0000	1.6
+++ openacs-4/packages/simulation/www/siminst/map-tasks.adp	6 Jan 2004 13:05:48 -0000	1.7
@@ -7,5 +7,5 @@
 <p></p>
 
 <a href="." class="action">Back to simulation list</a>
+<p>TODO: don't show initial task on this list
 
-
Index: openacs-4/packages/simulation/www/simplay/control-bar.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/www/simplay/control-bar.adp,v
diff -u -r1.6 -r1.7
--- openacs-4/packages/simulation/www/simplay/control-bar.adp	5 Jan 2004 12:33:41 -0000	1.6
+++ openacs-4/packages/simulation/www/simplay/control-bar.adp	6 Jan 2004 13:05:48 -0000	1.7
@@ -6,7 +6,9 @@
   <li>Archive
   <li>About this simulation
 </ul>
-
+<p>TODO: link Archive to "portfolio" of all objects this actor owns
+  (since the actor is specific to role and case, these are objects in the same case)
+<p>TODO: Link "about this simulation" to something
 <h4>Your Role<if @roles:rowcount@ gt 1>s</if></h4>
 
 <ul>
Index: openacs-4/packages/simulation/www/simplay/index.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/www/simplay/index.adp,v
diff -u -r1.14 -r1.15
--- openacs-4/packages/simulation/www/simplay/index.adp	5 Jan 2004 14:09:33 -0000	1.14
+++ openacs-4/packages/simulation/www/simplay/index.adp	6 Jan 2004 13:05:48 -0000	1.15
@@ -19,3 +19,4 @@
 </if>
 
 <p>TODO: how can a player access completed cases?</p>
+<P>TODO: in admin mode, show all adminned cases
\ No newline at end of file
Index: openacs-4/packages/simulation/www/simplay/message.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/www/simplay/message.adp,v
diff -u -r1.4 -r1.5
--- openacs-4/packages/simulation/www/simplay/message.adp	5 Jan 2004 17:05:07 -0000	1.4
+++ openacs-4/packages/simulation/www/simplay/message.adp	6 Jan 2004 13:05:48 -0000	1.5
@@ -16,3 +16,12 @@
   </property>
   
 <formtemplate id="message"></formtemplate>
+<p>TODO: Task should be part of heading
+<p>TODO: description and "document"
+link should be inline before the message form instead of being part of
+the form
+<p>TODO: B: If there's only one possible recipient, don't show the
+checkbox 
+<p>TODO: fix problem that To: is blank in view mode
+<p>TODO: make attachments be read-only links in display mode, instead
+of showing checkboxes
Index: openacs-4/packages/simulation/www/simplay/messages.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/simulation/www/simplay/messages.adp,v
diff -u -r1.7 -r1.8
--- openacs-4/packages/simulation/www/simplay/messages.adp	5 Jan 2004 10:49:59 -0000	1.7
+++ openacs-4/packages/simulation/www/simplay/messages.adp	6 Jan 2004 13:05:48 -0000	1.8
@@ -5,3 +5,4 @@
 <include src="/packages/simulation/lib/messages" user_id="@user_id@" case_id="@case_id@"></include>
 
 
+<p>TODO: show sent messages as well
\ No newline at end of file