[17.0][FIX] account_invoice_report_grouped_by_picking: Method abs is called when remaining_qty is near of zero

account_invoice_report_grouped_by_picking/models/account_move.py

@@ -1,12 +1,11 @@
-# Copyright 2017 Tecnativa - Carlos Dauden
+# Copyright 2017-2024 Tecnativa - Carlos Dauden
 # Copyright 2018 Tecnativa - David Vidal
 # Copyright 2018-2019 Tecnativa - Pedro M. Baeza
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
-import datetime
-from collections import OrderedDict
+from datetime import datetime
 
 from odoo import api, models
-from odoo.tools import float_is_zero
+from odoo.tools import float_round
 
 
 class AccountMove(models.Model):
@@ -15,7 +14,7 @@
     @api.model
     def _sort_grouped_lines(self, lines_dic):
         DTF = "%Y-%m-%d %H:%M:%S"
-        min_date = datetime.datetime.min
+        min_date = datetime.min
         return sorted(
             lines_dic,
             key=lambda x: (
@@ -26,38 +25,44 @@
                         DTF
                     ),
                 )
-                or ("", "")
+                or ("", ""),
+                x.get("is_last_section_notes", False),
             ),
         )
 
     def _get_signed_quantity_done(self, invoice_line, move, sign):
         """Hook method. Usage example:
         account_invoice_report_grouped_by_picking_sale_mrp module
         """
-        qty = 0
         if move.location_id.usage == "customer":
-            qty = -move.quantity * sign
-        elif move.location_dest_id.usage == "customer":
-            qty = move.quantity * sign
-        return qty
+            return -move.quantity * sign
+        if move.location_dest_id.usage == "customer":
+            return move.quantity * sign
+        return 0
 
     def _process_section_note_lines_grouped(
         self, previous_section, previous_note, lines_dic, pick_order=None
     ):
-        key_section = (pick_order, previous_section) if pick_order else previous_section
-        if previous_section and key_section not in lines_dic:
-            lines_dic[key_section] = 0.0
-        key_note = (pick_order, previous_note) if pick_order else previous_note
-        if previous_note and key_note not in lines_dic:
-            lines_dic[key_note] = 0.0
+        """Processes section and note lines, grouping them by order."""
+        for line in [previous_section, previous_note]:
+            if line:
+                key = (pick_order, line) if pick_order else line
+                lines_dic.setdefault(key, 0.0)
+
+    def _get_grouped_by_picking_sorted_lines(self):
+        """Sorts the invoice lines to be grouped by picking."""
+        return self.invoice_line_ids.sorted(
+            lambda ln: (-ln.sequence, ln.date, ln.move_name, -ln.id), reverse=True
+        )
 
     def lines_grouped_by_picking(self):
         """This prepares a data structure for printing the invoice report
         grouped by pickings."""
         self.ensure_one()
-        picking_dict = OrderedDict()
-        lines_dict = OrderedDict()
+        picking_dict = {}
+        lines_dict = {}
         picking_obj = self.env["stock.picking"]
+        # Determine the sign based on the move type
         # Not change sign if the credit note has been created from reverse move option
         # and it has the same pickings related than the reversed invoice instead of sale
         # order invoicing process after picking reverse transfer
@@ -71,61 +76,80 @@
             else 1.0
         )
         # Let's get first a correspondance between pickings and sales order
-        so_dict = {x.sale_id: x for x in self.picking_ids if x.sale_id}
+        so_dict = {p.sale_id: p for p in self.picking_ids if p.sale_id}
         # Now group by picking by direct link or via same SO as picking's one
         previous_section = previous_note = False
