Merge lp:~magentoerpconnect-community/magentoerpconnect/no-bom-defined-yet into lp:magentoerpconnect/oerp6.1-oldstable

Status: Merged
Merge reported by: Guewen Baconnier @ Camptocamp
Merged at revision: not available
Proposed branch: lp:~magentoerpconnect-community/magentoerpconnect/no-bom-defined-yet
Merge into: lp:magentoerpconnect/oerp6.1-oldstable
Diff against target: 12 lines (+2/-0)
1 file modified
magentoerpconnect/ (+2/-0)
To merge this branch: bzr merge lp:~magentoerpconnect-community/magentoerpconnect/no-bom-defined-yet
Reviewer Review Type Date Requested Status
Guewen Baconnier @ Camptocamp Approve
Review via email:
To post a comment you must log in.
554. By Maxime Chambreuil (


555. By Maxime Chambreuil (

[FIX] local variable 'quantities' referenced before assignment if no BOM defined

Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

Thanks Maxime

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'magentoerpconnect/'
2--- magentoerpconnect/ 2012-03-14 12:23:14 +0000
3+++ magentoerpconnect/ 2012-03-14 13:07:22 +0000
4@@ -1178,6 +1178,8 @@
5 quantities.update({sku: x[1]})
6 if child_ids: #it is an assembly and it contains the products child_ids:
7 self.ext_export(cr, uid, child_ids, external_referential_ids, defaults, context) #so we export them
8+ else:
9+ return False
10 else:
11 logger.notifyChannel('ext synchro', netsvc.LOG_ERROR, "OpenERP 'grouped' products will export to Magento as 'grouped products' only if they have a BOM and if the 'mrp' BOM module is installed")
12 return quantities, child_ids