audit trail - changes in o2m resources are NOT logged

Bug #832635 reported by Ferdinand on 2011-08-24
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Odoo Server (MOVED TO GITHUB)
Status tracked in Trunk
6.0
Won't Fix
Low
OpenERP Publisher's Warranty Team
Trunk
New
Low
OpenERP's Framework R&D

Bug Description

example
want to log changes in invoice lines
set a audit log for
* account_invoice
* account_invoice_line

changes of invoice lines produce
* an empty entry in account_invoice
* no entry for invoice_lines

Related branches

Changed in openobject-server:
assignee: nobody → OpenERP's Framework R&D (openerp-dev-framework)
importance: Undecided → Low
status: New → Confirmed
Changed in openobject-server:
assignee: OpenERP's Framework R&D (openerp-dev-framework) → OpenERP Publisher's Warranty Team (openerp-opw)
tags: added: maintenance
Naresh(OpenERP) (nch-openerp) wrote :

Hello,

The Audit trail module needs heavy refactoring. due to current code which makes many extra and unnecessary calls to db needs to re-factored to speed up logging. currently its very slow.

I would also assign the same to trunk version as major improvements will be in trunk

@ OPW: you can fix the bug for the stable version. The improvements has already started in trunk.

Thanks

Naresh(OpenERP) (nch-openerp) wrote :

Hello,

Thanks for reporting !

The bug contains the fix in the merge proposal lp:~openerp-dev/openobject-addons/trunk-bug-832635-nch along with refactoring and improvements !

please provide your feedback !

Thanks,

The complexity of the proposed fix makes it inadequate for the stable version.
It needs proper testing in the trunk before being released.

I propose to keep the fix for the trunk, and not fix version 6.0.

Naresh(OpenERP) (nch-openerp) wrote :

Hello Raphael,

yes you are right ! It complex but it was really needed to refactore the whole module because it was vary badly coded. with many calls to database and much more... so the proposed branched indeed can be seen as "a fix + improvements + refactoring"

Thanks,

Hello,

By taking reference from Raphael Collet comment #4 I am closing this issue for version 6.

Thanks.

qdp (OpenERP) (qdp) wrote :

merge proposal needs fixing

Naresh(OpenERP) (nch-openerp) wrote :

merge proposal improved please re review !

This has been fixed in 5754 revid:<email address hidden>

Ferdinand (office-chricar) wrote :

actually nothing is logged using demo data logging of partner
- and yes the rule is "subscribed" and openerp restarted (in case of)

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers