From 56e903db8c0a9e529c1668b25c8384dd17ac647e Mon Sep 17 00:00:00 2001 From: Lukasz Zajaczkowski Date: Thu, 29 Mar 2018 07:45:25 +0200 Subject: [PATCH] Fix misleading validation error Story: 2001758 Task: 12156 Change-Id: I76154ba2d3bc2993b5b3b8e37351ce2282a13f90 --- monitoring/alarmdefs/workflows.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/monitoring/alarmdefs/workflows.py b/monitoring/alarmdefs/workflows.py index 07c0808b..49c40836 100644 --- a/monitoring/alarmdefs/workflows.py +++ b/monitoring/alarmdefs/workflows.py @@ -100,13 +100,15 @@ class SetAlarmDefinitionAction(workflows.Action): def clean(self): cleaned_data = super(SetAlarmDefinitionAction, self).clean() - alarm_def_name = cleaned_data.get('name', '') + alarm_def_name = cleaned_data.get('name', '').strip() + if not alarm_def_name: + return is_name_valid = self._is_alarm_def_name_unique_validator( alarm_def_name) if not is_name_valid: self.add_error('name', _('Alarm definition with %s name already exists') - % alarm_def_name) + % alarm_def_name) def _is_alarm_def_name_unique_validator(self, value): try: