Merge lp:~akretion-team/openerp-product-attributes/fix-1230290 into lp:~product-core-editors/openerp-product-attributes/7.0

Proposed by arthru
Status: Merged
Merged at revision: 211
Proposed branch: lp:~akretion-team/openerp-product-attributes/fix-1230290
Merge into: lp:~product-core-editors/openerp-product-attributes/7.0
Diff against target: 12 lines (+1/-1)
1 file modified
product_images/product.py (+1/-1)
To merge this branch: bzr merge lp:~akretion-team/openerp-product-attributes/fix-1230290
Reviewer Review Type Date Requested Status
Pedro Manuel Baeza code review, no test Approve
Quentin THEURET @Amaris (community) without functional test Approve
Raphaël Valyi - http://www.akretion.com Approve
Review via email: mp+187520@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Raphaël Valyi - http://www.akretion.com (rvalyi) :
review: Approve
Revision history for this message
Quentin THEURET @Amaris (qtheuret) wrote :

LGTM (without functional test)

review: Approve (without functional test)
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

LGTM... a typographical error ;)

review: Approve (code review, no test)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'product_images/product.py'
2--- product_images/product.py 2013-02-11 22:52:23 +0000
3+++ product_images/product.py 2013-09-25 13:59:30 +0000
4@@ -33,7 +33,7 @@
5 default = {}
6 original = self.read(cr, uid, id, fields=['default_code', 'image_ids'], context=context)
7 default.update({
8- 'images_ids': False,
9+ 'image_ids': False,
10 })
11 local_media_repository = self.pool.get('res.company').get_local_media_repository(cr, uid, context=context)
12 if local_media_repository:

Subscribers

People subscribed via source and target branches