Merge lp:~elbati/sale-wkfl/sale_delivery_term_ondelete_constraints into lp:~sale-core-editors/sale-wkfl/7.0

Proposed by Lorenzo Battistini
Status: Merged
Merged at revision: 11
Proposed branch: lp:~elbati/sale-wkfl/sale_delivery_term_ondelete_constraints
Merge into: lp:~sale-core-editors/sale-wkfl/7.0
Diff against target: 30 lines (+4/-2)
1 file modified
sale_delivery_term/sale.py (+4/-2)
To merge this branch: bzr merge lp:~elbati/sale-wkfl/sale_delivery_term_ondelete_constraints
Reviewer Review Type Date Requested Status
Guewen Baconnier @ Camptocamp code review, no test Approve
Review via email: mp+160117@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

LGTM

review: Approve (code review, no test)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'sale_delivery_term/sale.py'
--- sale_delivery_term/sale.py 2013-03-15 09:21:42 +0000
+++ sale_delivery_term/sale.py 2013-04-22 14:06:23 +0000
@@ -49,7 +49,7 @@
49 _name = 'sale.delivery.term.line'49 _name = 'sale.delivery.term.line'
50 _rec_name = 'term_id'50 _rec_name = 'term_id'
51 _columns = {51 _columns = {
52 'term_id': fields.many2one('sale.delivery.term', 'Term'),52 'term_id': fields.many2one('sale.delivery.term', 'Term', ondelete='cascade'),
53 'quantity_perc': fields.float('Quantity percentage', required=True, help="For 20% set '0.2'"),53 'quantity_perc': fields.float('Quantity percentage', required=True, help="For 20% set '0.2'"),
54 'delay': fields.float('Delivery Lead Time', required=True,54 'delay': fields.float('Delivery Lead Time', required=True,
55 help="Number of days between the order confirmation and the shipping of the products to the customer"),55 help="Number of days between the order confirmation and the shipping of the products to the customer"),
@@ -65,6 +65,7 @@
65 if res_dict['value'].has_key('type'):65 if res_dict['value'].has_key('type'):
66 del res_dict['value']['type']66 del res_dict['value']['type']
67 if res_dict['value'].has_key('tax_id'):67 if res_dict['value'].has_key('tax_id'):
68 res_dict['value']['tax_ids'] = res_dict['value']['tax_id']
68 del res_dict['value']['tax_id']69 del res_dict['value']['tax_id']
69 return res_dict70 return res_dict
70 71
@@ -113,7 +114,8 @@
113 _name = 'sale.order.line.master'114 _name = 'sale.order.line.master'
114 _columns = {115 _columns = {
115 'order_id': fields.many2one('sale.order', 'Order Reference', required=True, ondelete='cascade'),116 'order_id': fields.many2one('sale.order', 'Order Reference', required=True, ondelete='cascade'),
116 'delivery_term_id': fields.many2one('sale.delivery.term', 'Delivery term', required=True),117 'delivery_term_id': fields.many2one('sale.delivery.term', 'Delivery term',
118 required=True, ondelete='restrict'),
117 'name': fields.char('Description', size=256, required=True),119 'name': fields.char('Description', size=256, required=True),
118 'product_id': fields.many2one('product.product', 'Product', domain=[('sale_ok', '=', True)]),120 'product_id': fields.many2one('product.product', 'Product', domain=[('sale_ok', '=', True)]),
119 'price_unit': fields.float('Unit Price', required=True, digits_compute= dp.get_precision('Sale Price')),121 'price_unit': fields.float('Unit Price', required=True, digits_compute= dp.get_precision('Sale Price')),

Subscribers

People subscribed via source and target branches