From ea232315bf6b6e3de413ab0d2ac260b279cd5a48 Mon Sep 17 00:00:00 2001 From: Alan Rominger Date: Wed, 3 Jul 2024 16:07:46 -0400 Subject: [PATCH] Do not reference self.messages when it does not exist (#15331) --- awx/main/access.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/awx/main/access.py b/awx/main/access.py index 7c93a7b8d8..9765b2ff91 100644 --- a/awx/main/access.py +++ b/awx/main/access.py @@ -2100,15 +2100,18 @@ class WorkflowJobTemplateAccess(NotificationAttachMixin, BaseAccess): if not self.check_related('organization', Organization, data, role_field='workflow_admin_role', mandatory=True): if data.get('organization', None) is None: - self.messages['organization'] = [_('An organization is required to create a workflow job template for normal user')] + if self.save_messages: + self.messages['organization'] = [_('An organization is required to create a workflow job template for normal user')] return False if not self.check_related('inventory', Inventory, data, role_field='use_role'): - self.messages['inventory'] = [_('You do not have use_role to the inventory')] + if self.save_messages: + self.messages['inventory'] = [_('You do not have use_role to the inventory')] return False if not self.check_related('execution_environment', ExecutionEnvironment, data, role_field='read_role'): - self.messages['execution_environment'] = [_('You do not have read_role to the execution environment')] + if self.save_messages: + self.messages['execution_environment'] = [_('You do not have read_role to the execution environment')] return False return True