diff --git a/src/documents/signals/handlers.py b/src/documents/signals/handlers.py index 0e582f2f3..6cafbc1f8 100644 --- a/src/documents/signals/handlers.py +++ b/src/documents/signals/handlers.py @@ -731,13 +731,6 @@ def run_workflows( else: apply_removal_to_document(action, document, doc_tag_ids) elif action.type == WorkflowAction.WorkflowActionType.EMAIL: - if not settings.EMAIL_ENABLED: - logger.error( - "Email backend has not been configured, cannot send email notifications", - extra={"group": logging_group}, - ) - continue - context = build_workflow_action_context(document, overrides) execute_email_action( action, diff --git a/src/documents/tests/test_workflows.py b/src/documents/tests/test_workflows.py index b3436ed33..e606bc5a0 100644 --- a/src/documents/tests/test_workflows.py +++ b/src/documents/tests/test_workflows.py @@ -3096,7 +3096,7 @@ class TestWorkflows( original_filename="sample.pdf", ) - with self.assertLogs("paperless.handlers", level="ERROR") as cm: + with self.assertLogs("paperless.workflows.actions", level="ERROR") as cm: run_workflows(WorkflowTrigger.WorkflowTriggerType.DOCUMENT_UPDATED, doc) expected_str = "Email backend has not been configured" diff --git a/src/documents/workflows/actions.py b/src/documents/workflows/actions.py index ab66bd646..040cbc127 100644 --- a/src/documents/workflows/actions.py +++ b/src/documents/workflows/actions.py @@ -90,6 +90,13 @@ def execute_email_action( Execute an email action for a workflow. """ + if not settings.EMAIL_ENABLED: + logger.error( + "Email backend has not been configured, cannot send email notifications", + extra={"group": logging_group}, + ) + return + subject = ( parse_w_workflow_placeholders( action.email.subject,