Xavier ALT (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-server/6.0-opw-513433-xal into 
lp:openobject-server/6.0.

Requested reviews:
  OpenERP Publisher's Warranty Team (openerp-opw)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-server/6.0-opw-513433-xal/+merge/94990

Hi,

This branch fix some common problems with res.log context value bigger than 
250. This happen on object using log() method (by having _log_create = True set 
on object or calling log() manually) mixed with user having big "user context" 
(ex: tz: America/North_Dakota/New_Salem).

Please review this.

Cheers,
Xavier
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/6.0-opw-513433-xal/+merge/94990
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/6.0-opw-513433-xal.
=== modified file 'bin/addons/base/res/res_log.py'
--- bin/addons/base/res/res_log.py	2012-02-28 11:12:15 +0000
+++ bin/addons/base/res/res_log.py	2012-02-28 15:59:30 +0000
@@ -50,7 +50,19 @@
                        self._index_name)
 
     def create(self, cr, uid, vals, context=None):
+
+        def filter_context_value(c):
+            """filter unrequired value from context"""
+            # We remove user context and web client related value as those
+            # values will be re-set when accessing res.log item, depending
+            # on user or client settings
+            if isinstance(c, dict):
+                FILTER_OUT_KEYS = ['tz', 'lang', 'client', 'bin_size', '_terp_view_name']
+                for context_key in FILTER_OUT_KEYS:
+                    c.pop(context_key, None)
+            return c
         create_context = context and dict(context) or {}
+        create_context = filter_context_value(create_context)
         if 'res_log_read' in create_context:
             vals['read'] = create_context.pop('res_log_read')
         if create_context and not vals.get('context'):

_______________________________________________
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