[IMP] website_sale_tax_toggle: black, isort

sergio-teruel · CarlosRoca13 · commit c5578212901d · 2022-05-13T11:26:08.000+02:00
diff --git a/website_sale_tax_toggle/__manifest__.py b/website_sale_tax_toggle/__manifest__.py
@@ -1,22 +1,17 @@
 # Copyright 2020 Tecnativa - Sergio Teruel
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 {
-    'name': 'Website Sale Tax Toggle',
-    'summary': 'Allow display price in Shop with or without taxes',
-    'version': '12.0.1.1.1',
-    'development_status': 'Beta',
-    'category': 'Website',
-    'website': 'https://github.com/OCA/e-commerce',
-    'author': 'Tecnativa, Odoo Community Association (OCA)',
-    'license': 'AGPL-3',
-    'application': False,
-    'installable': True,
-    'depends': [
-        'website_sale',
-    ],
-    'data': [
-        'views/assets.xml',
-        'views/templates.xml',
-    ],
+    "name": "Website Sale Tax Toggle",
+    "summary": "Allow display price in Shop with or without taxes",
+    "version": "12.0.1.1.1",
+    "development_status": "Beta",
+    "category": "Website",
+    "website": "https://github.com/OCA/e-commerce",
+    "author": "Tecnativa, Odoo Community Association (OCA)",
+    "license": "AGPL-3",
+    "application": False,
+    "installable": True,
+    "depends": ["website_sale",],
+    "data": ["views/assets.xml", "views/templates.xml",],
     "demo": ["demo/assets.xml"],
 }
diff --git a/website_sale_tax_toggle/controllers/main.py b/website_sale_tax_toggle/controllers/main.py
@@ -1,17 +1,17 @@
 # Copyright 2020 Tecnativa - Sergio Teruel
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
-from odoo.addons.website_sale.controllers.main import WebsiteSale
 from odoo import http
 from odoo.http import request
 
+from odoo.addons.website_sale.controllers.main import WebsiteSale
+
 
 class WebsiteSaleTaxToggle(WebsiteSale):
-
-    @http.route(
-        ['/website/tax_toggle'], type='json', auth="public", website=True)
+    @http.route(["/website/tax_toggle"], type="json", auth="public", website=True)
     def tax_toggle(self):
         # Create a session variable
-        request.session['tax_toggle_taxed'] = not request.session.get(
-            'tax_toggle_taxed', False)
-        return request.session['tax_toggle_taxed']
+        request.session["tax_toggle_taxed"] = not request.session.get(
+            "tax_toggle_taxed", False
+        )
+        return request.session["tax_toggle_taxed"]
diff --git a/website_sale_tax_toggle/demo/assets.xml b/website_sale_tax_toggle/demo/assets.xml
@@ -1,13 +1,13 @@
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8" ?>
 <!-- Copyright 2020 Tecnativa - Jairo Llopis
      License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). -->
 <data>
-
     <template id="assets_frontend_demo" inherit_id="website.assets_frontend">
         <xpath expr=".">
-            <script type="text/javascript"
-                    src="/website_sale_tax_toggle/static/src/js/website_sale_tax_toggle_tour.js"/>
+            <script
+                type="text/javascript"
+                src="/website_sale_tax_toggle/static/src/js/website_sale_tax_toggle_tour.js"
+            />
         </xpath>
     </template>
-
 </data>
diff --git a/website_sale_tax_toggle/models/res_users.py b/website_sale_tax_toggle/models/res_users.py
@@ -10,23 +10,25 @@ class ResUsers(models.Model):
 
     @api.model
     def has_group(self, group_ext_id):
-        tax_included = 'account.group_show_line_subtotals_tax_included'
-        tax_excluded = 'account.group_show_line_subtotals_tax_excluded'
-        if (self.env.context.get('website_id') and
-                group_ext_id in [tax_included, tax_excluded] and
-                not self.env.context.get('skip_tax_toggle_check')):
+        tax_included = "account.group_show_line_subtotals_tax_included"
+        tax_excluded = "account.group_show_line_subtotals_tax_excluded"
+        if (
+            self.env.context.get("website_id")
+            and group_ext_id in [tax_included, tax_excluded]
+            and not self.env.context.get("skip_tax_toggle_check")
+        ):
             taxed = request.session.get(
-                'tax_toggle_taxed',
-                self.env.user.with_context(
-                    skip_tax_toggle_check=True
-                ).has_group(tax_included)
+                "tax_toggle_taxed",
+                self.env.user.with_context(skip_tax_toggle_check=True).has_group(
+                    tax_included
+                ),
             )
             return group_ext_id != (tax_excluded if taxed else tax_included)
         return super().has_group(group_ext_id)
 
     # HACK: To clear cache called from res.users write method
     @api.model
