[16.0][IMP] stock_inventory_discrepancy: better inheritance

stock_inventory_discrepancy/__init__.py

@@ -2,4 +2,3 @@
 
 from . import models
 from . import wizards
-from .hooks import post_load_hook

stock_inventory_discrepancy/__manifest__.py

@@ -21,7 +21,6 @@
         "wizards/confirm_discrepancy_wiz.xml",
     ],
     "license": "AGPL-3",
-    "post_load": "post_load_hook",
     "installable": True,
     "application": False,
 }

stock_inventory_discrepancy/models/stock_quant.py

@@ -90,3 +90,44 @@
             )
             return action
         return super().action_apply_inventory()
+
+    def user_has_groups(self, groups):
+        # - Allow specific group to validate inventory
+        # - Allow validate on pending status
+        ret = super().user_has_groups(groups)
+        if not (
+            self.env.context.get("from_apply_inventory")
+            and groups == "stock.group_stock_manager"
+        ):
+            return ret
+        if (
+            not ret
+            and not super().user_has_groups(
+                "stock_inventory_discrepancy.group_stock_inventory_validation"
+            )
+            and not super().user_has_groups(
+                "stock_inventory_discrepancy.group_stock_inventory_validation_always"
+            )
+        ):
+            raise UserError(
+                _("Only a stock manager can validate an inventory adjustment.")
+            )
+        else:
+            return True
+
+    def _apply_inventory(self):
+        if (
+            not self.user_has_groups("stock.group_stock_manager")
+            and not self.user_has_groups(
+                "stock_inventory_discrepancy.group_stock_inventory_validation"
+            )
+            and not self.user_has_groups(
+                "stock_inventory_discrepancy.group_stock_inventory_validation_always"
+            )
+        ):
+            raise UserError(
+                _("Only a stock manager can validate an inventory adjustment.")
+            )
+        # Allow to write last_inventory_date on stock.location
+        self = self.sudo().with_context(from_apply_inventory=True)
+        return super(StockQuant, self)._apply_inventory()