banking-addons-team team mailing list archive

["Stefan Rijnhart \(Therp\)" <stefan@xxxxxxxx>]

Stefan Rijnhart (Therp) has proposed merging lp:~therp-nl/banking-addons/ba61-lp870242-reference_not_on_journal_entry into lp:banking-addons/6.1.

Requested reviews:
  Banking Addons Core Editors (banking-addons-team)
Related bugs:
  Bug #870242 in Banking Addons: "Bank Statement - Reference does not appear in journal entry"
  https://bugs.launchpad.net/banking-addons/+bug/870242

For more details, see:
https://code.launchpad.net/~therp-nl/banking-addons/ba61-lp870242-reference_not_on_journal_entry/+merge/183104
-- 
https://code.launchpad.net/~therp-nl/banking-addons/ba61-lp870242-reference_not_on_journal_entry/+merge/183104
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~therp-nl/banking-addons/ba61-lp870242-reference_not_on_journal_entry into lp:banking-addons/6.1.

=== modified file 'account_banking/account_banking.py'
--- account_banking/account_banking.py	2013-03-20 19:25:57 +0000
+++ account_banking/account_banking.py	2013-08-30 09:17:25 +0000
@@ -501,6 +501,7 @@
             'period_id': period_id, # AB
             'date': st_line.date,
             'name': st_line_number,
+            'ref': st_line.ref,
         }, context=context)
         account_bank_statement_line_obj.write(cr, uid, [st_line.id], {
             'move_ids': [(4, move_id, False)]
@@ -524,7 +525,6 @@
         val = {
             'name': st_line.name,
             'date': st_line.date,
-            'ref': st_line.ref,
             'move_id': move_id,
             'partner_id': (((st_line.partner_id) and st_line.partner_id.id) or
                            False),
@@ -566,7 +566,6 @@
         account_move_line_obj.create(cr, uid, {
             'name': st_line.name,
             'date': st_line.date,
-            'ref': st_line.ref,
             'move_id': move_id,
             'partner_id': (((st_line.partner_id) and st_line.partner_id.id) or
                            False),