Merge lp:~mathieu-julius/openobject-extension/openobject-extension-add-all-fields into lp:openobject-extension/oerp6.1-stable

Proposed by Mathieu Vatel - Julius Network Solutions
Status: Merged
Merge reported by: Sébastien BEAU - http://www.akretion.com
Merged at revision: not available
Proposed branch: lp:~mathieu-julius/openobject-extension/openobject-extension-add-all-fields
Merge into: lp:openobject-extension/oerp6.1-stable
Diff against target: 15 lines (+3/-1)
1 file modified
base_external_referentials/external_referentials.py (+3/-1)
To merge this branch: bzr merge lp:~mathieu-julius/openobject-extension/openobject-extension-add-all-fields
Reviewer Review Type Date Requested Status
Sébastien BEAU - http://www.akretion.com Approve
Review via email: mp+92630@code.launchpad.net

Description of the change

I've change _default to _defaults in external_mapping_line class
+ I've added 'evaluation_type' and 'external_type' in defaults because there are now required. This fixing the error when click on "Add all object fields"

To post a comment you must log in.
Revision history for this message
Sébastien BEAU - http://www.akretion.com (sebastien.beau) :
review: Approve
Revision history for this message
Sébastien BEAU - http://www.akretion.com (sebastien.beau) wrote :

merge in main branch rev 193
Thank you mathieu

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'base_external_referentials/external_referentials.py' (properties changed: -x to +x)
--- base_external_referentials/external_referentials.py 2012-02-10 12:42:14 +0000
+++ base_external_referentials/external_referentials.py 2012-02-11 18:20:22 +0000
@@ -381,8 +381,10 @@
381 ),381 ),
382 }382 }
383 383
384 _default = {384 _defaults = {
385 'type' : lambda * a: 'in_out',385 'type' : lambda * a: 'in_out',
386 'external_type': lambda *a: 'unicode',
387 'evaluation_type': lambda *a: 'direct',
386 }388 }
387 389
388 def _check_mapping_line_name(self, cr, uid, ids):390 def _check_mapping_line_name(self, cr, uid, ids):