[IMP] delivery_estimated_package_quantity_by_weight: pre-commit auto fixes

Author: BhaveshHeliconia

delivery_estimated_package_quantity_by_weight/README.rst

@@ -17,32 +17,33 @@ Delivery Estimated Package Quantity By Weight
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fdelivery--carrier-lightgray.png?logo=github
-    :target: https://github.com/OCA/delivery-carrier/tree/16.0/delivery_estimated_package_quantity_by_weight
+    :target: https://github.com/OCA/delivery-carrier/tree/18.0/delivery_estimated_package_quantity_by_weight
     :alt: OCA/delivery-carrier
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-delivery_estimated_package_quantity_by_weight
+    :target: https://translation.odoo-community.org/projects/delivery-carrier-18-0/delivery-carrier-18-0-delivery_estimated_package_quantity_by_weight
     :alt: Translate me on Weblate
 .. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
-    :target: https://runboat.odoo-community.org/builds?repo=OCA/delivery-carrier&target_branch=16.0
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/delivery-carrier&target_branch=18.0
     :alt: Try me on Runboat
 
 |badge1| |badge2| |badge3| |badge4| |badge5|
 
 This module computes the amount of packages a picking out should have
-depending on the weight of the products and the limit fixed by the carrier.
-It's fully independent of the delivery_package_number module.
+depending on the weight of the products and the limit fixed by the
+carrier. It's fully independent of the delivery_package_number module.
 
-A warning is given if the number of packages for the picking out is above what
-is considered as the theoretical number of packages for this picking and the
-chosen carrier. The goal is to minimize the number of packages billed by the
-carrier.
+A warning is given if the number of packages for the picking out is
+above what is considered as the theoretical number of packages for this
+picking and the chosen carrier. The goal is to minimize the number of
+packages billed by the carrier.
 
 The chosen strategy for the theoretical number of packages is as follow:
- * Split the product_weights into as many items as we have
- * Try to fit the heaviest product with the lightest.
-   If it does not work then the heaviest should have a box for itself
- * While the weight of products does not exceed the limit, continue adding
-   products in the same package
+
+- Split the product_weights into as many items as we have
+- Try to fit the heaviest product with the lightest. If it does not work
+  then the heaviest should have a box for itself
+- While the weight of products does not exceed the limit, continue
+  adding products in the same package
 
 **Table of contents**
 
@@ -53,36 +54,37 @@ Usage
 =====
 
 Here are some limitations of the module:
- * Product packagings are not supported
- * Multiple package types having different max weight for a shipping method
-   are not supported
+
+- Product packagings are not supported
+- Multiple package types having different max weight for a shipping
+  method are not supported
 
 Bug Tracker
 ===========
 
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/delivery-carrier/issues>`_.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-`feedback <https://github.com/OCA/delivery-carrier/issues/new?body=module:%20delivery_estimated_package_quantity_by_weight%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/delivery-carrier/issues/new?body=module:%20delivery_estimated_package_quantity_by_weight%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 
 Do not contact contributors directly about support or help with technical issues.
 
 Credits
 =======
 
 Authors
-~~~~~~~
+-------
 
 * ACSONE SA/NV
 
 Contributors
-~~~~~~~~~~~~
+------------
 
-* Lindsay Marion <lindsay.marion@acsone.eu>
-* Hughes Damry <hughes.damry@acsone.eu>
+- Lindsay Marion <lindsay.marion@acsone.eu>
+- Hughes Damry <hughes.damry@acsone.eu>
 
 Maintainers
-~~~~~~~~~~~
+-----------
 
 This module is maintained by the OCA.
 
@@ -94,6 +96,6 @@ OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.
 
