### Steps to reproduce:

- Enable packages in the inventory settings
- Create a storable product tracked by Lot and put 4 units in stock: - 2 Units for "LOT001" - 2 Units for "LOT002"
- Create and confirm a sale order for 2 units
- Go to the barcode app > Operations > Delivery orders > your Delivery Order
- Register "LOT002" manually
> One unit will be added to your delivery moves.
- Put in pack
> You can unroll the move to see that one move line was created for 1
unit with a package and 1 unit is still missing.
- Register a second unit of "LOT002" manually to put in an other pack
> The line is updated to 2 units instead of 1 so that the additional unit can not be packed in an other pack

### Expected behavior:

You should be able to pack units of the same lot in different packages.

### Cause of the issue:

To determine which line is going to be updated by our lot registration the `_findLine` method is called. This method look for each line in pageLines and when it finds a candidate move line it continues to find a better candidate. However, fully packed moves should not be considered as valid candidates.

opw-4068334

Forward-Port-Of: odoo/enterprise#68535

**[FIX] documents: skip spreadsheets while toggling preview in FileViewer**

Issue:

- preview any document that can be previewed.
- toggle the preview to make the FileViewer preview a spreadsheet.
- as the spreadsheet lacks a preview, a blank preview is seen when the FileViewer
  lands on the spreadsheet.

Reason:

- while preparing the `documentsRecords` there is actually a filter present to filter out only previewable records.
  please refer --- https://github.com/odoo/enterprise/blob/17.0/documents/static/src/views/hooks.js#L311
- but for some reason the function `[isRecordPreviewable]` is not passed as a parameter from neither the 
  `DocumentsKanbanController` nor the `DocumentsListController` and as a result, rendered the `.filter()` useless.

Fix:

- We migrate the `isRecordPreviewable()` method to the `DocumentsSpreadsheetControllerMixin` so that it overrides 
  the newly created `isRecordPreviewable()` method in the `DocumentsKanbanController`, which is then accessed by 
  the  `.filter()` mentioned above.

Task-4029257

**Current behavior:**
When adding an unplanned move line to a transfer in barcode, if the line is added to an existing move which is a subcontract transfer, it will not take the subcontract location as its source location value. Rather, it will use whatever value is defined on the picking.

**Expected behavior:**
Use the more precise location defined on the move rather than the picking.

**Steps to reproduce:**
1. Create a product with a subcontract BOM

2. Create a receipt for that product from the subcontractor

3. Open the transfer in Barcode and manually add another line for the same product -> validate

4. In the receipt form, look at the move lines (detailed operations) and group by location to see the discrepancy

**Cause of the issue:**
When creating a move line here, it does not initially get a `move_id` value. Later when the move is written to the line, we would not generally want to override whatever value is on the line already because it (normally) is more precise/was purposely changed later than what value is on the move.

**Fix:**
In the case of lines which are created and then linked to a subcontract move, take the `location_id` of the move rather than whatever value is on the MoveLine.

opw-4007423