c2c-oerpscenario team mailing list archive

[filsys <office@xxxxxxxxxxxx>]

Public bug reported:

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))]
.....

** Affects: openobject-addons
     Importance: Undecided
         Status: New

-- 
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: New

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))]
.....