Merge lp:~pedro.baeza/ocb-addons/ocb-addons-bug-1000776 into lp:ocb-addons/6.1

Proposed by Pedro Manuel Baeza
Status: Merged
Merged at revision: 6753
Proposed branch: lp:~pedro.baeza/ocb-addons/ocb-addons-bug-1000776
Merge into: lp:ocb-addons/6.1
Diff against target: 12 lines (+1/-1)
1 file modified
account/account.py (+1/-1)
To merge this branch: bzr merge lp:~pedro.baeza/ocb-addons/ocb-addons-bug-1000776
Reviewer Review Type Date Requested Status
Holger Brunn (Therp) code review Approve
Cristian Salamea (community) Approve
Stefan Rijnhart (Opener) Approve
Raphaël Valyi - http://www.akretion.com Approve
Review via email: mp+168995@code.launchpad.net

Description of the change

Bug #1000776
[FIX] account: Add an ondelete=cascade on the payment term line

To post a comment you must log in.
Revision history for this message
Raphaël Valyi - http://www.akretion.com (rvalyi) wrote :

of course it should be "cascade"! well spotted!

review: Approve
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :
review: Approve
Revision history for this message
Cristian Salamea (ovnicraft) wrote :

Thanks for fix it !

review: Approve
Revision history for this message
Holger Brunn (Therp) (hbrunn) :
review: Approve (code review)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account/account.py'
2--- account/account.py 2013-04-26 13:10:17 +0000
3+++ account/account.py 2013-06-12 16:28:23 +0000
4@@ -107,7 +107,7 @@
5 'days': fields.integer('Number of Days', required=True, help="Number of days to add before computation of the day of month." \
6 "If Date=15/01, Number of Days=22, Day of Month=-1, then the due date is 28/02."),
7 'days2': fields.integer('Day of the Month', required=True, help="Day of the month, set -1 for the last day of the current month. If it's positive, it gives the day of the next month. Set 0 for net days (otherwise it's based on the beginning of the month)."),
8- 'payment_id': fields.many2one('account.payment.term', 'Payment Term', required=True, select=True),
9+ 'payment_id': fields.many2one('account.payment.term', 'Payment Term', required=True, select=True, ondelete='cascade'),
10 }
11 _defaults = {
12 'value': 'balance',

Subscribers

People subscribed via source and target branches