-    @ormcache('self._uid', 'group_ext_id')
+    @ormcache("self._uid", "group_ext_id")
     def _has_group(self, group_ext_id):
         return super()._has_group(group_ext_id)
 
diff --git a/website_sale_tax_toggle/static/src/js/website_sale_tax_toggle.js b/website_sale_tax_toggle/static/src/js/website_sale_tax_toggle.js
@@ -1,28 +1,26 @@
 /* Copyright 2020 Sergio Teruel
  * License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). */
 
-odoo.define("website_sale_tax_toggle.tax_toggle_button", function (require) {
+odoo.define("website_sale_tax_toggle.tax_toggle_button", function(require) {
     "use strict";
 
     var sAnimation = require("website.content.snippets.animation");
 
     sAnimation.registry.tax_toggle_button = sAnimation.Class.extend({
         selector: ".js_tax_toggle_management",
         events: {
-            'click .js_tax_toggle_btn': '_onPublishBtnClick',
+            "click .js_tax_toggle_btn": "_onPublishBtnClick",
         },
-        _onPublishBtnClick: function (ev) {
+        _onPublishBtnClick: function(ev) {
             ev.preventDefault();
             var self = this;
             var $data = $(ev.currentTarget).parents(".js_tax_toggle_management:first");
             this._rpc({
-                route: $data.data('controller'),
-            })
-            .done(function (result) {
-                $data.find('input').prop("checked", result);
+                route: $data.data("controller"),
+            }).done(function(result) {
+                $data.find("input").prop("checked", result);
                 window.location.reload();
             });
         },
     });
-
 });
diff --git a/website_sale_tax_toggle/static/src/js/website_sale_tax_toggle_tour.js b/website_sale_tax_toggle/static/src/js/website_sale_tax_toggle_tour.js
@@ -1,7 +1,7 @@
 /* Copyright 2020 Sergio Teruel
  * License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). */
 
-odoo.define("website_sale_tax_toggle.tour", function (require) {
+odoo.define("website_sale_tax_toggle.tour", function(require) {
     "use strict";
 
     var tour = require("web_tour.tour");
@@ -13,24 +13,29 @@ odoo.define("website_sale_tax_toggle.tour", function (require) {
     var steps = [
         {
             content: "Toggle tax button click from list page",
-            trigger: '.js_tax_toggle_btn',
-            extra_trigger: ".oe_product_cart:contains('Product test tax toggle') b .oe_currency_value:containsExact('750.00')",
+            trigger: ".js_tax_toggle_btn",
+            extra_trigger:
+                ".oe_product_cart:contains('Product test tax toggle') b .oe_currency_value:containsExact('750.00')",
         },
         {
             content: "Enter the product page",
-            trigger: ".oe_product_cart:has(b .oe_currency_value:containsExact('862.50')) a:contains('Product test tax toggle')",
+            trigger:
+                ".oe_product_cart:has(b .oe_currency_value:containsExact('862.50')) a:contains('Product test tax toggle')",
         },
         {
             content: "Toggle tax button click from product page",
-            trigger: '.js_tax_toggle_btn',
-            extra_trigger: "#product_details .oe_price .oe_currency_value:containsExact('862.50')",
+            trigger: ".js_tax_toggle_btn",
+            extra_trigger:
+                "#product_details .oe_price .oe_currency_value:containsExact('862.50')",
         },
         {
             content: "Check the product price is back to what it should",
-            trigger: "#product_details .oe_price .oe_currency_value:containsExact('750.00')",
+            trigger:
+                "#product_details .oe_price .oe_currency_value:containsExact('750.00')",
         },
     ];
-    tour.register("website_sale_tax_toggle",
+    tour.register(
+        "website_sale_tax_toggle",
         {
             url: "/shop",
             test: true,
diff --git a/website_sale_tax_toggle/static/src/scss/website_sale_tax_toggle.scss b/website_sale_tax_toggle/static/src/scss/website_sale_tax_toggle.scss
@@ -12,7 +12,7 @@
 
         + span {
             background-color: $o-we-switch-inactive-color;
-            box-shadow: inset 0 0 0px 1px darken($o-we-switch-inactive-color,10%);
+            box-shadow: inset 0 0 0px 1px darken($o-we-switch-inactive-color, 10%);
             border-radius: 100rem;
             height: $o-we-switch-size;
             width: $o-we-switch-size * 1.81;
@@ -31,7 +31,8 @@
                 margin-left: 0.1ex;
                 border-radius: 100rem;
                 transition: all 0.3s $o-we-md-ease;
-                box-shadow: 0 1px 1px darken($o-we-switch-inactive-color,35%), inset 0 0 0 1px lighten($o-we-switch-inactive-color,10%);
+                box-shadow: 0 1px 1px darken($o-we-switch-inactive-color, 35%),
+                    inset 0 0 0 1px lighten($o-we-switch-inactive-color, 10%);
             }
         }
 
@@ -40,7 +41,7 @@
             background: $o-we-color-success;
 
             &:after {
-                margin-left: ($o-we-switch-size*1.8 - $o-we-switch-size) + 0.1;
+                margin-left: ($o-we-switch-size * 1.8 - $o-we-switch-size) + 0.1;
             }
         }
     }
diff --git a/website_sale_tax_toggle/tests/test_website_sale_tax_toggle.py b/website_sale_tax_toggle/tests/test_website_sale_tax_toggle.py
@@ -4,35 +4,37 @@
 
 
 class WebsiteSaleTaxesToggleHttpCase(HttpCase):
-
     def setUp(self):
         super().setUp()
         # Get company for Mitchel Admin user
-        self.user_admin = self.env.ref('base.user_admin')
+        self.user_admin = self.env.ref("base.user_admin")
         user_company = self.user_admin.company_id
-        self.tax = self.env['account.tax'].create({
-            'name': 'Taxes toggle test tax',
-            'amount_type': 'percent',
-            'amount': 15,
-            'type_tax_use': 'sale',
-            'company_id': user_company.id,
-        })
-        self.product_template = self.env['product.template'].create({
-            'name': 'Product test tax toggle',
-            'list_price': 750.00,
-            'taxes_id': [(6, 0, self.tax.ids)],
-            'website_published': True,
-            'website_sequence': 9999,
-        })
-        pricelist = self.env['product.pricelist'].create({
-            'name': 'Price list for tests',
-            'currency_id': user_company.currency_id.id
-        })
+        self.tax = self.env["account.tax"].create(
+            {
+                "name": "Taxes toggle test tax",
+                "amount_type": "percent",
+                "amount": 15,
+                "type_tax_use": "sale",
+                "company_id": user_company.id,
+            }
+        )
+        self.product_template = self.env["product.template"].create(
+            {
+                "name": "Product test tax toggle",
+                "list_price": 750.00,
+                "taxes_id": [(6, 0, self.tax.ids)],
+                "website_published": True,
+                "website_sequence": 9999,
+            }
+        )
+        pricelist = self.env["product.pricelist"].create(
+            {"name": "Price list for tests", "currency_id": user_company.currency_id.id}
+        )
         self.env.user.partner_id.property_product_pricelist = pricelist
         # To avoid currency converter
-        self.env['res.currency.rate'].search([]).write({
-            'rate': 1,
-        })
+        self.env["res.currency.rate"].search([]).write(
+            {"rate": 1,}
+        )
 
     def test_ui_website(self):
         """Test frontend tour."""
@@ -44,5 +46,5 @@ def test_ui_website(self):
             url_path="/shop",
             code="%s.run('%s')" % tour,
             ready="%s.tours['%s'].ready" % tour,
-            login="admin"
+            login="admin",
         )
diff --git a/website_sale_tax_toggle/views/assets.xml b/website_sale_tax_toggle/views/assets.xml
@@ -1,11 +1,16 @@
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8" ?>
 <odoo>
     <template id="assets_frontend" inherit_id="website.assets_frontend">
         <xpath expr=".">
-            <script type="text/javascript"
-                    src="/website_sale_tax_toggle/static/src/js/website_sale_tax_toggle.js"/>
-            <link type="text/scss" rel="stylesheet"
-                  href="/website_sale_tax_toggle/static/src/scss/website_sale_tax_toggle.scss"/>
+            <script
+                type="text/javascript"
+                src="/website_sale_tax_toggle/static/src/js/website_sale_tax_toggle.js"
+            />
+            <link
+                type="text/scss"
+                rel="stylesheet"
+                href="/website_sale_tax_toggle/static/src/scss/website_sale_tax_toggle.scss"
+            />
         </xpath>
     </template>
 </odoo>
diff --git a/website_sale_tax_toggle/views/templates.xml b/website_sale_tax_toggle/views/templates.xml
@@ -1,35 +1,48 @@
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8" ?>
 <!-- Copyright 2020 Tecnativa - Sergio Teruel
      License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). -->
 <odoo>
-
     <template id="tax_toggle_template">
-        <t t-set="taxed" t-value="request.session.get('tax_toggle_taxed', request.env.user.with_context(skip_tax_toggle_check=True).has_group('account.group_show_line_subtotals_tax_included'))"/>
+        <t
+            t-set="taxed"
+            t-value="request.session.get('tax_toggle_taxed', request.env.user.with_context(skip_tax_toggle_check=True).has_group('account.group_show_line_subtotals_tax_included'))"
+        />
         <div class="js_tax_toggle_management" data-controller="/website/tax_toggle">
             <label class="o_switch o_switch_danger js_tax_toggle_btn">
-                <input type="checkbox" t-att-checked="taxed"/>
-                <span/>
+                <input type="checkbox" t-att-checked="taxed" />
+                <span />
                 <span>Show prices with taxes included</span>
             </label>
         </div>
     </template>
-
-    <template id="products" inherit_id="website_sale.products" active="True" customize_show="True" name="Taxes toggle" priority="999">
+    <template
+        id="products"
+        inherit_id="website_sale.products"
+        active="True"
+        customize_show="True"
+        name="Taxes toggle"
+        priority="999"
+    >
         <xpath expr="//div[hasclass('products_pager')]" position="after">
             <div class="row mt16 mb0">
                 <div class="col">
                     <div class="pull-right">
-                        <t t-call="website_sale_tax_toggle.tax_toggle_template"/>
+                        <t t-call="website_sale_tax_toggle.tax_toggle_template" />
                     </div>
                 </div>
             </div>
         </xpath>
     </template>
-
-    <template id="product" inherit_id="website_sale.product" active="True" customize_show="True" name="Taxes toggle" priority="999">
+    <template
+        id="product"
+        inherit_id="website_sale.product"
+        active="True"
+        customize_show="True"
+        name="Taxes toggle"
+        priority="999"
+    >
         <xpath expr="//t[@t-call='website_sale.product_price']" position="before">
-            <t t-call="website_sale_tax_toggle.tax_toggle_template"/>
+            <t t-call="website_sale_tax_toggle.tax_toggle_template" />
         </xpath>
     </template>
-
 </odoo>

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@`
`12`	`12`
`13`	`13`	`+ span {`
`14`	`14`	`background-color: $o-we-switch-inactive-color;`
`15`		`- box-shadow: inset 0 0 0px 1px darken($o-we-switch-inactive-color,10%);`
	`15`	`+ box-shadow: inset 0 0 0px 1px darken($o-we-switch-inactive-color, 10%);`
`16`	`16`	`border-radius: 100rem;`
`17`	`17`	`height: $o-we-switch-size;`
`18`	`18`	`width: $o-we-switch-size * 1.81;`
`@@ -31,7 +31,8 @@`
`31`	`31`	`margin-left: 0.1ex;`
`32`	`32`	`border-radius: 100rem;`
`33`	`33`	`transition: all 0.3s $o-we-md-ease;`
`34`		`- box-shadow: 0 1px 1px darken($o-we-switch-inactive-color,35%), inset 0 0 0 1px lighten($o-we-switch-inactive-color,10%);`
	`34`	`+ box-shadow: 0 1px 1px darken($o-we-switch-inactive-color, 35%),`
	`35`	`+ inset 0 0 0 1px lighten($o-we-switch-inactive-color, 10%);`
`35`	`36`	`}`
`36`	`37`	`}`
`37`	`38`
`@@ -40,7 +41,7 @@`
`40`	`41`	`background: $o-we-color-success;`
`41`	`42`
`42`	`43`	`&:after {`
`43`		`- margin-left: ($o-we-switch-size*1.8 - $o-we-switch-size) + 0.1;`
	`44`	`+ margin-left: ($o-we-switch-size * 1.8 - $o-we-switch-size) + 0.1;`
`44`	`45`	`}`
`45`	`46`	`}`
`46`	`47`	`}`