Skip to content

Commit

Permalink
[FIX] account_asset_management: Allow to unlink w/o billing permission
Browse files Browse the repository at this point in the history
Some users may be allowed to unlink invoices with billing access,
but with current code, they are not able as the asset line check is
done without sudo, and other users except billing or higher ones
are allowed to acces to that model.

TT49673
  • Loading branch information
pedrobaeza committed Jun 18, 2024
1 parent 09ff13b commit 8b8f2af
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions account_asset_management/models/account_move.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,12 @@ def _compute_asset_count(self):

def unlink(self):
# for move in self:
deprs = self.env["account.asset.line"].search(
[("move_id", "in", self.ids), ("type", "in", ["depreciate", "remove"])]
deprs = (
self.env["account.asset.line"]
.sudo()
.search(
[("move_id", "in", self.ids), ("type", "in", ["depreciate", "remove"])]
)
)
if deprs and not self.env.context.get("unlink_from_asset"):
raise UserError(
Expand Down

0 comments on commit 8b8f2af

Please sign in to comment.