From b65367fb6de31352cab6f1f11c0f394dc87f1fbc Mon Sep 17 00:00:00 2001 From: TekH Date: Mon, 9 Mar 2026 21:26:52 +0100 Subject: [PATCH] Refactor FinalizeDocumentJob dependencies and method call Removed unused constructor parameters from FinalizeDocumentJob for cleaner dependency injection. Updated actionService.FinalizeEnvelope to accept a cancellation token. Added missing using directive for Microsoft.Extensions.Options. --- EnvelopeGenerator.ServiceHost/Jobs/FinalizeDocumentJob.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/EnvelopeGenerator.ServiceHost/Jobs/FinalizeDocumentJob.cs b/EnvelopeGenerator.ServiceHost/Jobs/FinalizeDocumentJob.cs index 94faa23b..5811fad0 100644 --- a/EnvelopeGenerator.ServiceHost/Jobs/FinalizeDocumentJob.cs +++ b/EnvelopeGenerator.ServiceHost/Jobs/FinalizeDocumentJob.cs @@ -19,7 +19,7 @@ using System.Data; namespace EnvelopeGenerator.ServiceHost.Jobs; [Obsolete("ActionService is a placeholder service added by copilot. Migrate the actual logic from CommonServices.Jobs")] -public class FinalizeDocumentJob(IOptions options, IConfiguration config, ILogger logger, TempFiles tempFiles, ActionService actionService, PDFBurner pdfBurner, PDFMerger pdfMerger, ReportCreator reportCreator, ReportModel _reportModel, MSSQLServer _database, GdViewer? _gdViewer, LicenseManager licenseManager, IMediator mediator, IRepository envRepo, IRepository docStatusRepo) +public class FinalizeDocumentJob(IOptions options, ILogger logger, TempFiles tempFiles, ActionService actionService, PDFBurner pdfBurner, PDFMerger pdfMerger, ReportCreator reportCreator, GdViewer? _gdViewer, LicenseManager licenseManager, IMediator mediator, IRepository envRepo, IRepository docStatusRepo) { private readonly WorkerOptions _options = options.Value; @@ -108,7 +108,7 @@ public class FinalizeDocumentJob(IOptions options, IConfiguration SendFinalEmails(envelope); - actionService?.FinalizeEnvelope(envelope); + actionService?.FinalizeEnvelope(envelope, cancel); } private bool SendFinalEmails(Envelope envelope)