fix field value for hidden edit[s|à] fields tls-sprint
authorsylvain.thenault@logilab.fr
Fri, 20 Feb 2009 10:38:34 +0100
branchtls-sprint
changeset 888 603327e9aef4
parent 887 51e371245bc5
child 889 609edca88406
fix field value for hidden edit[s|à] fields
web/form.py
--- a/web/form.py	Fri Feb 20 01:50:45 2009 +0100
+++ b/web/form.py	Fri Feb 20 10:38:34 2009 +0100
@@ -607,8 +607,10 @@
         """
         fieldname = field.name
         if fieldname.startswith('edits-') or fieldname.startswith('edito-'):
-            value = self.form_field_value(field.visible_field, values)
-            if value is None or not self.entity.has_eid():
+            # edit[s|o]- fieds must have the actual value stored on the entity
+            if self.entity.has_eid():
+                value = self.form_field_entity_value(field, default_initial=False)
+            else:
                 value = INTERNAL_FIELD_VALUE
         elif fieldname == '__type':
             value = self.entity.id
@@ -619,11 +621,11 @@
         elif fieldname in self.req.form:
             value = self.req.form[fieldname]
         else:
-            if self.entity.has_eid():
+            if self.entity.has_eid() and field.eidparam:
                 # use value found on the entity or field's initial value if it's
                 # not an attribute of the entity (XXX may conflicts and get
                 # undesired value)
-                value = getattr(self.entity, fieldname, field.initial)
+                value = self.form_field_entity_value(field, default_initial=True)
             else:
                 defaultattr = 'default_%s' % fieldname
                 if hasattr(self.entity, defaultattr):
@@ -641,6 +643,15 @@
                 values = value()
         return value # field.format_value(self.req, value) 
 
+    def form_field_entity_value(self, field, default_initial=True):
+        attr = field.name 
+        if field.role == 'object':
+            attr += '_object'
+        if default_initial:
+            return getattr(self.entity, attr, field.initial)
+        else:
+            return getattr(self.entity, attr)
+        
     def form_field_name(self, field):
         if field.eidparam:
             return eid_param(field.name, self.entity.eid)