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 without 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
aren't allowed to access to that model.

TT49673
  • Loading branch information
pedrobaeza committed Jun 19, 2024
1 parent 09ff13b commit 660f05f
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 660f05f

Please sign in to comment.