[IMP] pos_ticket_send_by_mail: pre-commit auto fixes

Author: phucph

pos_ticket_send_by_mail/__manifest__.py

@@ -4,24 +4,24 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html
 
 {
-    'name': 'POS Receipt By Email',
-    'version': '12.0.1.0.0',
-    'category': 'Custom',
-    'author': 'Druidoo',
-    'website': 'https://www.druidoo.io',
-    'license': 'AGPL-3',
-    'depends': [
-        'point_of_sale',
+    "name": "POS Receipt By Email",
+    "version": "12.0.1.0.0",
+    "category": "Custom",
+    "author": "Druidoo",
+    "website": "https://github.com/OCA/pos",
+    "license": "AGPL-3",
+    "depends": [
+        "point_of_sale",
     ],
-    'data': [
-        'security/ir.model.access.csv',
-        'views/report_paperformat.xml',
-        'views/view_pos_config_settings.xml',
-        'data/email_template_data.xml',
-        'data/ir_cron_data.xml',
-        'views/view_res_partner.xml',
-        'views/report_receipt.xml',
-        'static/src/xml/templates.xml',
+    "data": [
+        "security/ir.model.access.csv",
+        "views/report_paperformat.xml",
+        "views/view_pos_config_settings.xml",
+        "data/email_template_data.xml",
+        "data/ir_cron_data.xml",
+        "views/view_res_partner.xml",
+        "views/report_receipt.xml",
+        "static/src/xml/templates.xml",
     ],
-    'installable': True,
+    "installable": True,
 }

pos_ticket_send_by_mail/data/email_template_data.xml

@@ -1,25 +1,31 @@
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8" ?>
 <odoo noupdate="1">
-
     <!--Email template -->
     <record id="email_send_pos_receipt" model="mail.template">
         <field name="name">Send Received</field>
-        <field name="email_from">ticket@${object.user_id.company_id.email.split('@')[1] or ''|safe}</field>
-        <field name="subject">${object.user_id.company_id.name} Received ${object.pos_reference and object.pos_reference.split(' ')[1] or ''}</field>
+        <field
+            name="email_from"
+        >ticket@${object.user_id.company_id.email.split('@')[1] or ''|safe}</field>
+        <field
+            name="subject"
+        >${object.user_id.company_id.name} Received ${object.pos_reference and object.pos_reference.split(' ')[1] or ''}</field>
         <field name="partner_to">${object.partner_id.id}</field>
         <field name="model_id" ref="point_of_sale.model_pos_order" />
         <field name="auto_delete" eval="True" />
-        <field name="report_template" ref="pos_ticket_send_by_mail.action_report_pos_receipt" />
+        <field
+            name="report_template"
+            ref="pos_ticket_send_by_mail.action_report_pos_receipt"
+        />
         <field name="report_name">Ticket ${object.pos_reference}</field>
         <field name="lang">${object.partner_id.lang}</field>
-        <field name="body_html"><![CDATA[
+        <field
+            name="body_html"
+        ><![CDATA[
             <p>Thank you for your visit !</p>
             <p>You will find your receipt ${object.pos_reference and object.pos_reference.split(' ')[1] or ''} attached.<p/>
             <p>Sincerely,</p>
             <p>The team of the cooperative</p>
                         ]]>
         </field>
     </record>
-
 </odoo>
-

pos_ticket_send_by_mail/data/ir_cron_data.xml

@@ -1,15 +1,13 @@
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8" ?>
 <odoo noupdate="1">
-
     <record id="ir_cron_sent_pos_ticket" model="ir.cron">
         <field name="name">Send Reciept via Email</field>
         <field name="interval_number">1</field>
         <field name="interval_type">minutes</field>
         <field name="numbercall">-1</field>
         <field name="doall" eval="False" />
-        <field name="model_id" ref="point_of_sale.model_pos_order"/>
+        <field name="model_id" ref="point_of_sale.model_pos_order" />
         <field name="code">model._send_order_cron()</field>
         <field name="state">code</field>
     </record>
-
 </odoo>

pos_ticket_send_by_mail/models/pos_order.py

@@ -3,29 +3,31 @@
 # @author: La Louve
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html
 
-from odoo import models, fields, api
 import logging
+
+from odoo import api, fields, models
+
 _logger = logging.getLogger(__name__)
 
 
 class PosOrder(models.Model):
-    _inherit = 'pos.order'
+    _inherit = "pos.order"
 
-    email_status = fields.Selection([
-        ('no_send', 'Do not Send'),
-        ('to_send', 'To send'),
-        ('sent', 'Sent')],
-        default="no_send", copy=False, string="Send Status")
+    email_status = fields.Selection(
+        [("no_send", "Do not Send"), ("to_send", "To send"), ("sent", "Sent")],
+        default="no_send",
+        copy=False,
+        string="Send Status",
+    )
 
     @api.model
     def _send_order_cron(self):
         _logger.info("------------------------------------------------------")
-        mail_template = self.env.ref(
-            "pos_ticket_send_by_mail.email_send_pos_receipt")
+        mail_template = self.env.ref("pos_ticket_send_by_mail.email_send_pos_receipt")
         _logger.info("Start to send ticket")
-        for order in self.search([('email_status', '=', 'to_send')]):
+        for order in self.search([("email_status", "=", "to_send")]):
             mail_template.send_mail(order.id, force_send=True)
-            order.email_status = 'sent'
+            order.email_status = "sent"
             # Make sure we commit the change to not send ticket twice
             self.env.cr.commit()
 
@@ -36,15 +38,18 @@ def action_pos_order_paid(self):
         # that we config in pos.config.settings
         #  receipt_option = 1: Don't send e-receipt
         #  receipt_option = 2 or 3: Send e-receipt
-        res = super(PosOrder, self).action_pos_order_paid()
+        res = super().action_pos_order_paid()
         self._set_order_to_send()
         return res
 
     def _set_order_to_send(self):
-        icp_sudo = self.env['ir.config_parameter'].sudo()
-        receipt_options = icp_sudo.get_param('point_of_sale.receipt_options')
+        icp_sudo = self.env["ir.config_parameter"].sudo()
+        receipt_options = icp_sudo.get_param("point_of_sale.receipt_options")
         receipt_options = receipt_options and int(receipt_options) or False
         for order in self:
-            if receipt_options in [2, 3, 4] and order.partner_id.email and \
-                    not order.partner_id.no_email_pos_receipt:
-                order.email_status = 'to_send'
+            if (
+                receipt_options in [2, 3, 4]
+                and order.partner_id.email
+                and not order.partner_id.no_email_pos_receipt
+            ):
+                order.email_status = "to_send"

pos_ticket_send_by_mail/models/res_config_settings.py

@@ -3,44 +3,51 @@
 # @author: La Louve
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html
 
-from odoo import models, fields, api
+from odoo import api, fields, models
 
 
 class ResConfigSettings(models.TransientModel):
-    _inherit = 'res.config.settings'
+    _inherit = "res.config.settings"
 
     receipt_options = fields.Selection(
         [
-            ('1', 'Do not send receipt via email'),
-            ('2', 'Email receipt and print it'),
-            ('3', 'Email receipt and print it unless configured on user that \
-                   he only receives electronically'),
-            ('4', 'Email receipt')
-        ], string="Receipt",
-        config_parameter='point_of_sale.receipt_options'
+            ("1", "Do not send receipt via email"),
+            ("2", "Email receipt and print it"),
+            (
+                "3",
+                "Email receipt and print it unless configured on user that \
+                   he only receives electronically",
+            ),
+            ("4", "Email receipt"),
+        ],
+        string="Receipt",
+        config_parameter="point_of_sale.receipt_options",
     )
 
     @api.model
     def get_values(self):
-        res = super(ResConfigSettings, self).get_values()
-        icp_sudo = self.env['ir.config_parameter'].sudo()
-        receipt_options = icp_sudo.get_param('point_of_sale.receipt_options')
-        res.update(
-            receipt_options=receipt_options
-        )
+        res = super().get_values()
+        icp_sudo = self.env["ir.config_parameter"].sudo()
+        receipt_options = icp_sudo.get_param("point_of_sale.receipt_options")
+        res.update(receipt_options=receipt_options)
         return res
 
     @api.model
     def get_default_receipt_options(self):
-        receipt_options = self.env['ir.values'].get_default(
-            'res.config.settings', 'receipt_options')
+        receipt_options = self.env["ir.values"].get_default(
+            "res.config.settings", "receipt_options"
+        )
         return {
-            'receipt_options': receipt_options,
+            "receipt_options": receipt_options,
         }
 
     @api.multi
     def set_default_receipt_options(self):
         self.ensure_one()
-        return self.env['ir.values'].sudo().set_default(
-            'res.config.settings', 'receipt_options',
-            self.receipt_options or None)
+        return (
+            self.env["ir.values"]
+            .sudo()
+            .set_default(
+                "res.config.settings", "receipt_options", self.receipt_options or None
+            )
+        )

pos_ticket_send_by_mail/models/res_partner.py

@@ -13,18 +13,18 @@ class ResPartner(models.Model):
         string="E-receipt",
         default=False,
         help="If you tick this box and option 3 is selected for 'Receipt'\
-         in point of sale settings, the user will only receive e-receipt ")
+         in point of sale settings, the user will only receive e-receipt ",
+    )
     no_email_pos_receipt = fields.Boolean(
         string="No E-receipt",
         default=False,
-        help="If you tick this box, the user will not receive e-receipt ")
-    config_receipt_options = fields.Integer(
-        compute="_compute_config_receipt_options"
+        help="If you tick this box, the user will not receive e-receipt ",
     )
+    config_receipt_options = fields.Integer(compute="_compute_config_receipt_options")
 
     def _compute_config_receipt_options(self):
-        icp_sudo = self.env['ir.config_parameter'].sudo()
-        receipt_options = icp_sudo.get_param('point_of_sale.receipt_options')
+        icp_sudo = self.env["ir.config_parameter"].sudo()
+        receipt_options = icp_sudo.get_param("point_of_sale.receipt_options")
         receipt_options = receipt_options and int(receipt_options) or 0
         for record in self:
             record.config_receipt_options = receipt_options

pos_ticket_send_by_mail/pyproject.toml

@@ -0,0 +1,3 @@
+[build-system]
+requires = ["whool"]
+build-backend = "whool.buildapi"

pos_ticket_send_by_mail/static/src/js/pos_model.js

@@ -6,19 +6,23 @@
 
 */
 
-odoo.define('pos_ticket_send_by_mail.pos_model', function (require) {
+odoo.define("pos_ticket_send_by_mail.pos_model", function (require) {
     "use strict";
-    var pos_model = require('point_of_sale.models');
+    var pos_model = require("point_of_sale.models");
     pos_model.load_fields("res.partner", "email_pos_receipt");
-    pos_model.load_models([{
-        model: "res.config.settings",
-        fields: ["receipt_options"],
-        loaded: function(self, config_settings){
-            const config_setting = config_settings.length > 0 ? config_settings.reduce(function(prev, current) {
-                return (prev.id > current.id) ? prev : current
-            }) : false; //returns object
-            self.config_settings = config_setting;
-        }
-    }]);
-
+    pos_model.load_models([
+        {
+            model: "res.config.settings",
+            fields: ["receipt_options"],
+            loaded: function (self, config_settings) {
+                const config_setting =
+                    config_settings.length > 0
+                        ? config_settings.reduce(function (prev, current) {
+                              return prev.id > current.id ? prev : current;
+                          })
+                        : false; // Returns object
+                self.config_settings = config_setting;
+            },
+        },
+    ]);
 });

pos_ticket_send_by_mail/static/src/js/receipt_screen_widget.js

@@ -7,41 +7,43 @@
 
 */
 
-odoo.define('pos_ticket_send_by_mail.receipt_screen_widget', function (require) {
-    
+odoo.define("pos_ticket_send_by_mail.receipt_screen_widget", function (require) {
     "use strict";
-    var models = require('point_of_sale.models');
-    var screens = require('point_of_sale.screens');
-    var core = require('web.core');
-    var gui = require('point_of_sale.gui');
+    var models = require("point_of_sale.models");
+    var screens = require("point_of_sale.screens");
+    var core = require("web.core");
+    var gui = require("point_of_sale.gui");
     var _t = core._t;
 
     screens.ReceiptScreenWidget.include({
-
-        print_web: function() {
+        print_web: function () {
             var client = this.pos.get_client();
-            var email_pos_receipt = client ? client.email_pos_receipt: false;
-            var receipt_options = this.pos.config_settings ? this.pos.config_settings.receipt_options : false;
-            if (receipt_options && receipt_options == '3' && email_pos_receipt) {
+            var email_pos_receipt = client ? client.email_pos_receipt : false;
+            var receipt_options = this.pos.config_settings
+                ? this.pos.config_settings.receipt_options
+                : false;
+            if (receipt_options && receipt_options == "3" && email_pos_receipt) {
                 console.log("Skip print receipt by web");
-                return
-            } else {
-            console.log("1 print receipt by web");
-                return this._super();
+                return;
             }
+                console.log("1 print receipt by web");
+                return this._super();
+
         },
 
-        print_xml: function() {
+        print_xml: function () {
             var client = this.pos.get_client();
-            var email_pos_receipt = client ? client.email_pos_receipt: false;
-            var receipt_options = this.pos.config_settings ? this.pos.config_settings.receipt_options : false;
-            if (receipt_options && receipt_options == '3' && email_pos_receipt) {
+            var email_pos_receipt = client ? client.email_pos_receipt : false;
+            var receipt_options = this.pos.config_settings
+                ? this.pos.config_settings.receipt_options
+                : false;
+            if (receipt_options && receipt_options == "3" && email_pos_receipt) {
                 console.log("Skip print receipt by web");
-                return
-            } else {
-            console.log(" print receipt by web");
-                return this._super();
+                return;
             }
+                console.log(" print receipt by web");
+                return this._super();
+
         },
     });
 });

pos_ticket_send_by_mail/static/src/xml/templates.xml

@@ -1,13 +1,15 @@
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8" ?>
 <odoo>
-
     <template id="assets_frontend" inherit_id="point_of_sale.assets">
         <xpath expr="." position="inside">
-            <script type="text/javascript"
-                src="/pos_ticket_send_by_mail/static/src/js/pos_model.js" />
-            <script type="text/javascript"
-                src="/pos_ticket_send_by_mail/static/src/js/receipt_screen_widget.js" />
+            <script
+                type="text/javascript"
+                src="/pos_ticket_send_by_mail/static/src/js/pos_model.js"
+            />
+            <script
+                type="text/javascript"
+                src="/pos_ticket_send_by_mail/static/src/js/receipt_screen_widget.js"
+            />
         </xpath>
     </template>
-
 </odoo>