Merge lp:~openerp-dev/openobject-addons/6.1-opw-587330-han into lp:openobject-addons/6.1

Proposed by Hardik Ansodariya (OpenERP)
Status: Rejected
Rejected by: Chris Biersbach (OpenERP)
Proposed branch: lp:~openerp-dev/openobject-addons/6.1-opw-587330-han
Merge into: lp:openobject-addons/6.1
Diff against target: 19 lines (+2/-0)
1 file modified
account/account.py (+2/-0)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.1-opw-587330-han
Reviewer Review Type Date Requested Status
Naresh(OpenERP) Pending
Review via email: mp+152341@code.launchpad.net

Description of the change

Hello,

Fixed the issue: Copy of Journal Entry remains linked to Bank Statement.

Description:

If a journal entry (account.move) is created via a bank statement then the entry lines (account.move.line) have the statement_id field populated.

If such a journal entry is then duplicated using the copy() method of account.move then the lines on the copy also have their statement_id fields populated and so the copy appears against the bank statement.

The duplicate journal entry should not appear against the original bank statement.

With reference of Maintenance case: 587330

Thanks

To post a comment you must log in.
Revision history for this message
Chris Biersbach (OpenERP) (cbi-openerp) wrote :

I am rejecting this because 6.1 is not an LTS and we no longer merge into it.

Unmerged revisions

7170. By Hardik Ansodariya (OpenERP)

[FIX]: account.move: Fixed the issue of Copy of Journal Entry remains linked to Bank Statement(Maintenance Case: 587330)

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-03-01 09:01:02 +0000
3+++ account/account.py 2013-03-08 08:44:28 +0000
4@@ -1399,6 +1399,7 @@
5 'ref':False,
6 'balance':False,
7 'account_tax_id':False,
8+ 'statement_id': False,
9 })
10
11 if 'journal_id' in vals and vals.get('journal_id', False):
12@@ -1432,6 +1433,7 @@
13 context = {}
14 default.update({
15 'state':'draft',
16+ 'ref': False,
17 'name':'/',
18 })
19 context.update({