c2c-oerpscenario team mailing list archive

["ksa\(OpenERP\)" <ksa@xxxxxxxxxxx>]

Hello filsys ,

it has been fixed in branch lp:~openerp-dev/openobject-addons/ksa-addons2
Revision ID: ksa@xxxxxxxxxxxxx-20101118124504-m9c14n0rrxmuw9cj
Revision NO: 3853

Thanks.

** Changed in: openobject-addons
       Status: Confirmed => Fix Released

-- 
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: Fix Released

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