[IMP] stock_inventory_discrepancy: better inheritance

During the migration to 13.0 of this module it was decided to introduce a hook.

Refactoring to a regular override plays better with 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,40 @@ def action_apply_inventory(self):
             )
             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 (
+            self.env.context.get("from_apply_inventory")
+            and groups == "stock.group_stock_manager"
+            and 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.")
+            )
+        return ret
+
+    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()