refactor: use sudo() in product archive and write methods to bypass multi-company restrictions
This commit is contained in:
parent
48fa2d0dd7
commit
971f004a26
@ -5,40 +5,24 @@ class ProductTemplate(models.Model):
|
||||
_inherit = 'product.template'
|
||||
|
||||
def action_archive(self):
|
||||
if not self.env.su:
|
||||
if hasattr(self.env.user, '_get_company_ids'):
|
||||
allowed_companies = list(self.env.user._get_company_ids())
|
||||
else:
|
||||
allowed_companies = self.env.user.company_ids.filtered('active').ids
|
||||
self = self.with_context(allowed_company_ids=allowed_companies)
|
||||
return super().action_archive()
|
||||
# Use sudo() to bypass multi-company checks during archival cascade
|
||||
return super(ProductTemplate, self.sudo()).action_archive()
|
||||
|
||||
def write(self, vals):
|
||||
if not self.env.su and 'active' in vals and not vals['active']:
|
||||
if hasattr(self.env.user, '_get_company_ids'):
|
||||
allowed_companies = list(self.env.user._get_company_ids())
|
||||
else:
|
||||
allowed_companies = self.env.user.company_ids.filtered('active').ids
|
||||
self = self.with_context(allowed_company_ids=allowed_companies)
|
||||
# Use sudo() during archival write to bypass multi-company checks
|
||||
if 'active' in vals and not vals['active']:
|
||||
return super(ProductTemplate, self.sudo()).write(vals)
|
||||
return super().write(vals)
|
||||
|
||||
class ProductProduct(models.Model):
|
||||
_inherit = 'product.product'
|
||||
|
||||
def action_archive(self):
|
||||
if not self.env.su:
|
||||
if hasattr(self.env.user, '_get_company_ids'):
|
||||
allowed_companies = list(self.env.user._get_company_ids())
|
||||
else:
|
||||
allowed_companies = self.env.user.company_ids.filtered('active').ids
|
||||
self = self.with_context(allowed_company_ids=allowed_companies)
|
||||
return super().action_archive()
|
||||
# Use sudo() to bypass multi-company checks during archival cascade
|
||||
return super(ProductProduct, self.sudo()).action_archive()
|
||||
|
||||
def write(self, vals):
|
||||
if not self.env.su and 'active' in vals and not vals['active']:
|
||||
if hasattr(self.env.user, '_get_company_ids'):
|
||||
allowed_companies = list(self.env.user._get_company_ids())
|
||||
else:
|
||||
allowed_companies = self.env.user.company_ids.filtered('active').ids
|
||||
self = self.with_context(allowed_company_ids=allowed_companies)
|
||||
# Use sudo() during archival write to bypass multi-company checks
|
||||
if 'active' in vals and not vals['active']:
|
||||
return super(ProductProduct, self.sudo()).write(vals)
|
||||
return super().write(vals)
|
||||
|
||||
Loading…
Reference in New Issue
Block a user