Merge lp:~camptocamp/openerp-swiss-localization/rem-copy-reference-invoice into lp:~camptocamp/openerp-swiss-localization/7.0

Proposed by Vincent Renaville@camptocamp
Status: Rejected
Rejected by: Joël Grand-Guillaume @ camptocamp
Proposed branch: lp:~camptocamp/openerp-swiss-localization/rem-copy-reference-invoice
Merge into: lp:~camptocamp/openerp-swiss-localization/7.0
Diff against target: 14 lines (+0/-4)
1 file modified
l10n_ch_payment_slip/invoice.py (+0/-4)
To merge this branch: bzr merge lp:~camptocamp/openerp-swiss-localization/rem-copy-reference-invoice
Reviewer Review Type Date Requested Status
Yannick Vaucher @ Camptocamp Needs Information
Nicolas Bessi - Camptocamp (community) Disapprove
Review via email: mp+176386@code.launchpad.net

Description of the change

remove unuseful copy that set reference on invoice to null, it cause pro
blem on duplication of customer invoice with recurrent reference

To post a comment you must log in.
Revision history for this message
Nicolas Bessi - Camptocamp (nbessi-c2c-deactivatedaccount) wrote :

As discussed with Fred won't fix without a more contextual details.

review: Disapprove
Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote :

Where is the bug report? :)

review: Needs Information

Unmerged revisions

177. By Vincent Renaville@camptocamp

[FIX] remove unuseful copy that set reference on invoice to null, it cause problem on duplication of customer invoice with recurrent reference

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'l10n_ch_payment_slip/invoice.py'
2--- l10n_ch_payment_slip/invoice.py 2013-05-29 07:43:23 +0000
3+++ l10n_ch_payment_slip/invoice.py 2013-07-23 14:06:48 +0000
4@@ -102,10 +102,6 @@
5 (ref, move_id.id))
6 return res
7
8- def copy(self, cursor, uid, inv_id, default=None, context=None):
9- default = default or {}
10- default.update({'reference': False})
11- return super(AccountInvoice, self).copy(cursor, uid, inv_id, default, context)
12
13
14 class AccountTaxCode(Model):

Subscribers

People subscribed via source and target branches

to all changes: