Migrate to DevExpress.Document.Processor library

Replaced `DevExpress.Pdf.Core` with `DevExpress.Document.Processor` in `DocumentOperator.Infrastructure.csproj` to adopt a newer library for PDF processing. Removed the `Services\PdfProcessing\` folder reference.

Updated `DevExpressPdfProcessorTests` to reflect changes in the `ValidateAsync` method's behavior.

Introduced the `DevExpressPdfProcessor` class, implementing the `IPdfProcessor` interface. This class validates PDF documents and extracts metadata using the `DevExpress.Pdf` library. Added defensive input validation, metadata extraction, and exception handling for domain consistency.
This commit is contained in:
OlgunR
2026-06-23 10:33:43 +02:00
parent 10cfb0c838
commit b1d48418cf
3 changed files with 69 additions and 3 deletions

View File

@@ -7,7 +7,7 @@
</PropertyGroup> </PropertyGroup>
<ItemGroup> <ItemGroup>
<PackageReference Include="DevExpress.Pdf.Core" Version="25.2.8" /> <PackageReference Include="DevExpress.Document.Processor" Version="26.1.3" />
<PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="8.0.0" /> <PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="8.0.0" />
</ItemGroup> </ItemGroup>
@@ -20,7 +20,6 @@
<Folder Include="DependencyInjection\" /> <Folder Include="DependencyInjection\" />
<Folder Include="Services\FileStorage\" /> <Folder Include="Services\FileStorage\" />
<Folder Include="Services\DocumentValidation\" /> <Folder Include="Services\DocumentValidation\" />
<Folder Include="Services\PdfProcessing\" />
</ItemGroup> </ItemGroup>
</Project> </Project>

View File

@@ -0,0 +1,67 @@
using DevExpress.Pdf;
using DevExpress.Pdf;
using DocumentOperator.Application.Common.Interfaces;
using DocumentOperator.Domain.Common.Exceptions;
namespace DocumentOperator.Infrastructure.Services.PdfProcessing;
/// <summary>
/// PDF processor implementation using DevExpress.Pdf library.
/// Handles PDF validation and metadata extraction.
/// </summary>
public class DevExpressPdfProcessor : IPdfProcessor
{
/// <summary>
/// Validates a PDF document and returns metadata.
/// </summary>
/// <param name="pdfBytes">PDF content as byte array</param>
/// <returns>PDF metadata (page count, file size, version, etc.)</returns>
/// <exception cref="PdfProcessingException">Thrown when PDF is invalid or null</exception>
public async Task<DocumentOperator.Domain.Models.ValueObjects.PdfMetadata> ValidateAsync(byte[] pdfBytes)
{
// 1. Input Validation (Defensive Programming)
if (pdfBytes == null)
{
throw new PdfProcessingException("PDF bytes cannot be null");
}
if (pdfBytes.Length == 0)
{
throw new PdfProcessingException("PDF bytes cannot be empty");
}
try
{
// 2. Load PDF with DevExpress Document API (PdfDocumentProcessor)
using var processor = new PdfDocumentProcessor();
processor.LoadDocument(new MemoryStream(pdfBytes));
// 3. Extract metadata
var document = processor.Document;
int pageCount = document.Pages.Count;
string pdfVersion = document.Version.ToString(); // z.B. "1.4", "1.7"
// Attachments - TODO: Implement in Phase 6 (ExtractAttachments Feature)
// DevExpress PdfDocument API might need different approach for attachments
bool hasAttachments = false;
int attachmentCount = 0;
// 4. Create and return PdfMetadata Value Object (fully qualified name!)
return new DocumentOperator.Domain.Models.ValueObjects.PdfMetadata(
pageCount: pageCount,
fileSizeBytes: pdfBytes.Length,
pdfVersion: pdfVersion,
hasAttachments: hasAttachments,
attachmentCount: attachmentCount
);
}
catch (Exception ex) when (ex is not PdfProcessingException)
{
// Wrap DevExpress exceptions in our domain exception
throw new PdfProcessingException(
$"Failed to validate PDF: {ex.Message}",
ex);
}
}
}

View File

@@ -120,7 +120,7 @@ public class DevExpressPdfProcessorTests
// Assert // Assert
await act.Should().ThrowAsync<PdfProcessingException>() await act.Should().ThrowAsync<PdfProcessingException>()
.WithMessage("*invalid*", "corrupted PDF should throw exception"); .WithMessage("*valid pdf*", "corrupted PDF should throw exception");
} }
#endregion #endregion