-        for line in self.invoice_line_ids.sorted(
-            lambda ln: (-ln.sequence, ln.date, ln.move_name, -ln.id), reverse=True
-        ):
-            if line.display_type == "line_section":
-                previous_section = line
-                continue
-            if line.display_type == "line_note":
-                previous_note = line
+        last_section_notes = []
+        sorted_lines = self._get_grouped_by_picking_sorted_lines()
+        for line in sorted_lines:
+            # Process section or note lines
+            if line.display_type in ["line_section", "line_note"]:
+                if line.display_type == "line_section":
+                    previous_section = line
+                else:
+                    previous_note = line
+                last_section_notes.append(
+                    {
+                        "picking": picking_obj,
+                        "line": line,
+                        "qty": 0.0,
+                        "is_last_section_notes": True,
+                    }
+                )
                 continue
+            # Reset sections and notes when encountering a regular line
+            last_section_notes = []
             has_returned_qty = False
             remaining_qty = line.quantity
+            # Process moves related to the line
             for move in line.move_line_ids:
                 key = (move.picking_id, line)
                 self._process_section_note_lines_grouped(
                     previous_section, previous_note, picking_dict, move.picking_id
                 )
-                picking_dict.setdefault(key, 0)
-                if move.location_id.usage == "customer":
-                    has_returned_qty = True
                 qty = self._get_signed_quantity_done(line, move, sign)
-                picking_dict[key] += qty
+                picking_dict[key] = picking_dict.get(key, 0.0) + qty
                 remaining_qty -= qty
+                if move.location_id.usage == "customer":
+                    has_returned_qty = True
+            # Process sale order lines without moves
             if not line.move_line_ids and line.sale_line_ids:
                 for so_line in line.sale_line_ids:
-                    if so_dict.get(so_line.order_id):
-                        key = (so_dict[so_line.order_id], line)
+                    picking = so_dict.get(so_line.order_id)
+                    if picking:
+                        key = (picking, line)
                         self._process_section_note_lines_grouped(
-                            previous_section,
-                            previous_note,
-                            picking_dict,
-                            so_dict[so_line.order_id],
+                            previous_section, previous_note, picking_dict, picking
                         )
-                        picking_dict.setdefault(key, 0)
-                        qty = so_line.product_uom_qty
-                        picking_dict[key] += qty
+                        qty = min(so_line.product_uom_qty, remaining_qty)
+                        picking_dict[key] = picking_dict.get(key, 0.0) + qty
                         remaining_qty -= qty
+            # Process lines without moves or sale orders
             elif not line.move_line_ids and not line.sale_line_ids:
                 key = (picking_obj, line)
-                picking_dict.setdefault(key, 0)
+                self._process_section_note_lines_grouped(
+                    previous_section, previous_note, lines_dict
+                )
                 qty = line.quantity
-                picking_dict[key] += qty
+                picking_dict[key] = picking_dict.get(key, 0.0) + qty
                 remaining_qty -= qty
             # To avoid to print duplicate lines because the invoice is a refund
             # without returned goods to refund.
-            if self.move_type == "out_refund" and not has_returned_qty and picking_dict:
-                remaining_qty = 0.0
-                for key in picking_dict:
-                    picking_dict[key] = abs(picking_dict[key])
-            if not float_is_zero(
+            remaining_qty = float_round(
                 remaining_qty,
                 precision_rounding=line.product_id.uom_id.rounding or 0.01,
+            )
+            if (
+                self.move_type == "out_refund"
+                and not has_returned_qty
+                and remaining_qty
+                and line.product_id.type != "service"
+                and picking_dict
             ):
+                remaining_qty = 0.0
+                for key in picking_dict:
+                    picking_dict[key] = abs(picking_dict[key])
+            if remaining_qty:
                 self._process_section_note_lines_grouped(
                     previous_section, previous_note, lines_dict
                 )
@@ -138,4 +162,4 @@
             {"picking": key[0], "line": key[1], "quantity": value}
             for key, value in picking_dict.items()
         ]
-        return no_picking + self._sort_grouped_lines(with_picking)
+        return no_picking + self._sort_grouped_lines(with_picking + last_section_notes)