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
1=== modified file 'base_external_referentials/external_referentials.py' (properties changed: -x to +x)
2--- base_external_referentials/external_referentials.py 2012-02-10 12:42:14 +0000
3+++ base_external_referentials/external_referentials.py 2012-02-11 18:20:22 +0000
4@@ -381,8 +381,10 @@
5 ),
6 }
7
8- _default = {
9+ _defaults = {
10 'type' : lambda * a: 'in_out',
11+ 'external_type': lambda *a: 'unicode',
12+ 'evaluation_type': lambda *a: 'direct',
13 }
14
15 def _check_mapping_line_name(self, cr, uid, ids):