Merge lp:~camptocamp/c2c-addons/6.0-hr_timesheet_reminder-fix-lp1089390 into lp:c2c-addons/6.0

[Guewen Baconnier @ Camptocamp]

FIX] never use a mutable in the second argument of fromkeys

fix lp:1089390

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

LGTM

review: Approve (code review, no test)

Unmerged revisions

65. By Guewen Baconnier @ Camptocamp <email address hidden> on 2012-12-13

[FIX] never use a mutable in the second argument of fromkeys

=== modified file 'hr_timesheet_reminder/company.py'
--- hr_timesheet_reminder/company.py	2011-09-20 09:48:06 +0000
+++ hr_timesheet_reminder/company.py	2012-12-13 08:19:25 +0000
@@ -42,16 +42,15 @@
 
     def get_reminder_recipients(self, cr, uid, ids, context=None):
         """Return the list of users that must receive the email"""
-        res = {}.fromkeys(ids, [])
+        res = dict((company_id, []) for company_id in ids)
         employee_obj = self.pool.get('hr.employee')
 
-        companies = self.browse(cr, uid, ids, context=context)
-
-        for company in companies:
-            employee_ids = employee_obj.search(cr, uid,
-                                [('company_id', '=', company.id),
-                                 ('active', '=', True)],
-                                context=context)
+        for company in self.browse(cr, uid, ids, context=context):
+            employee_ids = employee_obj.search(
+                    cr, uid,
+                    [('company_id', '=', company.id),
+                     ('active', '=', True)],
+                    context=context)
             employees = employee_obj.browse(cr, uid, employee_ids, context=context)
 
             #periods