doc fixes, one more fix to the permissions-in-listings fix

Author: Richard Jones

doc/customizing.txt

@@ -2,7 +2,7 @@
 Customising Roundup
 ===================
 
-:Version: $Revision: 1.170 $
+:Version: $Revision: 1.171 $
 
 .. This document borrows from the ZopeBook section on ZPT. The original is at:
    http://www.zope.org/Documentation/Books/ZopeBook/current/ZPT.stx
@@ -1927,7 +1927,24 @@ pretty      Date properties - render the date as "dd Mon YYYY" (eg. "19
             in the standard ``strftime`` call (see the `Python Library
             Reference: time module`__)
 menu        only on Link and Multilink properties - render a form select
-            list for this property
+            list for this property. Takes a number of optional arguments
+
+            size
+               is used to limit the length of the list labels
+            height
+               is used to set the <select> tag's "size" attribute
+            showid
+               includes the item ids in the list labels
+            additional
+               lists properties which should be included in the label
+            sort_on
+                indicates the property to sort the list on as (direction,
+                property) where direction is '+' or '-'.
+
+            The remaining keyword arguments are used as conditions for
+            filtering the items in the list - they're passed as the
+            "filterspec" argument to a Class.filter() call.
+
 sorted      only on Multilink properties - produce a list of the linked
             items sorted by some property, for example::
 
@@ -2626,7 +2643,7 @@ the due date, or causing automatic actions to fire if the due date passes.
     issue = IssueClass(db, "issue", 
                     assignedto=Link("user"), topic=Multilink("keyword"),
                     priority=Link("priority"), status=Link("status"),
-                    due_dat=Date())
+                    due_date=Date())
 
 2. add an edit field to the issue.item.html template::
 
@@ -2640,7 +2657,7 @@ the due date, or causing automatic actions to fire if the due date passes.
     (in the heading row)
       <th tal:condition="request/show/due_date">Due Date</th>
     (in the data row)
-      <td tal:condition="request/show/priority" tal:content="i/due_date" />
+      <td tal:condition="request/show/due_date" tal:content="i/due_date" />
 
 4. add the property to the issue.search.html page::
 

roundup/cgi/templating.py

@@ -1596,6 +1596,18 @@ def menu(self, size=None, height=None, showid=0, additional=[],
             sort_on=None, **conditions):
         ''' Render a form select list for this property
 
+            "size" is used to limit the length of the list labels
+            "height" is used to set the <select> tag's "size" attribute
+            "showid" includes the item ids in the list labels
+            "additional" lists properties which should be included in the
+                label
+            "sort_on" indicates the property to sort the list on as
+                (direction, property) where direction is '+' or '-'.
+            
+            The remaining keyword arguments are used as conditions for
+            filtering the items in the list - they're passed as the
+            "filterspec" argument to a Class.filter() call.
+
             If not editable, just display the value via plain().
         '''
         if not self.is_edit_ok():
@@ -1764,7 +1776,19 @@ def field(self, size=30, showid=0):
 
     def menu(self, size=None, height=None, showid=0, additional=[],
             sort_on=None, **conditions):
-        ''' Render a form select list for this property
+        ''' Render a form <select> list for this property.
+
+            "size" is used to limit the length of the list labels
+            "height" is used to set the <select> tag's "size" attribute
+            "showid" includes the item ids in the list labels
+            "additional" lists properties which should be included in the
+                label
+            "sort_on" indicates the property to sort the list on as
+                (direction, property) where direction is '+' or '-'.
+            
+            The remaining keyword arguments are used as conditions for
+            filtering the items in the list - they're passed as the
+            "filterspec" argument to a Class.filter() call.
 
             If not editable, just display the value via plain().
         '''

templates/classic/html/user.index.html

@@ -19,8 +19,7 @@
  <th tal:condition="context/is_edit_ok" i18n:translate="">Retire</th>
 </tr>
 <tal:block repeat="user context/list">
-<tr tal:condition="user/is_view_ok"
-    tal:attributes="class python:['normal', 'alt'][repeat['user'].index%6/3]">
+<tr tal:attributes="class python:['normal', 'alt'][repeat['user'].index%6/3]">
  <td>
   <a tal:attributes="href string:user${user/id}"
      tal:content="user/username">username</a>