[IMP] sale_margin_delivered: Store margin delivered percent as a frat…

…ion of 1

As it's done into the sale_margin addon from odoo

sale_margin_delivered/__manifest__.py

@@ -2,7 +2,7 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 {
     "name": "Sale Margin Delivered",
-    "version": "16.0.1.0.5",
+    "version": "16.0.2.0.0",
     "author": "Tecnativa, Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/margin-analysis",
     "category": "Sales",

sale_margin_delivered/migrations/16.0.2.0.0/post-migration.py

@@ -0,0 +1,14 @@
+import logging
+
+_logger = logging.getLogger(__name__)
+
+
+def migrate(env, version):
+    """Store margin delivered precentage as a fraction of 1"""
+    env.cr.execute(
+        """
+        UPDATE sale_order_line
+        SET margin_delivered_percent = margin_delivered_percent / 100
+        """
+    )
+    _logger.info("Updated %d records", env.cr.rowcount)

sale_margin_delivered/models/sale_margin.py

@@ -101,7 +101,5 @@ def _compute_margin_delivered(self):
             # quantities
             if price_reduce_taxexcl:
                 line.margin_delivered_percent = (
-                    (price_reduce_taxexcl - line.purchase_price_delivery)
-                    / price_reduce_taxexcl
-                    * 100.0
-                )
+                    price_reduce_taxexcl - line.purchase_price_delivery
+                ) / price_reduce_taxexcl

sale_margin_delivered/static/description/index.html

@@ -8,10 +8,11 @@
 
 /*
 :Author: David Goodger ([email protected])
-: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.
@@ -274,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 }
@@ -300,7 +301,7 @@
 span.pre {
   white-space: pre }
 
-span.problematic {
+span.problematic, pre.problematic {
   color: red }
 
 span.section-subtitle {
@@ -452,7 +453,9 @@ <h2><a class="toc-backref" href="#toc-entry-7">Contributors</a></h2>
 <div class="section" id="maintainers">
 <h2><a class="toc-backref" href="#toc-entry-8">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>

sale_margin_delivered/tests/test_sale_margin_delivered.py

@@ -91,7 +91,7 @@ def test_sale_margin_delivered(self):
         picking._action_done()
         order_line = sale_order.order_line[:1]
         self.assertEqual(order_line.margin_delivered, 30.0)
-        self.assertEqual(order_line.margin_delivered_percent, 50.0)
+        self.assertEqual(order_line.margin_delivered_percent, 0.5)
         self.assertEqual(order_line.purchase_price_delivery, order_line.purchase_price)
 
     def test_sale_margin_delivered_excess(self):
@@ -104,7 +104,7 @@ def test_sale_margin_delivered_excess(self):
         picking._action_done()
         order_line = sale_order.order_line[:1]
         self.assertEqual(order_line.margin_delivered, 120.0)
-        self.assertEqual(order_line.margin_delivered_percent, 50.0)
+        self.assertEqual(order_line.margin_delivered_percent, 0.5)
         self.assertEqual(order_line.purchase_price_delivery, order_line.purchase_price)
 
     def test_sale_margin_zero(self):
@@ -142,7 +142,7 @@ def test_sale_margin_delivered_return_to_refund(self):
         picking_return._action_done()
         order_line = sale_order.order_line[:1]
         self.assertEqual(order_line.margin_delivered, 30.0)
-        self.assertEqual(order_line.margin_delivered_percent, 50.0)
+        self.assertEqual(order_line.margin_delivered_percent, 0.5)
         self.assertEqual(order_line.purchase_price_delivery, order_line.purchase_price)
 
     def test_sale_margin_delivered_return_to_refund_excess(self):
@@ -155,7 +155,7 @@ def test_sale_margin_delivered_return_to_refund_excess(self):
         picking_return._action_done()
         order_line = sale_order.order_line[:1]
         self.assertEqual(order_line.margin_delivered, 90.0)
-        self.assertEqual(order_line.margin_delivered_percent, 50.0)
+        self.assertEqual(order_line.margin_delivered_percent, 0.5)
         self.assertEqual(order_line.purchase_price_delivery, order_line.purchase_price)
 
     def test_sale_margin_delivered_return_no_refund(self):
@@ -168,7 +168,7 @@ def test_sale_margin_delivered_return_no_refund(self):
         picking_return._action_done()
         order_line = sale_order.order_line[:1]
         self.assertEqual(order_line.margin_delivered, 60.0)
-        self.assertEqual(order_line.margin_delivered_percent, 50.0)
+        self.assertEqual(order_line.margin_delivered_percent, 0.5)
         self.assertEqual(order_line.purchase_price_delivery, order_line.purchase_price)
 
     def test_sale_margin_delivered_return_no_refund_excess(self):
@@ -181,7 +181,7 @@ def test_sale_margin_delivered_return_no_refund_excess(self):
         picking_return._action_done()
         order_line = sale_order.order_line[:1]
         self.assertEqual(order_line.margin_delivered, 120.0)
-        self.assertEqual(order_line.margin_delivered_percent, 50.0)
+        self.assertEqual(order_line.margin_delivered_percent, 0.5)
         self.assertEqual(order_line.purchase_price_delivery, order_line.purchase_price)
 
     def test_sale_margin_delivered_precision(self):
@@ -201,5 +201,20 @@ def test_sale_margin_delivered_precision(self):
         # it should not be rounded to 16.74
         # margin_delivered: round(6 * ((100.45 /6) - 10.30)) != round(6 * (16.74 - 10.30))
         self.assertEqual(order_line.margin_delivered, 38.65)
-        self.assertAlmostEqual(order_line.margin_delivered_percent, 38.47685415629666)
+        self.assertAlmostEqual(order_line.margin_delivered_percent, 0.38476854156296)
+        self.assertEqual(order_line.purchase_price_delivery, order_line.purchase_price)
+
+    def test_sale_margin_no_cost(self):
+        self.product.standard_price = 0.0
+        self.product.list_price = 20
+        sale_order = self._new_sale_order()
+        sale_order.action_confirm()
+        picking = sale_order.picking_ids
+        picking.action_assign()
+        picking.move_line_ids.qty_done = 6.0
+        picking._action_done()
+        order_line = sale_order.order_line[:1]
+        # price_subtotal is rounded
+        self.assertEqual(order_line.margin_delivered, 120)
+        self.assertAlmostEqual(order_line.margin_delivered_percent, 1)
         self.assertEqual(order_line.purchase_price_delivery, order_line.purchase_price)

sale_margin_delivered/views/sale_margin_delivered_view.xml

@@ -16,6 +16,8 @@
                     name="margin_delivered_percent"
                     string="Margin dlvd. (%)"
                     optional="hide"
+                    attrs="{'invisible': [('price_subtotal', '=', 0)]}"
+                    widget="percentage"
                 />
                 <field name="margin_delivered" string="Margin dlvd." optional="hide" />
             </xpath>