Merge lp:~ajite/openobject-addons/elico-7.0-add-0002 into lp:~openerp-community/openobject-addons/elico-7.0

Status: Merged
Approved by: Eric Caudal - www.elico-corp.com
Approved revision: no longer in the source branch.
Merged at revision: 31
Proposed branch: lp:~ajite/openobject-addons/elico-7.0-add-0002
Merge into: lp:~openerp-community/openobject-addons/elico-7.0
Diff against target: 34 lines (+11/-6)
1 file modified
base_intercompany/unit/export_synchronizer.py (+11/-6)
To merge this branch: bzr merge lp:~ajite/openobject-addons/elico-7.0-add-0002
Reviewer Review Type Date Requested Status
LIN Yu Pending
Review via email: mp+207813@code.launchpad.net

Description of the change

[FIX] Fixed a bug that was preventing a user to create normal SO/PO

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'base_intercompany/unit/export_synchronizer.py'
2--- base_intercompany/unit/export_synchronizer.py 2014-02-18 12:28:09 +0000
3+++ base_intercompany/unit/export_synchronizer.py 2014-02-23 01:07:25 +0000
4@@ -128,19 +128,24 @@
5 for icops in self._get_icops():
6 backend = self._get_backend_with_permission(icops)
7 self._set_icops(icops, backend)
8- try:
9- map_record = self._map_data(fields=fields)
10- except MappingError as e:
11- continue
12+ map_record = self._map_data(fields=fields)
13 if self.icops_ids:
14- record = self._update_data(map_record, fields=fields)
15+ record = None
16+ try:
17+ record = self._update_data(map_record, fields=fields)
18+ except MappingError as e:
19+ continue
20 if not record:
21 continue
22 nb_records += 1
23 self._validate_data(record)
24 self._routing(record, fields)
25 else:
26- record = self._create_data(map_record, fields=fields)
27+ record = None
28+ try:
29+ record = self._create_data(map_record, fields=fields)
30+ except MappingError as e:
31+ continue
32 if not record:
33 continue
34 nb_records += 1