Merge lp:~openerp-dev/openobject-addons/trunk-bug-1190185-dharmraj into lp:openobject-addons

Proposed by Dharmraj Zala(OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-bug-1190185-dharmraj
Merge into: lp:openobject-addons
Diff against target: 25 lines (+8/-0)
1 file modified
sale/sale.py (+8/-0)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/trunk-bug-1190185-dharmraj
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+168925@code.launchpad.net

Description of the change

Hello,

I have fixed this issue by adding appropriate validation/constraint on discount percentage on sale order line.

Thanks,
Dharmraj.

To post a comment you must log in.

Unmerged revisions

8758. By Dharmraj Zala(OpenERP)

[FIX] Added Validation Constraint on discount percentage

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'sale/sale.py'
2--- sale/sale.py 2013-06-07 10:23:59 +0000
3+++ sale/sale.py 2013-06-12 12:02:31 +0000
4@@ -690,6 +690,12 @@
5 sale_order_line sol ON (sol.order_id = rel.order_id)
6 WHERE rel.invoice_id = ANY(%s)""", (list(ids),))
7 return [i[0] for i in cr.fetchall()]
8+
9+ def _check_discount(self, cr, uid, ids, context=None):
10+ for soline in self.browse(cr, uid, ids,context=context):
11+ if soline.discount < 0.0 or soline.discount > 100.00:
12+ return False
13+ return True
14
15 _name = 'sale.order.line'
16 _description = 'Sales Order Line'
17@@ -736,6 +742,8 @@
18 'type': 'make_to_stock',
19 'price_unit': 0.0,
20 }
21+
22+ _constraints = [(_check_discount,'Error! \n Enter Valid Discount (Between 0.0 to 100.0).',['discount']),]
23
24 def _get_line_qty(self, cr, uid, line, context=None):
25 if (line.order_id.invoice_quantity=='order'):

Subscribers

People subscribed via source and target branches

to all changes: