[MIG] stock_return_request: Migration to version 18.0

Author: carlos-lopez-tecnativa

stock_return_request/README.rst

@@ -90,6 +90,7 @@ Contributors
   - Pedro M. Baeza
   - David Vidal
   - César A. Sánchez
+  - Carlos Lopez
 
 - `Pro Thai <http://prothaitechnology.com>`__:
 

stock_return_request/__manifest__.py

@@ -2,7 +2,7 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 {
     "name": "Stock Return Request",
-    "version": "15.0.1.0.2",
+    "version": "18.0.1.0.0",
     "category": "Stock",
     "website": "https://github.com/OCA/stock-logistics-request",
     "author": "Tecnativa, " "Odoo Community Association (OCA)",

stock_return_request/data/stock_return_request_data.xml

@@ -7,5 +7,6 @@
         <field name="padding">5</field>
         <field name="number_next">1</field>
         <field name="number_increment">1</field>
+        <field name="company_id" eval="False" />
     </record>
 </odoo>

stock_return_request/models/stock_move.py

@@ -10,21 +10,18 @@ class StockMove(models.Model):
         digits="Product Unit of Measure",
         string="Returnable Quantity",
         compute="_compute_qty_returnable",
-        readonly=True,
     )
 
     def _compute_qty_returnable(self):
         """Looks for chained returned moves to compute how much quantity
         from the original can be returned"""
+        self.qty_returnable = 0
         for move in self.filtered(lambda x: x.state not in ["draft", "cancel"]):
             if not move.returned_move_ids:
                 if move.state == "done":
-                    move.qty_returnable = move.quantity_done
-                else:
-                    # TODO: Review if this option has sense
-                    move.qty_returnable = move.reserved_availability
+                    move.qty_returnable = move.quantity
                 continue
-            move.qty_returnable = move.quantity_done - sum(
+            move.qty_returnable = move.quantity - sum(
                 move.returned_move_ids.mapped("qty_returnable")
             )
 
@@ -33,15 +30,12 @@ def _get_lot_returnable_qty(self, lot_id, qty=0):
         from the original can be returned for a given lot"""
         for move in self.filtered(lambda x: x.state not in ["draft", "cancel"]):
             mls = move.move_line_ids.filtered(lambda x: x.lot_id == lot_id)
-            if move.state == "done":
-                qty += sum(mls.mapped("qty_done"))
-            else:
-                qty += sum(mls.mapped("product_uom_qty"))
+            qty += sum(mls.mapped("quantity"))
             qty -= move.returned_move_ids._get_lot_returnable_qty(lot_id)
         return qty
 
     def _action_assign(self):
         if self.env.context.get("skip_assign_move", False):
             # Avoid assign stock moves allowing create stock move lines manually
             return
-        return super(StockMove, self)._action_assign()
+        return super()._action_assign()