odoo-addon-stock-barcodes 15.0.3.1.5.2__py3-none-any.whl → 15.0.3.2.0__py3-none-any.whl

odoo/addons/stock_barcodes/models/stock_picking_type.py

@@ -26,17 +26,15 @@ class StockPickingType(models.Model):
             "picking_mode": "picking",
         }
         if self.code == "outgoing":
-            location_dest_id = (
+            vals["location_dest_id"] = (
                 self.default_location_dest_id.id
                 or self.env.ref("stock.stock_location_customers").id
             )
-            vals["location_dest_id"] = location_dest_id
-        if self.code == "incoming":
-            location_src_id = (
+        elif self.code == "incoming":
+            vals["location_id"] = (
                 self.default_location_src_id.id
                 or self.env.ref("stock.stock_location_suppliers").id
             )
-            vals["location_id"] = location_src_id
         if self.barcode_option_group_id.get_option_value(
             "location_id", "filled_default"
         ):
@@ -46,8 +44,8 @@ class StockPickingType(models.Model):
         ):
             vals["location_dest_id"] = self.default_location_dest_id.id
         wiz = self.env["wiz.stock.barcodes.read.picking"].create(vals)
-        wiz.determine_todo_action()
         wiz.fill_pending_moves()
+        wiz.determine_todo_action()
         action = self.env["ir.actions.actions"]._for_xml_id(
             "stock_barcodes.action_stock_barcodes_read_picking"
         )
@@ -55,6 +53,7 @@ class StockPickingType(models.Model):
         return action
 
     def action_barcode_new_picking(self):
+        self.ensure_one()
         picking = (
             self.env["stock.picking"]
             .with_context(default_immediate_transfer=True)

odoo/addons/stock_barcodes/static/description/index.html

@@ -8,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.
@@ -274,7 +275,7 @@ pre.literal-block, pre.doctest-block, pre.math, pre.code {
   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.option {
 span.pre {
   white-space: pre }
 
-span.problematic {
+span.problematic, pre.problematic {
   color: red }
 
 span.section-subtitle {
@@ -366,7 +367,7 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:2cb108800d4184b951795fa6c906ec0d63a06035061daad42aca10893c49a5be
+!! source digest: sha256:aaf3270f55f43b616a2ceb165378f0909ba3cc1a3d0d36d21a65e4c04c06998f
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
 <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/stock-logistics-barcode/tree/15.0/stock_barcodes"><img alt="OCA/stock-logistics-barcode" src="https://img.shields.io/badge/github-OCA%2Fstock--logistics--barcode-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/stock-logistics-barcode-15-0/stock-logistics-barcode-15-0-stock_barcodes"><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/stock-logistics-barcode&amp;target_branch=15.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>This module provides a barcode reader interface for stock module.</p>
@@ -556,7 +557,9 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
 <div class="section" id="maintainers">
 <h2><a class="toc-backref" href="#toc-entry-16">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>

odoo/addons/stock_barcodes/tests/test_stock_barcodes_picking.py

@@ -294,6 +294,7 @@ class TestStockBarcodesPicking(TestStockBarcodes):
         self.product_tracking.categ_id.removal_strategy_id = self.env.ref(
             "stock.removal_lifo"
         )
+        self.wiz_scan_picking_out.action_clean_values()
         self.action_barcode_scanned(self.wiz_scan_picking_out, "8433281006850")
         self.assertEqual(self.wiz_scan_picking_out.lot_id, lot_3)
 

odoo/addons/stock_barcodes/wizard/stock_barcodes_read.py

@@ -76,7 +76,6 @@ class WizStockBarcodesRead(models.AbstractModel):
     show_scan_log = fields.Boolean(compute="_compute_is_manual_qty")
     # Technical field to allow use in attrs
     display_menu = fields.Boolean()
-    qty_available = fields.Float(compute="_compute_qty_available")
     auto_lot = fields.Boolean(
         string="Get lots automatically",
         help="If checked the lot will be set automatically with the same "
@@ -122,24 +121,6 @@ class WizStockBarcodesRead(models.AbstractModel):
         for rec in self:
             rec.create_lot = rec.option_group_id.create_lot
 
-    @api.depends("location_id", "product_id", "lot_id")
-    def _compute_qty_available(self):
-        if not self.product_id or self.location_id.usage != "internal":
-            self.qty_available = 0.0
-            return
-        domain_quant = [
-            ("product_id", "=", self.product_id.id),
-            ("location_id", "=", self.location_id.id),
-        ]
-        if self.lot_id:
-            domain_quant.append(("lot_id", "=", self.lot_id.id))
-        # if self.package_id:
-        #     domain_quant.append(('package_id', '=', self.package_id.id))
-        groups = self.env["stock.quant"].read_group(
-            domain_quant, ["quantity"], [], orderby="id"
-        )
-        self.qty_available = groups[0]["quantity"]
-
     @api.depends("product_id")
     def _compute_display_assign_serial(self):
         for rec in self:
@@ -258,6 +239,8 @@ class WizStockBarcodesRead(models.AbstractModel):
                             "more_match",
                             _("No stock available for this lot with screen values"),
                         )
+                        self.lot_id = False
+                        self.lot_name = False
                         return False
                     if quants:
                         self.set_info_from_quants(quants)
@@ -371,12 +354,14 @@ class WizStockBarcodesRead(models.AbstractModel):
     def process_barcode_packaging_id(self):
         domain = self._barcode_domain(self.barcode)
         if self.env.user.has_group("product.group_stock_packaging"):
+            domain.append(("product_id", "!=", False))
             packaging = self.env["product.packaging"].search(domain)
             if packaging:
                 if len(packaging) > 1:
                     self._set_messagge_info(
                         "more_match", _("More than one package found")
                     )
+                    self.packaging_id = False
                     return False
                 self.action_packaging_scaned_post(packaging)
                 return True
@@ -399,14 +384,15 @@ class WizStockBarcodesRead(models.AbstractModel):
             option_func = getattr(self, "process_barcode_%s" % option.field_name, False)
             if option_func:
                 res = option_func()
-                if option.required:
-                    self.play_sounds(res)
                 if res:
                     barcode_found = True
+                    self.play_sounds(barcode_found)
                     break
                 elif self.message_type != "success":
+                    self.play_sounds(False)
                     return False
         if not barcode_found:
+            self.play_sounds(barcode_found)
             if self.option_group_id.ignore_filled_fields:
                 self._set_messagge_info(
                     "info", _("Barcode not found or field already filled")
@@ -625,7 +611,9 @@ class WizStockBarcodesRead(models.AbstractModel):
 
     def action_clean_values(self):
         options = self.option_group_id.option_ids
-        options_to_clean = options.filtered("clean_after_done")
+        options_to_clean = options.filtered(
+            lambda op: op.clean_after_done and op.field_name in self
+        )
         for option in options_to_clean:
             if option.field_name == "result_package_id" and self.keep_result_package:
                 continue
@@ -749,6 +737,7 @@ class WizStockBarcodesRead(models.AbstractModel):
 
     def action_confirm(self):
         if not self.check_option_required():
+            self.play_sounds(False)
             return False
         record = self.browse(self.ids)
         record.write(self._convert_to_write(self._cache))
@@ -839,7 +828,9 @@ class WizStockBarcodesRead(models.AbstractModel):
          options.
          sticky: Permanent notification until user removes it
         """
-        if self.option_group_id.display_notification:
+        if self.option_group_id.display_notification and not self.env.context.get(
+            "skip_display_notification", False
+        ):
             message = {"message": message, "type": message_type, "sticky": sticky}
             if title:
                 message["title"] = title

odoo/addons/stock_barcodes/wizard/stock_barcodes_read_picking.py

@@ -1,11 +1,13 @@
 # Copyright 2019 Sergio Teruel <sergio.teruel@tecnativa.com>
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 import logging
+from collections import OrderedDict, defaultdict
 
 from odoo import _, api, fields, models
 from odoo.exceptions import ValidationError
 from odoo.fields import first
-from odoo.tools.float_utils import float_compare
+from odoo.tools.float_utils import float_compare, float_round
+from odoo.tools.safe_eval import safe_eval
 
 _logger = logging.getLogger(__name__)
 
@@ -74,6 +76,9 @@ class WizStockBarcodesReadPicking(models.TransientModel):
     picking_location_id = fields.Many2one(related="picking_id.location_id")
     picking_location_dest_id = fields.Many2one(related="picking_id.location_dest_id")
     company_id = fields.Many2one(related="picking_id.company_id")
+    todo_line_is_extra_line = fields.Boolean(related="todo_line_id.is_extra_line")
+    forced_todo_key = fields.Char()
+    qty_available = fields.Float(compute="_compute_qty_available")
 
     @api.depends("todo_line_id")
     def _compute_todo_line_display_ids(self):
@@ -85,6 +90,9 @@ class WizStockBarcodesReadPicking(models.TransientModel):
         if self.option_group_id.show_pending_moves:
             self.pending_move_ids = self.todo_line_ids.filtered(
                 lambda t: t.state == "pending"
+                and any(
+                    sm.barcode_backorder_action == "pending" for sm in t.stock_move_ids
+                )
             )
         else:
             self.pending_move_ids = False
@@ -108,6 +116,40 @@ class WizStockBarcodesReadPicking(models.TransientModel):
                 rec.total_product_uom_qty += line.product_uom_qty
                 rec.total_product_qty_done += line.quantity_done
 
+    @api.depends("location_id", "product_id", "lot_id")
+    def _compute_qty_available(self):
+        if not self.product_id or self.location_id.usage != "internal":
+            self.qty_available = 0.0
+            return
+        domain_quant = [
+            ("product_id", "=", self.product_id.id),
+            ("location_id", "=", self.location_id.id),
+        ]
+        if self.lot_id:
+            domain_quant.append(("lot_id", "=", self.lot_id.id))
+        # if self.package_id:
+        #     domain_quant.append(('package_id', '=', self.package_id.id))
+        groups = self.env["stock.quant"].read_group(
+            domain_quant, ["quantity"], [], orderby="id"
+        )
+        self.qty_available = groups[0]["quantity"]
+        # Unexpected done quantities must reduce qty_available
+        if self.lot_id:
+            done_move_lines = self.move_line_ids.filtered(
+                lambda m: m.product_id == self.product_id and m.lot_id == self.lot_id
+            )
+        else:
+            done_move_lines = self.move_line_ids.filtered(
+                lambda m: m.product_id == self.product_id
+            )
+        for sml in done_move_lines:
+            over_done_qty = float_round(
+                sml.qty_done - sml.product_uom_qty,
+                precision_rounding=sml.product_uom_id.rounding,
+            )
+            if over_done_qty > 0.0:
+                self.qty_available -= over_done_qty
+
     def name_get(self):
         return [
             (
@@ -141,8 +183,8 @@ class WizStockBarcodesReadPicking(models.TransientModel):
         # view, so for create a candidate picking with the same default picking
         # we need create it in this onchange
         self._set_default_picking()
-        self.determine_todo_action()
         self.fill_pending_moves()
+        self.determine_todo_action()
 
     def get_sorted_move_lines(self, move_lines):
         location_field = self.option_group_id.location_field_to_sort
@@ -180,12 +222,8 @@ class WizStockBarcodesReadPicking(models.TransientModel):
         return move_lines
 
     def fill_pending_moves(self):
-        if (
-            self.option_group_id.barcode_guided_mode != "guided"
-            and self.option_group_id.show_pending_moves
-            and not self.todo_line_ids
-        ):
-            self.fill_todo_records()
+        # TODO: Unify method
+        self.fill_todo_records()
 
     def get_moves_or_move_lines(self):
         if self.option_group_id.source_pending_moves == "move_line_ids":
@@ -193,9 +231,12 @@ class WizStockBarcodesReadPicking(models.TransientModel):
         else:
             return self.picking_id.move_lines
 
+    def get_moves(self):
+        return self.picking_id.move_lines
+
     def fill_todo_records(self):
         move_lines = self.get_sorted_move_lines(self.get_moves_or_move_lines())
-        self.env["wiz.stock.barcodes.read.todo"].fill_records(self, [move_lines])
+        self.fill_records([move_lines])
 
     @api.model
     def _get_fields_filled_special(self):
@@ -211,12 +252,6 @@ class WizStockBarcodesReadPicking(models.TransientModel):
         self.visible_force_done = self.env.context.get("visible_force_done", False)
         if not self.option_group_id.barcode_guided_mode == "guided":
             return False
-        if not self.todo_line_ids:
-            self.fill_todo_records()
-        # When scanning all information in one step (e.g. using GS-1), the
-        # status and qty processed might have not been update, we ensure it
-        # invalidating the cache.
-        self.todo_line_ids.invalidate_cache()
         self.todo_line_id = (
             forced_todo_line
             or self.todo_line_ids.filtered(lambda t: t._origin.state == "pending")[:1]
@@ -278,9 +313,24 @@ class WizStockBarcodesReadPicking(models.TransientModel):
                 if not self.keep_screen_values or self.todo_line_id.state != "pending":
                     if not self.env.context.get("skip_clean_values", False):
                         self.action_clean_values()
-                    self.determine_todo_action()
+                    keep_vals = {}
                 else:
-                    self.action_show_step()
+                    keep_vals = self._convert_to_write(self._cache)
+                self.fill_todo_records()
+                if self.forced_todo_key:
+                    self.todo_line_id = self.pending_move_ids.filtered(
+                        lambda ln: str(self._group_key(ln)) == self.forced_todo_key
+                    )[:1]
+                    self.selected_pending_move_id = self.todo_line_id
+                    self.determine_todo_action(self.todo_line_id)
+                else:
+                    self.determine_todo_action()
+                self.action_show_step()
+                if keep_vals:
+                    self.update_keep_values(keep_vals)
+            # Force refresh candidate pickings to show green if not pending moves
+            if not self.pending_move_ids:
+                self._set_candidate_pickings(self.picking_id)
             # Now we can add read log with details.
             _logger.info("Add scanned log barcode:{}".format(self.barcode))
             self._add_read_log(log_detail=move_dic)
@@ -290,6 +340,13 @@ class WizStockBarcodesReadPicking(models.TransientModel):
         self._add_read_log()
         return res
 
+    def update_keep_values(self, keep_vals):
+        options = self.option_group_id.option_ids
+        fields_to_keep = options.filtered(
+            lambda op: self._fields[op.field_name].type != "float"
+        ).mapped("field_name")
+        self.update({f_name: keep_vals[f_name] for f_name in fields_to_keep})
+
     def action_manual_entry(self):
         result = super().action_manual_entry()
         if result:
@@ -456,6 +513,10 @@ class WizStockBarcodesReadPicking(models.TransientModel):
         domain = self._prepare_stock_moves_domain()
         if self.option_group_id.barcode_guided_mode == "guided":
             moves_todo = self.todo_line_id.stock_move_ids
+        elif self.picking_id:
+            moves_todo = self.picking_id.move_lines.filtered(
+                lambda sm: sm.product_id == self.product_id
+            )
         else:
             moves_todo = StockMove.search(domain)
         if not getattr(
@@ -615,13 +676,14 @@ class WizStockBarcodesReadPicking(models.TransientModel):
             # link this new lines to the todo line details
             # If user scan a product distinct of the todo line we need link to other
             # alternative move
-            if move_to_link_in_todo_line and self.todo_line_id:
-                todo_line = self.todo_line_id
-            else:
-                todo_line = self.todo_line_ids.filtered(
-                    lambda ln: ln.product_id == self.product_id
-                )
-            todo_line.line_ids = [(4, sml.id) for sml in stock_move_lines]
+            if self.option_group_id.source_pending_moves != "move_line_ids":
+                if move_to_link_in_todo_line and self.todo_line_id:
+                    todo_line = self.todo_line_id
+                else:
+                    todo_line = self.todo_line_ids.filtered(
+                        lambda ln: ln.product_id == self.product_id
+                    )
+                todo_line.line_ids = [(4, sml.id) for sml in stock_move_lines]
         self.update_fields_after_process_stock(moves_todo)
         return move_lines_dic
 
@@ -784,6 +846,150 @@ class WizStockBarcodesReadPicking(models.TransientModel):
             return bool(self.location_dest_id)
         return super()._option_required_hook(option_required)
 
+    def _group_key(self, line):
+        group_key_for_todo_records = self.option_group_id.group_key_for_todo_records
+        if group_key_for_todo_records:
+            return safe_eval(group_key_for_todo_records, globals_dict={"object": line})
+        if self.option_group_id.source_pending_moves == "move_line_ids":
+            return (
+                line.location_id.id,
+                line.product_id.id,
+                line.lot_id.id,
+                line.package_id.id,
+            )
+        else:
+            return (line.location_id.id, line.product_id.id)
+
+    def _get_all_products_quantities_in_package(self, package):
+        res = {}
+        for quant in package._get_contained_quants():
+            if quant.product_id not in res:
+                res[quant.product_id] = 0
+            res[quant.product_id] += quant.quantity
+        return res
+
+    def _prepare_fill_record_values(self, line, position):
+        vals = {
+            "wiz_barcode_id": self.id,
+            "product_id": line.product_id.id,
+            "product_uom_qty": line.product_uom_qty,
+            "name": "To do action",
+            "position_index": position,
+            "picking_code": line.picking_code,
+        }
+        if line._name == "stock.move.line":
+            package_product_dic = self._get_all_products_quantities_in_package(
+                line.package_id
+            )
+            vals.update(
+                {
+                    "location_id": line.location_id.id,
+                    "location_dest_id": line.location_dest_id.id,
+                    "lot_id": line.lot_id.id,
+                    "package_id": line.package_id.id,
+                    "result_package_id": line.result_package_id.id,
+                    "uom_id": line.product_uom_id.id,
+                    "product_qty_reserved": line.product_qty,
+                    "line_ids": [(6, 0, line.ids)],
+                    "stock_move_ids": [(6, 0, line.move_id.ids)],
+                    "package_product_qty": package_product_dic
+                    and package_product_dic[line.product_id]
+                    or 0.0,
+                    "is_stock_move_line_origin": True,
+                }
+            )
+        else:
+            vals.update(
+                {
+                    "location_id": (line.move_line_ids[:1] or line).location_id.id,
+                    "location_dest_id": (
+                        line.move_line_ids[:1] or line
+                    ).location_dest_id.id,
+                    "uom_id": line.product_uom.id,
+                    "product_qty_reserved": line.move_line_ids
+                    and sum(line.move_line_ids.mapped("product_qty"))
+                    or line.product_uom_qty,
+                    "line_ids": [(6, 0, line.move_line_ids.ids)],
+                    "stock_move_ids": [(6, 0, line.ids)],
+                    "is_stock_move_line_origin": False,
+                }
+            )
+        return vals
+
+    def _update_fill_record_values(self, line, vals):
+        vals["product_uom_qty"] += line.product_uom_qty
+        if vals["is_stock_move_line_origin"]:
+            vals["product_qty_reserved"] += line.product_qty
+            vals["line_ids"][0][2].append(line.id)
+            vals["stock_move_ids"][0][2].append(line.move_id.id)
+        else:
+            vals["product_qty_reserved"] += (
+                line.move_line_ids
+                and sum(line.move_line_ids.mapped("product_qty"))
+                or line.product_uom_qty
+            )
+            vals["line_ids"][0][2].extend(line.move_line_ids.ids)
+            vals["stock_move_ids"][0][2].extend(line.ids)
+        return vals
+
+    @api.model
+    def fill_records(self, lines_list):
+        """
+        :param lines_list: browse list
+        :return:
+        """
+        self.forced_todo_key = str(
+            self._group_key(self.todo_line_id or self.selected_pending_move_id)
+        )
+        self.todo_line_ids.unlink()
+        self.todo_line_id = False
+        # self.position_index = 0
+        todo_vals = OrderedDict()
+        position = 0
+        move_qty_dic = defaultdict(float)
+        is_stock_move_line_origin = lines_list[0]._name == "stock.move.line"
+        for lines in lines_list:
+            for line in lines:
+                key = self._group_key(line)
+                if key not in todo_vals:
+                    todo_vals[key] = self._prepare_fill_record_values(line, position)
+                    position += 1
+                else:
+                    todo_vals[key] = self._update_fill_record_values(
+                        line, todo_vals[key]
+                    )
+                if is_stock_move_line_origin:
+                    move_qty_dic[line.move_id] += max(
+                        line.product_uom_qty, line.qty_done
+                    )
+                else:
+                    move_qty_dic[line] += max(line.product_uom_qty, line.quantity_done)
+        for move in self.get_moves():
+            qty = move_qty_dic[move]
+            if (
+                move.barcode_backorder_action == "pending"
+                and move.product_uom_qty > qty
+            ):
+                vals = self._prepare_fill_record_values(move, position)
+                vals.update(
+                    {
+                        "product_uom_qty": move.product_uom_qty - qty,
+                        "product_qty_reserved": 0.0,
+                        "line_ids": False,
+                        "is_extra_line": True,
+                    }
+                )
+                todo_vals[
+                    (
+                        move,
+                        "M",
+                    )
+                ] = vals
+                position += 1
+        self.todo_line_ids = self.env["wiz.stock.barcodes.read.todo"].create(
+            list(todo_vals.values())
+        )
+
 
 class WizCandidatePicking(models.TransientModel):
     """

odoo/addons/stock_barcodes/wizard/stock_barcodes_read_picking_views.xml

@@ -127,6 +127,8 @@
                 <field name="picking_location_id" invisible="1" />
                 <field name="picking_location_dest_id" invisible="1" />
                 <field name="company_id" invisible="1" />
+                <field name="todo_line_is_extra_line" invisible="1" />
+                <field name="qty_available" invisible="1" />
             </field>
             <field name="location_id" position="attributes">
                 <attribute
@@ -161,6 +163,12 @@
                     </div>
                 </group>
             </group>
+            <group name="scan_fields" position="attributes">
+                <!--  hide group scan_fields for extra todo lines -->
+                <attribute
+                    name="attrs"
+                >{'invisible': [('todo_line_is_extra_line', '!=', False)]}</attribute>
+            </group>
             <group name="scan_fields" position="after">
                 <group
                     string="Pending moves"
@@ -173,11 +181,14 @@
                         force_save="1"
                         mode="tree,kanban"
                     >
-                        <tree>
+                        <tree decoration-warning="is_extra_line">
                             <field name="state" invisible="1" />
+                            <field name="is_extra_line" invisible="1" />
                             <field name="product_id" />
                             <field name="product_uom_qty" />
                             <field name="line_ids" invisible="1" />
+                            <field name="location_id" optional="hide" />
+                            <field name="location_dest_id" optional="hide" />
                             <field
                                 name="lot_id"
                                 groups="stock.group_production_lot"