[Erp5-report] r9328 - /erp5/trunk/products/ERP5Form/ListBox.py

nobody at svn.erp5.org nobody at svn.erp5.org
Tue Aug 22 16:34:26 CEST 2006


Author: jp
Date: Tue Aug 22 16:34:24 2006
New Revision: 9328

URL: http://svn.erp5.org?rev=9328&view=rev
Log:
Required to support different renderable types in editable fields. (ex. lists, tokens, etc.)

Modified:
    erp5/trunk/products/ERP5Form/ListBox.py

Modified: erp5/trunk/products/ERP5Form/ListBox.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Form/ListBox.py?rev=9328&r1=9327&r2=9328&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Form/ListBox.py (original)
+++ erp5/trunk/products/ERP5Form/ListBox.py Tue Aug 22 16:34:24 2006
@@ -50,6 +50,7 @@
 
 import md5
 import cgi
+import types
 
 # For compatibility with Python 2.3.
 try:
@@ -1962,7 +1963,7 @@
           # XXX (JPS) - render_view does not get REQUEST - this breaks so many possibilities
           REQUEST = get_request() # Dirtymax hack by JPS - render_view API update required
           REQUEST.cell = self.getObject()
-          cell_html = editable_field.render( value   = display_value
+          cell_html = editable_field.render( value   = original_value
                                            , REQUEST = brain
                                            , key     = key
                                            )
@@ -2905,6 +2906,9 @@
   security.declareProtected('Access contents information', 'getListMethodName')
   def getListMethodName(self):
     """Return the name of the list method. If not defined, return None.
+
+       XXX - Is this method really necessary - I am not sure - JPS
+       Why not use Formulator API instead ?
     """
     list_method = self.get_value('list_method')
     try:




More information about the Erp5-report mailing list