[Erp5-report] r10047 - /erp5/trunk/products/ERP5Form/MultiRelationField.py

[nobody at svn.erp5.org]

Author: jp
Date: Sun Sep 17 14:28:51 2006
New Revision: 10047

URL: http://svn.erp5.org?rev=10047&view=rev
Log:
More comments

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

Modified: erp5/trunk/products/ERP5Form/MultiRelationField.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Form/MultiRelationField.py?rev=10047&r1=10046&r2=10047&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Form/MultiRelationField.py (original)
+++ erp5/trunk/products/ERP5Form/MultiRelationField.py Sun Sep 17 14:28:51 2006
@@ -428,14 +428,15 @@
         if self.relation_setter_id:
           relation_setter = getattr(o, self.relation_setter_id)
           relation_setter((), portal_type=self.portal_type_list)
-          relation_setter(relation_uid_list,
-                          portal_type=self.portal_type_list)
+          relation_setter(relation_uid_list,                  # relation setter is uid based
+                          portal_type=self.portal_type_list)  # maybe not the best solution
+                                                              # and inconsistent with bellow
         else:
           # we could call a generic method which create the setter method name
           if len(relation_object_list) == 1:
             set_method_name = '_set%sValue' % \
                          convertToUpperCase(self.base_category)
-            getattr(o, set_method_name)(relation_object_list[0], 
+            getattr(o, set_method_name)(relation_object_list[0],
                                         portal_type=self.portal_type_list)
           else:
             set_method_name = '_set%sValueList' % \