Merge pull request #75 from expsa/odex_theme_branch
fix: Correct active template handling in create method to ensure prop…
This commit is contained in:
commit
a5591dfaf6
|
|
@ -111,14 +111,13 @@ class ExpertLoginTemplate(models.Model):
|
||||||
def create(self, vals):
|
def create(self, vals):
|
||||||
"""Ensure only one template is active at a time"""
|
"""Ensure only one template is active at a time"""
|
||||||
# If no active value is set, default to False (don't auto-activate new templates)
|
# If no active value is set, default to False (don't auto-activate new templates)
|
||||||
if 'active' not in vals[0]:
|
if 'active' not in vals:
|
||||||
vals[0]['active'] = False
|
vals['active'] = False
|
||||||
# Only deactivate others if this one is being set to active
|
# Only deactivate others if this one is being set to active
|
||||||
print(vals[0])
|
if vals.get('active'):
|
||||||
if vals[0].get('active'):
|
|
||||||
# Deactivate all other templates (excluding the one being created)
|
# Deactivate all other templates (excluding the one being created)
|
||||||
self.search([('active', '=', True)]).write({'active': False})
|
self.search([('active', '=', True)]).write({'active': False})
|
||||||
return super(ExpertLoginTemplate, self).create(vals[0])
|
return super(ExpertLoginTemplate, self).create(vals)
|
||||||
|
|
||||||
def write(self, vals):
|
def write(self, vals):
|
||||||
"""Ensure only one template is active at a time"""
|
"""Ensure only one template is active at a time"""
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue