AttachmentDownloadService

Author: tatevikg1

src/Domain/Messaging/Exception/AttachmentFileNotFoundException.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PhpList\Core\Domain\Messaging\Exception;
+
+class AttachmentFileNotFoundException extends \RuntimeException
+{
+    public function __construct()
+    {
+        parent::__construct('Attachment file not available');
+    }
+}

src/Domain/Messaging/Model/Dto/DownloadableAttachment.php

@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PhpList\Core\Domain\Messaging\Model\Dto;
+
+use Psr\Http\Message\StreamInterface;
+
+final class DownloadableAttachment
+{
+    public function __construct(
+        public readonly string $filename,
+        public readonly string $mimeType,
+        public readonly int $size,
+        public readonly StreamInterface $content,
+    ) {
+    }
+}

src/Domain/Messaging/Service/AttachmentAdder.php

@@ -58,8 +58,7 @@ public function add(Email $email, int $campaignId, OutputFormat $format, bool $f
 
                 case OutputFormat::Text:
                     $hash = $forwarded ? 'forwarded' : $email->getTo()[0]->getAddress();
-                    // todo: add endpoint in rest-api project
-                    $viewUrl = $this->attachmentDownloadUrl . '/?id=' . $att->getId() . '&uid=' . $hash;
+                    $viewUrl = $this->attachmentDownloadUrl . '/' . $att->getId() . '/?uid=' . $hash;
 
                     $email->text(
                         $email->getTextBody()

src/Domain/Messaging/Service/AttachmentDownloadService.php

@@ -0,0 +1,53 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PhpList\Core\Domain\Messaging\Service;
+
+use GuzzleHttp\Psr7\Utils;
+use PhpList\Core\Domain\Messaging\Exception\AttachmentFileNotFoundException;
+use PhpList\Core\Domain\Messaging\Model\Attachment;
+use PhpList\Core\Domain\Messaging\Model\Dto\DownloadableAttachment;
+use Psr\Http\Message\StreamInterface;
+use Symfony\Component\DependencyInjection\Attribute\Autowire;
+use Symfony\Component\Mime\MimeTypes;
+
+class AttachmentDownloadService
+{
+    public function __construct(
+        #[Autowire('%phplist.attachment_repository_path%')] private readonly string $attachmentRepositoryPath = '/tmp',
+    ) {
+    }
+
+    public function getDownloadable(Attachment $attachment): DownloadableAttachment
+    {
+        $filename = $attachment->getFilename();
+        if ($filename === null || $filename === '') {
+            throw new AttachmentFileNotFoundException();
+        }
+
+        $path = rtrim($this->attachmentRepositoryPath, '/');
+        $filePath = $path . '/' . $filename;
+
+        if (!is_file($filePath) || !is_readable($filePath)) {
+            throw new AttachmentFileNotFoundException();
+        }
+
+        $mimeType = $attachment->getMimeType()
+            ?? MimeTypes::getDefault()->guessMimeType($filePath)
+            ?? 'application/octet-stream';
+
+        $size = filesize($filePath);
+        $size = $size === false ? null : $size;
+
+        /** @var StreamInterface $stream */
+        $stream = Utils::streamFor(Utils::tryFopen($filePath, 'rb'));
+
+        return new DownloadableAttachment(
+            filename: $filename,
+            mimeType: $mimeType,
+            size: $size,
+            content: $stream,
+        );
+    }
+}

tests/Unit/Domain/Messaging/Service/AttachmentAdderTest.php

@@ -91,7 +91,7 @@ public function testTextModePrependsNoticeAndLinks(): void
         );
         $this->assertStringContainsString('Doc description', $body);
         $this->assertStringContainsString('Location', $body);
-        $this->assertStringContainsString('https://dl.example/?id=42&uid=user@example.com', $body);
+        $this->assertStringContainsString('https://dl.example/42/?uid=user@example.com', $body);
     }
 
     public function testHtmlUsesRepositoryFileIfExists(): void