Add a warning about hardcoded values.

Add a forgotten security-related column.
Word-wrap at 80-car.


git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@17373 20353a03-c40f-0410-a6d1-a30d3c3de9de

product/ERP5Type/patches/WorkflowTool.py

@@ -543,8 +543,12 @@ def WorkflowTool_refreshWorklistCache(self):
     portal_catalog = getToolByName(self, 'portal_catalog')
     search_result = portal_catalog.unrestrictedSearchResults
     acceptable_key_dict = portal_catalog.getSQLCatalog().getColumnMap()
-    table_column_id_set = ImmutableSet([COUNT_COLUMN_TITLE, 'security_uid', 'simulation_state', 'validation_state', 'portal_type', 'owner'])
-    security_column_id_list = ['security_uid']
+    # XXX: those hardcoded lists should be grabbed from the table dynamicaly
+    # (and cached).
+    table_column_id_set = ImmutableSet([
+      COUNT_COLUMN_TITLE, 'security_uid', 'simulation_state',
+      'validation_state', 'portal_type', 'owner'])
+    security_column_id_list = ['security_uid', 'owner']
     (worklist_list_grouped_by_condition, worklist_metadata) = \
       groupWorklistListByCondition(
         worklist_dict=worklist_dict,