From 8828bc73081eaf817867e72fb0e6183da4d3963e Mon Sep 17 00:00:00 2001 From: Altahir Hassan Date: Tue, 6 Jan 2026 10:26:41 +0400 Subject: [PATCH] fix: Correct active template handling in create method to ensure proper activation logic --- odex30_base/expert_theme/models/expert_login_template.py | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/odex30_base/expert_theme/models/expert_login_template.py b/odex30_base/expert_theme/models/expert_login_template.py index 1d3c1da..613f77b 100644 --- a/odex30_base/expert_theme/models/expert_login_template.py +++ b/odex30_base/expert_theme/models/expert_login_template.py @@ -111,14 +111,13 @@ class ExpertLoginTemplate(models.Model): def create(self, vals): """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 'active' not in vals[0]: - vals[0]['active'] = False + if 'active' not in vals: + vals['active'] = False # Only deactivate others if this one is being set to active - print(vals[0]) - if vals[0].get('active'): + if vals.get('active'): # Deactivate all other templates (excluding the one being created) self.search([('active', '=', True)]).write({'active': False}) - return super(ExpertLoginTemplate, self).create(vals[0]) + return super(ExpertLoginTemplate, self).create(vals) def write(self, vals): """Ensure only one template is active at a time"""