From 63fbe72b9cc8f5a2552c24d86e91ec13e1cf7eca Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= <tde@openerp.com>
Date: Fri, 7 Sep 2012 18:08:41 +0200
Subject: [PATCH] [FIX] mail: res_users: fixed the previous hook, done directly
 in SQL, to avoid doing sunversive things in _auto_init.

bzr revid: tde@openerp.com-20120907160841-f5ej19t7cdml10pr
---
 addons/mail/res_users.py | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/addons/mail/res_users.py b/addons/mail/res_users.py
index 254d29aaad4..a5aea54bf9b 100644
--- a/addons/mail/res_users.py
+++ b/addons/mail/res_users.py
@@ -58,9 +58,16 @@ class res_users(osv.Model):
         """Installation hook to create aliases for all users and avoid constraint errors."""
         self.pool.get('mail.alias').migrate_to_alias(cr, self._name, self._table, super(res_users, self)._auto_init,
             self._columns['alias_id'], 'login', alias_force_key='id', context=context)
-        # make already existing users follow themselves
-        for user in self.browse(cr, SUPERUSER_ID, self.search(cr, SUPERUSER_ID, [], context=context), context=context):
-            self.pool.get('res.partner').message_subscribe(cr, SUPERUSER_ID, [user.partner_id.id], [user.partner_id.id], context=context)
+        # make already existing users follow themselves, using read to avoid unprefetched fields browse failure
+        cr.execute("""  SELECT p.id FROM res_partner p
+                        LEFT JOIN mail_followers n
+                        ON (n.partner_id = p.id AND n.res_model = 'res.partner' AND n.res_id = p.id)
+                        WHERE n.id IS NULL
+                    """)
+        params = [(res[0], res[0]) for res in cr.fetchall()]
+        cr.executemany("""  INSERT INTO mail_followers (partner_id, res_model, res_id)
+                            VALUES (%s, 'res.partner', %s)
+                        """, params)
 
     def create(self, cr, uid, data, context=None):
         # create default alias same as the login