Mayur Maheshwari(OpenERP) has proposed merging 
lp:~openerp-dev/openobject-addons/trunk-opw-575458-port-mma into 
lp:openobject-addons.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-opw-575458-port-mma/+merge/132450

Hello,

When we have installed delivery and sale module and try to subscribe log for 
sale.order,
it raises key error for 'id'.

Steps to produce:
1. install sale and delivery modules
2. create log rule for sale.order
3. create a sales order and perform any operation, it will raise key error.

Delivery module adds 'id' field into sale.order and in audittrail while we 
fetch data(from get_data()),
we explicitly skip storing value for 'id' field and then it tries to log for 
'id'.

I have put additional check while logging the line.
Code is forward port from 6.1

Thanks
Mayur
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-opw-575458-port-mma/+merge/132450
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-opw-575458-port-mma.
=== modified file 'audittrail/audittrail.py'
--- audittrail/audittrail.py	2012-10-02 11:12:31 +0000
+++ audittrail/audittrail.py	2012-11-01 05:36:22 +0000
@@ -388,6 +388,8 @@
         }
         # loop on all the fields
         for field_name, field_definition in pool.get(model.model)._all_columns.items():
+            if field_name in ('__last_update', 'id'):
+                continue
             #if the field_list param is given, skip all the fields not in that list
             if field_list and field_name not in field_list:
                 continue

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to