account moves with same account in debit and credit

[filsys]

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

[Kirti Savalia(OpenERP) (ksa-openerp)]

Hello filsys ,

it has been fixed in branch lp:~openerp-dev/openobject-addons/ksa-addons2
Revision ID: <email address hidden>
Revision NO: 3853

Thanks.