[IMP] shopinvader: black, isort, prettier

.pre-commit-config.yaml

@@ -3,7 +3,6 @@ exclude: |
   # NOT INSTALLABLE ADDONS
   ^partner_contact_company/|
   ^product_online_category/|
-  ^shopinvader/|
   ^shopinvader_algolia/|
   ^shopinvader_assortment/|
   ^shopinvader_backend_image_proxy/|

setup/shopinvader/odoo/addons/shopinvader

@@ -0,0 +1 @@
+../../../../shopinvader

setup/shopinvader/setup.py

@@ -0,0 +1,6 @@
+import setuptools
+
+setuptools.setup(
+    setup_requires=['setuptools-odoo'],
+    odoo_addon=True,
+)

shopinvader/__manifest__.py

@@ -11,7 +11,7 @@
     "author": "Akretion",
     "license": "AGPL-3",
     "application": True,
-    "installable": False,
+    "installable": True,
     "external_dependencies": {"python": ["cerberus", "unidecode"], "bin": []},
     "depends": [
         "base_rest",

shopinvader/components/access_info.py

@@ -42,9 +42,7 @@ def for_address(self, address_id):
         if self.partner_user is not None:
             if not self.is_main_partner():
                 if not self.is_owner(address_id):
-                    info.update(
-                        {"read": True, "update": False, "delete": False}
-                    )
+                    info.update({"read": True, "update": False, "delete": False})
                 else:
                     # only main partner can delete your address
                     info["delete"] = False

shopinvader/components/partner_validator.py

@@ -21,9 +21,7 @@ def validate_partner(self, partner):
         if self.backend.validate_customers:
             validator = getattr(
                 self,
-                "_validate_partner_{}".format(
-                    self.backend.validate_customers_type
-                ),
+                "_validate_partner_{}".format(self.backend.validate_customers_type),
                 lambda partner: True,
             )
             validator(partner)
@@ -36,10 +34,7 @@ def _validate_partner_all(self, partner):
             pass
 
     def _validate_partner_address(self, partner):
-        if (
-            not partner.shopinvader_enabled
-            and not partner.address_type == "address"
-        ):
+        if not partner.shopinvader_enabled and not partner.address_type == "address":
             # raise PartnerNotValidatedError(
             #     "Address found but not validated yet."
             # )

shopinvader/components/product_product_event_listener.py

@@ -28,12 +28,8 @@ def on_record_create(self, record, fields=None):
         :param fields: list of str
         :return:
         """
-        shopinv_products = record.mapped(
-            "product_tmpl_id.shopinvader_bind_ids"
-        )
-        shopinv_variants = self._launch_shopinvader_variant_creation(
-            shopinv_products
-        )
+        shopinv_products = record.mapped("product_tmpl_id.shopinvader_bind_ids")
+        shopinv_variants = self._launch_shopinvader_variant_creation(shopinv_products)
         # If at least 1 is True, force False
         if any(shopinv_variants.mapped("active")):
             shopinv_variants.write({"active": False})

shopinvader/controllers/main.py

@@ -7,10 +7,11 @@
 import logging
 
 from odoo import _
-from odoo.addons.base_rest.controllers import main
 from odoo.exceptions import MissingError
 from odoo.http import request, route
 
+from odoo.addons.base_rest.controllers import main
+
 _logger = logging.getLogger(__name__)
 
 
@@ -23,9 +24,7 @@ class InvaderController(main.RestController):
     @route(["/shopinvader/<service>/<int:_id>/download"], methods=["GET"])
     def service_download(self, service, _id=None, **params):
         params["id"] = _id
-        return self._process_method(
-            service, "download", _id=_id, params=params
-        )
+        return self._process_method(service, "download", _id=_id, params=params)
 
     @classmethod
     def _get_partner_from_headers(cls, headers):
@@ -42,9 +41,7 @@ def _get_partner_from_headers(cls, headers):
                 if len(partner) > 1:
                     _logger.warning(
                         "More than one shopinvader.partner found for:"
-                        " backend_id={} email={}".format(
-                            backend.id, partner_email
-                        )
+                        " backend_id={} email={}".format(backend.id, partner_email)
                     )
                 # Could be because the email is not related to a partner or
                 # because the partner is inactive
@@ -57,9 +54,7 @@ def _find_partner(cls, backend, partner_email):
             ("partner_email", "=", partner_email),
             ("backend_id", "=", backend.id),
         ]
-        return request.env["shopinvader.partner"].search(
-            partner_domain, limit=2
-        )
+        return request.env["shopinvader.partner"].search(partner_domain, limit=2)
 
     @classmethod
     def _validate_partner(cls, backend, partner):
@@ -92,9 +87,7 @@ def _get_component_context(self):
         * shopinvader_backend: current shopinvader backend (matching API key)
         """
         res = super(InvaderController, self)._get_component_context()
-        res[
-            "shopinvader_backend"
-        ] = self._get_shopinvader_backend_from_request()
+        res["shopinvader_backend"] = self._get_shopinvader_backend_from_request()
         headers = request.httprequest.environ
         # TODO: all services should rely on shopinvader partner
         # rather than the real partner
@@ -104,7 +97,5 @@ def _get_component_context(self):
         res["partner_user"] = partner
         # The partner user for the main account or for sale order may differ.
         res["partner"] = partner.get_shop_partner(res["shopinvader_backend"])
-        res[
-            "shopinvader_session"
-        ] = self._get_shopinvader_session_from_headers(headers)
+        res["shopinvader_session"] = self._get_shopinvader_session_from_headers(headers)
         return res

shopinvader/data/cart_step.xml

@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="UTF-8"?>
+<?xml version="1.0" encoding="UTF-8" ?>
 <odoo>
 
     <record id="cart_index" model="shopinvader.cart.step">