From 3a4ebf326618ae30ede150aba0e322ec018a7820 Mon Sep 17 00:00:00 2001
From: Olivier Dony <odo@openerp.com>
Date: Wed, 16 May 2012 18:31:22 +0200
Subject: [PATCH] [IMP] ir.filters: default user_id = uid ; comment about
 cryptic code

(Some minor updates after XMO's review)

bzr revid: odo@openerp.com-20120516163122-jsb3n2eo2i253o00
---
 openerp/addons/base/ir/ir_filters.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/openerp/addons/base/ir/ir_filters.py b/openerp/addons/base/ir/ir_filters.py
index 6ee67667ebe..02a723c116c 100644
--- a/openerp/addons/base/ir/ir_filters.py
+++ b/openerp/addons/base/ir/ir_filters.py
@@ -53,6 +53,9 @@ class ir_filters(osv.osv):
         lower_name = vals['name'].lower()
         matching_filters = [f for f in self.get_filters(cr, uid, vals['model_id'])
                                 if f['name'].lower() == lower_name
+                                # next line looks for matching user_ids (specific or global), i.e.
+                                # f.user_id is False and vals.user_id is False or missing,
+                                # or f.user_id.id == vals.user_id
                                 if (f['user_id'] and f['user_id'][0]) == vals.get('user_id', False)]
         # When a filter exists for the same (name, model, user) triple, we simply
         # replace its definition.
@@ -88,6 +91,7 @@ class ir_filters(osv.osv):
     _defaults = {
         'domain': '[]',
         'context':'{}',
+        'user_id': lambda self,cr,uid,context=None: uid,
     }
 
 ir_filters()