-This module is part of the `OCA/delivery-carrier <https://github.com/OCA/delivery-carrier/tree/16.0/delivery_estimated_package_quantity_by_weight>`_ project on GitHub.
+This module is part of the `OCA/delivery-carrier <https://github.com/OCA/delivery-carrier/tree/18.0/delivery_estimated_package_quantity_by_weight>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

delivery_estimated_package_quantity_by_weight/models/delivery_carrier.py

@@ -5,7 +5,6 @@
 
 
 class DeliveryCarrier(models.Model):
-
     _inherit = "delivery.carrier"
 
     maximum_weight_per_package = fields.Float(string="Maximum weight per package")

delivery_estimated_package_quantity_by_weight/models/stock_picking.py

@@ -5,7 +5,6 @@
 
 
 class StockPicking(models.Model):
-
     _inherit = "stock.picking"
 
     theoretical_number_of_packages = fields.Integer(
@@ -29,7 +28,6 @@ class StockPicking(models.Model):
         "picking_type_code", "carrier_id", "carrier_id.maximum_weight_per_package"
     )
     def _compute_is_number_of_packages_visible(self):
-
         for rec in self:
             if (
                 rec.picking_type_code == "outgoing"
@@ -80,10 +78,9 @@ def _compute_is_number_of_packages_outranged(self):
     def _number_of_packages(
         self, products_weights, number_of_items, maximum_weight_per_package
     ):
-
         # Split the product_weights into as many items as we haves
         products_weights_list = []
-        for weight, number in zip(products_weights, number_of_items):
+        for weight, number in zip(products_weights, number_of_items, strict=False):
             for _i in range(int(number)):
                 products_weights_list.append(weight)
 

delivery_estimated_package_quantity_by_weight/pyproject.toml

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

delivery_estimated_package_quantity_by_weight/readme/CONTRIBUTORS.md

@@ -0,0 +1,2 @@
+- Lindsay Marion \<<lindsay.marion@acsone.eu>\>
+- Hughes Damry \<<hughes.damry@acsone.eu>\>

delivery_estimated_package_quantity_by_weight/readme/CONTRIBUTORS.rst

@@ -0,0 +1,15 @@
+This module computes the amount of packages a picking out should have
+depending on the weight of the products and the limit fixed by the
+carrier. It's fully independent of the delivery_package_number module.
+
+A warning is given if the number of packages for the picking out is
+above what is considered as the theoretical number of packages for this
+picking and the chosen carrier. The goal is to minimize the number of
+packages billed by the carrier.
+
+The chosen strategy for the theoretical number of packages is as follow:  
+- Split the product_weights into as many items as we have
+- Try to fit the heaviest product with the lightest. If it does not work
+  then the heaviest should have a box for itself
+- While the weight of products does not exceed the limit, continue
+  adding products in the same package

delivery_estimated_package_quantity_by_weight/readme/DESCRIPTION.md

@@ -0,0 +1,4 @@
+Here are some limitations of the module:  
+- Product packagings are not supported
+- Multiple package types having different max weight for a shipping
+  method are not supported

delivery_estimated_package_quantity_by_weight/readme/DESCRIPTION.rst

@@ -1,4 +1,3 @@
-<?xml version="1.0" encoding="utf-8"?>
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
 <head>
@@ -9,10 +8,11 @@
 
 /*
 :Author: David Goodger (goodger@python.org)
-:Id: $Id: html4css1.css 8954 2022-01-20 10:10:25Z milde $
+:Id: $Id: html4css1.css 9511 2024-01-13 09:50:07Z milde $
 :Copyright: This stylesheet has been placed in the public domain.
 
 Default cascading style sheet for the HTML output of Docutils.
+Despite the name, some widely supported CSS2 features are used.
 
 See https://docutils.sourceforge.io/docs/howto/html-stylesheets.html for how to
 customize this style sheet.
@@ -275,7 +275,7 @@
   margin-left: 2em ;
   margin-right: 2em }
 
-pre.code .ln { color: grey; } /* line numbers */
+pre.code .ln { color: gray; } /* line numbers */
 pre.code, code { background-color: #eeeeee }
 pre.code .comment, code .comment { color: #5C6576 }
 pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold }
@@ -301,7 +301,7 @@
 span.pre {
   white-space: pre }
 
-span.problematic {
+span.problematic, pre.problematic {
   color: red }
 
 span.section-subtitle {
@@ -369,25 +369,22 @@ <h1 class="title">Delivery Estimated Package Quantity By Weight</h1>
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! source digest: sha256:a35ba2d1b127f3f76257bb0405764911e2b224b554865a756261c6f221cbe3de
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/delivery-carrier/tree/16.0/delivery_estimated_package_quantity_by_weight"><img alt="OCA/delivery-carrier" src="https://img.shields.io/badge/github-OCA%2Fdelivery--carrier-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-delivery_estimated_package_quantity_by_weight"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/delivery-carrier&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/delivery-carrier/tree/18.0/delivery_estimated_package_quantity_by_weight"><img alt="OCA/delivery-carrier" src="https://img.shields.io/badge/github-OCA%2Fdelivery--carrier-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/delivery-carrier-18-0/delivery-carrier-18-0-delivery_estimated_package_quantity_by_weight"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/delivery-carrier&amp;target_branch=18.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>This module computes the amount of packages a picking out should have
-depending on the weight of the products and the limit fixed by the carrier.
-It’s fully independent of the delivery_package_number module.</p>
-<p>A warning is given if the number of packages for the picking out is above what
-is considered as the theoretical number of packages for this picking and the
-chosen carrier. The goal is to minimize the number of packages billed by the
-carrier.</p>
-<dl class="docutils">
-<dt>The chosen strategy for the theoretical number of packages is as follow:</dt>
-<dd><ul class="first last simple">
+depending on the weight of the products and the limit fixed by the
+carrier. It’s fully independent of the delivery_package_number module.</p>
+<p>A warning is given if the number of packages for the picking out is
+above what is considered as the theoretical number of packages for this
+picking and the chosen carrier. The goal is to minimize the number of
+packages billed by the carrier.</p>
+<p>The chosen strategy for the theoretical number of packages is as follow:</p>
+<ul class="simple">
 <li>Split the product_weights into as many items as we have</li>
-<li>Try to fit the heaviest product with the lightest.
-If it does not work then the heaviest should have a box for itself</li>
-<li>While the weight of products does not exceed the limit, continue adding
-products in the same package</li>
+<li>Try to fit the heaviest product with the lightest. If it does not work
+then the heaviest should have a box for itself</li>
+<li>While the weight of products does not exceed the limit, continue
+adding products in the same package</li>
 </ul>
-</dd>
-</dl>
 <p><strong>Table of contents</strong></p>
 <div class="contents local topic" id="contents">
 <ul class="simple">
@@ -403,22 +400,19 @@ <h1 class="title">Delivery Estimated Package Quantity By Weight</h1>
 </div>
 <div class="section" id="usage">
 <h1><a class="toc-backref" href="#toc-entry-1">Usage</a></h1>
-<dl class="docutils">
-<dt>Here are some limitations of the module:</dt>
-<dd><ul class="first last simple">
+<p>Here are some limitations of the module:</p>
+<ul class="simple">
 <li>Product packagings are not supported</li>
-<li>Multiple package types having different max weight for a shipping method
-are not supported</li>
+<li>Multiple package types having different max weight for a shipping
+method are not supported</li>
 </ul>
-</dd>
-</dl>
 </div>
 <div class="section" id="bug-tracker">
 <h1><a class="toc-backref" href="#toc-entry-2">Bug Tracker</a></h1>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/delivery-carrier/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-<a class="reference external" href="https://github.com/OCA/delivery-carrier/issues/new?body=module:%20delivery_estimated_package_quantity_by_weight%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<a class="reference external" href="https://github.com/OCA/delivery-carrier/issues/new?body=module:%20delivery_estimated_package_quantity_by_weight%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
@@ -439,11 +433,13 @@ <h2><a class="toc-backref" href="#toc-entry-5">Contributors</a></h2>
 <div class="section" id="maintainers">
 <h2><a class="toc-backref" href="#toc-entry-6">Maintainers</a></h2>
 <p>This module is maintained by the OCA.</p>
-<a class="reference external image-reference" href="https://odoo-community.org"><img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" /></a>
+<a class="reference external image-reference" href="https://odoo-community.org">
+<img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" />
+</a>
 <p>OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/delivery-carrier/tree/16.0/delivery_estimated_package_quantity_by_weight">OCA/delivery-carrier</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/delivery-carrier/tree/18.0/delivery_estimated_package_quantity_by_weight">OCA/delivery-carrier</a> project on GitHub.</p>
 <p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
 </div>
 </div>

delivery_estimated_package_quantity_by_weight/readme/USAGE.md

@@ -220,7 +220,8 @@ def test_all_products(self):
         Data:
             All the products are in the SO, some are heavy, others light
         Test case:
-            Check the number of packages in the shipping. Each box should not exceed 37 kg
+            Check the number of packages in the shipping.
+            Each box should not exceed 37 kg
             We have a lot of products in this shipping:
             10 product1 with weight of 25kg each => 250kg
             10 product2 with weight of 30kg each => 300kg

delivery_estimated_package_quantity_by_weight/readme/USAGE.rst

@@ -2,7 +2,6 @@
 <!-- Copyright 2021 ACSONE SA/NV
      License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). -->
 <odoo>
-
     <record model="ir.ui.view" id="delivery_carrier_form_view">
         <field
             name="name"
@@ -11,11 +10,11 @@
         <field name="inherit_id" ref="delivery.view_delivery_carrier_form" />
         <field name="arch" type="xml">
             <field name="integration_level" position="before">
-                 <label for="maximum_weight_per_package" />
-                 <div class="o_row" name="maximum_weight_per_package">
-                     <field name="maximum_weight_per_package" class="oe_inline" />
-                     <field name="weight_uom_name" />
-                 </div>
+                <label for="maximum_weight_per_package" />
+                <div class="o_row" name="maximum_weight_per_package">
+                    <field name="maximum_weight_per_package" class="oe_inline" />
+                    <field name="weight_uom_name" />
+                </div>
             </field>
         </field>
     </record>
@@ -31,5 +30,4 @@
             </field>
         </field>
     </record>
-
 </odoo>