Invoice totals not updated when a line is moved (store={...} bug?)

[Numérigraphe]

In 5.0.10, if you record an invoice with a line, and then you move the line to another invoice (with XML-RPC or custom object code), then the amounts of the invoices are not updated.
Lionel

[Borja López Soilán (NeoPolus) (borjals)]

It's weird, cause the 'total' field of the invoices is 'watching' for changes on the invoice lines. The recomputation of the amount_ total field should be triggered if the lines of the invoice (the "invoice_line" field) change:

---
...
'invoice_line': fields.one2many('account.invoice.line', 'invoice_id', 'Invoice Lines', readonly=True, states={'draft':[('readonly',False)]}),
...
'amount_ total': fields.function(_amount_all, method=True, digits=(16, int(config['price_accuracy'])), string='Total',
            store={
                'account.invoice': (lambda self, cr, uid, ids, c={}: ids, ['invoice_line'], 20),
                ...
            },
            multi='all'),
...
---

Maybe the problem here is that the change in the many2one (invoice_id on the lines) is not triggering the 'store' triggers on the one2many side (invoice_line in the invoice) as we would expect.

[Numérigraphe (numerigraphe)]

Borja López Soilán (Pexego), I'm adding this bug to the server too, based on your latest comment.
Lionel.

[Borja López Soilán (NeoPolus) (borjals)]

By the way, this bug may be a duplicate of, or related to bug 589578

[Borja López Soilán (NeoPolus) (borjals)]

Numérigraphe, take also a look at the mra comment on bug 58988:

"And debit, credit, balance fields on account.analytic.account are function field with store=True but it does not work properly. For e.g if you create invoice with analytic account X, Its debit,credit,balance entry does not updated (I mean its functions _balance_calc, _debit_calc, ... not called) ...."

Looks like an orm problem: updating the many2one side (analytic lines in mra comment / invoice lines here) does not trigger the store computation on the one2many side (analytic account / invoice) even when the one2many field really changed.

[Jay Vora (Serpent Consulting Services) (jayvora)]

Hi Borja and Lionel,

Thanks for the analysis,we are investigating more for the same.

[Jay Vora (Serpent Consulting Services) (jayvora)]

This seems like an issue related to server.
Discussions are invited.
Thanks.

[Meera Trambadia (OpenERP) (mtr-openerp)]

Hello Borja and Lionel,

Its fixed in https://code.launchpad.net/~openerp-commiter/openobject-addons/trunk-dev-addons3-mtr1.
Revision ID: mtr@mtr-20101213122857-j1co587gjezuy92w
Revision no: 4648

It will be merged soon in the lp:openobject-addons.

Thanks
mtr

[Cristian Salamea (ovnicraft)]

@mtr IMHO for a better project status, commits in addons1,2,3 must be set the status as Fix Commited when merged in lp:openobject-addons change it to Fix Released.

My 2c,