c2c-oerpscenario team mailing list archive

["vra \(openerp\)" <675418@xxxxxxxxxxxxxxxxxx>]

** Changed in: openobject-addons
   Importance: Undecided => Low

** Changed in: openobject-addons
       Status: New => Confirmed

** Changed in: openobject-addons
     Assignee: (unassigned) => OpenERP R&D Addons Team 2 (openerp-dev-addons2)

-- 
account moves with same account in debit and credit
https://bugs.launchpad.net/bugs/675418
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.

Status in OpenObject Addons Modules: Confirmed

Bug description:
In 5.0, stock accounting (in action_done from stock.move) account.move.lines is created only if acc_src! = acc_dest, which is correct. Thereby avoiding the accounting records with the same account in debit and credit.
The 6.0 is different. You can create accounting entries with the same account in debit and credit. I think that is not correct. 
We propose:
    def _create_product_valuation_moves(self, cr, uid, move, context=None):
...
+                if acc_dest != acc_variation:
                    account_moves += [(journal_id, self._create_account_move_line(cr, uid, move, acc_variation, acc_dest, reference_amount, reference_currency_id, context))]

....
+                if acc_src != acc_variation:
                    account_moves += [(journal_id, self._create_account_move_line(cr, uid, move, acc_src, acc_variation, reference_amount, reference_currency_id, context))]
.....