[Erp5-report] r16516 - in /erp5/trunk/products/ERP5Form: ProxyField.py tests/testFields.py

[nobody at svn.erp5.org]

Author: yusei
Date: Thu Sep 20 17:14:52 2007
New Revision: 16516

URL: http://svn.erp5.org?rev=16516&view=rev
Log:
Fixed a bug that ProxyField.get_value raise error when proxyfield does not have template field.

Modified:
    erp5/trunk/products/ERP5Form/ProxyField.py
    erp5/trunk/products/ERP5Form/tests/testFields.py

Modified: erp5/trunk/products/ERP5Form/ProxyField.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Form/ProxyField.py?rev=16516&r1=16515&r2=16516&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Form/ProxyField.py (original)
+++ erp5/trunk/products/ERP5Form/ProxyField.py Thu Sep 20 17:14:52 2007
@@ -513,7 +513,10 @@
     """
       Return a callable expression
     """
-    tales_expr = self.get_tales_expression(id)
+    try:
+      tales_expr = self.get_tales_expression(id)
+    except ValueError:
+      return None
     if tales_expr:
       return TALESValue(tales_expr)
 

Modified: erp5/trunk/products/ERP5Form/tests/testFields.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Form/tests/testFields.py?rev=16516&r1=16515&r2=16516&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Form/tests/testFields.py (original)
+++ erp5/trunk/products/ERP5Form/tests/testFields.py Thu Sep 20 17:14:52 2007
@@ -130,6 +130,8 @@
                       'my_title', 'Not Title', 'ProxyField')
     proxy_field = self.container.Base_view.my_title
     self.assertEquals(None, proxy_field.getTemplateField())
+    self.assertEquals(None, proxy_field.get_value('enable'))
+    self.assertEquals(None, proxy_field.get_value('default'))
 
     proxy_field.manage_edit_xmlrpc(dict(form_id='Base_viewProxyFieldLibrary',
                                         field_id='my_title',))