[Zope3-checkins] SVN: Zope3/trunk/src/zope/app/workflow/browser/instance.py Removed XXX by quoting the name.

[Stephan Richter]

Log message for revision 26294:

Removed XXX by quoting the name.

Converted XXX to TODO.




-=-
Modified: Zope3/trunk/src/zope/app/workflow/browser/instance.py
===================================================================
--- Zope3/trunk/src/zope/app/workflow/browser/instance.py	2004-07-09 15:44:44 UTC (rev 26293)
+++ Zope3/trunk/src/zope/app/workflow/browser/instance.py	2004-07-09 15:44:46 UTC (rev 26294)
@@ -15,6 +15,7 @@
  
 $Id$
 """
+import urllib
 from zope.schema import getFieldNames
 from zope.app.pagetemplate.viewpagetemplatefile import ViewPageTemplateFile
 
@@ -31,10 +32,7 @@
         info = {}
         info['id'] = id
         info['object'] = obj
-
-        # XXX need to urlencode the id in this case !!!
-        info['url'] = "processinstance.html?pi_name=%s" % id
-
+        info['url'] = "processinstance.html?pi_name=%s" %urllib.quote_plus(id)
         return info
 
     def removeObjects(self, ids):
@@ -63,11 +61,13 @@
 
 
     # ProcessInstance Details
-    # XXX This is temporary till we find a better name to use
-    #     objects that are stored in annotations
-    #     Steve suggested a ++annotations++<key> Namespace for that.
-    #     we really want to traverse to the instance and display a view
 
+    # TODO:
+    # This is temporary till we find a better name to use
+    # objects that are stored in annotations
+    # Steve suggested a ++annotations++<key> Namespace for that.
+    # we really want to traverse to the instance and display a view
+
     def _getProcessInstanceData(self, data):
         names = []
         for interface in providedBy(data):