[FIX] crm_livechat: do not set public user partner as lead customer usign /lead command

When chatting with visitors, operator may use /lead command to directly create
a lead from discussion. That way commercial discussion can follow visitors
inquiries.

Anonymous (public) users may be part of channels, notably since f5fe11cf1e74216afea7976d8515d4448f054dd2.
In that case we don't want new leads to be associated with those users. Indeed

  * they are not real users, just technical users for website / frontend;
  * merge processes may think all leads having public users as customer are
    duplicates and should be merged, which means loosing information and
    leads;

We therefore set the following heuristics

  * if a public user is member of a channel -> consider this is a livechat
    with an anonymous and set customer_id to False;
  * otherwise try to find a share partner in channel members and link the lead
    to that partner;

Task ID-2389564

closes odoo/odoo#62006

Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>

1 files changed, 19 insertions, 3 deletions

diff --git a/addons/crm_livechat/models/mail_channel.py b/addons/crm_livechat/models/mail_channel.py
index f608bd1a0dd..b7650342d5a 100644
--- a/addons/crm_livechat/models/mail_channel.py
+++ b/addons/crm_livechat/models/mail_channel.py
@@ -26,16 +26,32 @@ class MailChannel(models.Model):
         self._send_transient_message(partner, msg)
 
     def _convert_visitor_to_lead(self, partner, channel_partners, key):
+        """ Create a lead from channel /lead command
+        :param partner: internal user partner (operator) that created the lead;
+        :param channel_partners: channel members;
+        :param key: operator input in chat ('/lead Lead about Product')
+        """
         description = ''.join(
             '%s: %s\n' % (message.author_id.name or self.anonymous_name, message.body)
             for message in self.channel_message_ids.sorted('id')
         )
+        # if public user is part of the chat: consider lead to be linked to an
+        # anonymous user whatever the participants. Otherwise keep only share
+        # partners (no user or portal user) to link to the lead.
+        customers = self.env['res.partner']
+        for customer in channel_partners.partner_id.filtered('partner_share'):
+            if customer.user_ids and all(user._is_public() for user in customer.user_ids):
+                customers = self.env['res.partner']
+                break
+            else:
+                customers |= customer
+
         utm_source = self.env.ref('crm_livechat.utm_source_livechat', raise_if_not_found=False)
         return self.env['crm.lead'].create({
             'name': html2plaintext(key[5:]),
-            'partner_id': channel_partners.partner_id.id,
-            'user_id': None,
-            'team_id': None,
+            'partner_id': customers[0].id if customers else False,
+            'user_id': False,
+            'team_id': False,
             'description': html2plaintext(description),
             'referred': partner.name,
             'source_id': utm_source and utm_source.id,