From 7866135d6cf903ef5c89e2e851e4f12c357d3115 Mon Sep 17 00:00:00 2001 From: Lila Date: Mon, 28 Nov 2022 15:26:23 -0500 Subject: [PATCH] Indented block and removed credential_name. --- awx/main/models/unified_jobs.py | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/awx/main/models/unified_jobs.py b/awx/main/models/unified_jobs.py index a9ab3abe0c..01f8f084f1 100644 --- a/awx/main/models/unified_jobs.py +++ b/awx/main/models/unified_jobs.py @@ -1351,14 +1351,13 @@ class UnifiedJob( for required in credential.credential_type.inputs.get('required', []): if required in defined_fields and not credential.has_input(required): missing_credential_inputs.append(required) - credential_name = credential.name - if missing_credential_inputs: - self.job_explanation = '{} cannot start because Credential {} does not provide one or more required fields ({}).'.format( - self._meta.verbose_name.title(), credential_name, ', '.join(sorted(missing_credential_inputs)) - ) - self.save(update_fields=['job_explanation']) - return (False, None) + if missing_credential_inputs: + self.job_explanation = '{} cannot start because Credential {} does not provide one or more required fields ({}).'.format( + self._meta.verbose_name.title(), credential.name, ', '.join(sorted(missing_credential_inputs)) + ) + self.save(update_fields=['job_explanation']) + return (False, None) needed = self.get_passwords_needed_to_start() try: