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
=== modified file 'account/account.py'
--- account/account.py 2013-04-26 13:10:17 +0000
+++ account/account.py 2013-06-12 16:28:23 +0000
@@ -107,7 +107,7 @@
107 'days': fields.integer('Number of Days', required=True, help="Number of days to add before computation of the day of month." \107 'days': fields.integer('Number of Days', required=True, help="Number of days to add before computation of the day of month." \
108 "If Date=15/01, Number of Days=22, Day of Month=-1, then the due date is 28/02."),108 "If Date=15/01, Number of Days=22, Day of Month=-1, then the due date is 28/02."),
109 '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)."),109 '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)."),
110 'payment_id': fields.many2one('account.payment.term', 'Payment Term', required=True, select=True),110 'payment_id': fields.many2one('account.payment.term', 'Payment Term', required=True, select=True, ondelete='cascade'),
111 }111 }
112 _defaults = {112 _defaults = {
113 'value': 'balance',113 'value': 'balance',

Subscribers

People subscribed via source and target branches