[MIG] mail_gategay_telegram: Migration to v17

WIP for test workflows of mail_gateway migration in #1614

Author: chienandalu

mail_gateway_telegram/__manifest__.py

@@ -5,7 +5,7 @@
     "name": "Mail Telegram Gateway",
     "summary": """
         Set a gateway for telegram""",
-    "version": "16.0.1.0.1",
+    "version": "17.0.1.0.1",
     "license": "AGPL-3",
     "author": "Creu Blanca,Dixmit,Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/social",

mail_gateway_telegram/models/__init__.py

@@ -1,3 +1,3 @@
-from . import mail_gateway_channel
+from . import discuss_channel
 from . import mail_gateway
 from . import mail_gateway_telegram

mail_gateway_telegram/models/mail_gateway_channel.py renamed to mail_gateway_telegram/models/discuss_channel.py

@@ -7,8 +7,8 @@
 from odoo.addons.base.models.avatar_mixin import get_hsl_from_seed
 
 
-class MailChannel(models.Model):
-    _inherit = "mail.channel"
+class DiscussChannel(models.Model):
+    _inherit = "discuss.channel"
 
     def _generate_avatar_gateway(self):
         if self.gateway_id.gateway_type == "telegram":

mail_gateway_telegram/models/mail_gateway_telegram.py

@@ -130,15 +130,16 @@ def _get_telegram_attachment_name(self, attachment):
     async def _process_telegram_attachment(self, attachment):
         if isinstance(attachment, tuple):
             attachment = attachment[-1]
-            # That might happen with images, we will get the last one as it is the bigger one.
+            # That might happen with images, we will get the last one as it is the
+            # bigger one.
         if isinstance(
             attachment,
             (
-                telegram.Game,
-                telegram.Invoice,
-                telegram.Location,
-                telegram.SuccessfulPayment,
-                telegram.Venue,
+                telegram.Game
+                | telegram.Invoice
+                | telegram.Location
+                | telegram.SuccessfulPayment
+                | telegram.Venue
             ),
         ):
             return
@@ -190,12 +191,9 @@ def _process_update(self, chat, update):
                 effective_attachment = current_attachment
             if isinstance(effective_attachment, telegram.Location):
                 body += (
-                    '<a target="_blank" href="https://www.google.com/'
-                    'maps/search/?api=1&query=%s,%s">Location</a>'
-                    % (
-                        effective_attachment.latitude,
-                        effective_attachment.longitude,
-                    )
+                    f'<a target="_blank" href="https://www.google.com/'
+                    f"maps/search/?api=1&query={effective_attachment.latitude},"
+                    f'{effective_attachment.longitude}">Location</a>'
                 )
             attachment_data = asyncio.run(
                 self._process_telegram_attachment(effective_attachment)
@@ -204,7 +202,12 @@ def _process_update(self, chat, update):
                 attachments.append(attachment_data)
         if len(body) > 0 or attachments:
             author = self._get_author(chat.gateway_id, update)
-            new_message = chat.message_post(
+            if author._name == "mail.guest":
+                chat = chat.with_user(self.env.ref("base.public_user").id).with_context(
+                    guest=author
+                )
+            # TODO: Check the sudo...
+            new_message = chat.sudo().message_post(
                 body=body,
                 author_id=author._name == "res.partner" and author.id,
                 gateway_type="telegram",

mail_gateway_telegram/static/src/components/message/message.xml

@@ -1,10 +1,10 @@
 <?xml version="1.0" encoding="UTF-8" ?>
 <templates xml:space="preserve">
-    <t t-inherit="mail.MessageViewNotification" t-inherit-mode="extension">
+    <t t-inherit="mail.Message.notification" t-inherit-mode="extension">
         <xpath expr="//div" position="after">
             <i
                 class="fa fa-paper-plane text-info"
-                t-if="messageView.message.gateway_type == 'telegram' and messageView.message.notifications.length == 0"
+                t-if="message.gateway_type == 'telegram' and message.notifications.length == 0"
             />
         </xpath>
     </t>

mail_gateway_telegram/views/mail_gateway.xml

@@ -12,14 +12,15 @@
             <field name="has_new_channel_security" position="after">
                 <field
                     name="telegram_security_key"
-                    attrs="{'invisible': ['|', ('gateway_type', '!=', 'telegram'), ('has_new_channel_security', '=', False)], 'required': [('gateway_type', '=', 'telegram'), ('has_new_channel_security', '=', True)]}"
+                    invisible="gateway_type != 'telegram' or not has_new_channel_security"
+                    required="gateway_type == 'telegram' and has_new_channel_security"
                 />
             </field>
             <notebook position="inside">
                 <page
                     name="telegram"
                     string="Telegram configuration"
-                    attrs="{'invisible': [('gateway_type', '!=', 'telegram')]}"
+                    invisible="gateway_type != 'telegram'"
                 >
                     <div>
                         <ol>