[trunk/7.0]Issue in suggested recipient functionality

[Sodexis]

In addons/mail/mail_thread.py/_message_add_suggested_recipient method,
The condition for checking already existing partner ID is not correct.

OpenERP Version 7.0

TEST CASE:

 Just for the simulation of this bug , I mentioned this example. In Real-Time, this example situation will never occur.

Create a lead with Customer= Administrator and Salesperson=Administrator. Click on Send a message option for that
document. It will show Administrator (<email address hidden>) twice in To: Followers of this document as suggested recipient. Make sure that the Administrator is not in the follower list.

CURRENT CODE:
if partner and partner in [val[0] for val in result[obj.id]]:

val[0] --gives--> partner_id. So its needs to be checked with partner.id instead of partner object.

CORRECTION:

The code should be
if partner and partner.id in [val[0] for val in result[obj.id]]:

Thanks.

[Dharmraj Zala(OpenERP) (dharmraj.zala-openerp)]

Hello Sodexis,

It has been fix commited in https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-1248589-dja
Revision ID : <email address hidden>
Revision No : 9020

Thanks,
dja