addHistory for unconfirmed

Author: tatevikg1

resources/translations/messages.en.xlf

@@ -692,14 +692,22 @@ Thank you.</target>
         <source>No user details changed</source>
         <target>__No user details changed</target>
       </trans-unit>
-      <trans-unit id="1R4L28q" resname="%field% = %new% *changed* from %old%">
+      <trans-unit id="qdTR5bi" resname="%field% = %new% *changed* from %old%">
         <source>%field% = %new% *changed* from %old%</source>
         <target>__%field% = %new% *changed* from %old%</target>
       </trans-unit>
       <trans-unit id="rns1KOG" resname="Subscribed to %list%">
         <source>Subscribed to %list%</source>
         <target>__Subscribed to %list%</target>
       </trans-unit>
+      <trans-unit id="eXllcQa" resname="Subscriber marked unconfirmed for invalid email address">
+        <source>Subscriber marked unconfirmed for invalid email address</source>
+        <target>__Subscriber marked unconfirmed for invalid email address</target>
+      </trans-unit>
+      <trans-unit id="oY0Gck3" resname="Marked unconfirmed while sending campaign %message_id%">
+        <source>Marked unconfirmed while sending campaign %message_id%</source>
+        <target>__Marked unconfirmed while sending campaign %message_id%</target>
+      </trans-unit>
     </body>
   </file>
 </xliff>

src/Domain/Messaging/Service/Processor/CampaignProcessor.php

@@ -14,6 +14,7 @@
 use PhpList\Core\Domain\Messaging\Service\RateLimitedCampaignMailer;
 use PhpList\Core\Domain\Messaging\Service\MaxProcessTimeLimiter;
 use PhpList\Core\Domain\Messaging\Service\MessageProcessingPreparator;
+use PhpList\Core\Domain\Subscription\Service\Manager\SubscriberHistoryManager;
 use PhpList\Core\Domain\Subscription\Service\Provider\SubscriberProvider;
 use PhpList\Core\Domain\Subscription\Model\Subscriber;
 use Psr\Log\LoggerInterface;
@@ -35,6 +36,7 @@ class CampaignProcessor
     private MaxProcessTimeLimiter $timeLimiter;
     private RequeueHandler $requeueHandler;
     private TranslatorInterface $translator;
+    private SubscriberHistoryManager $subscriberHistoryManager;
 
     public function __construct(
         RateLimitedCampaignMailer $mailer,
@@ -46,6 +48,7 @@ public function __construct(
         MaxProcessTimeLimiter $timeLimiter,
         RequeueHandler $requeueHandler,
         TranslatorInterface $translator,
+        SubscriberHistoryManager $subscriberHistoryManager,
     ) {
         $this->mailer = $mailer;
         $this->entityManager = $entityManager;
@@ -56,6 +59,7 @@ public function __construct(
         $this->timeLimiter = $timeLimiter;
         $this->requeueHandler = $requeueHandler;
         $this->translator = $translator;
+        $this->subscriberHistoryManager = $subscriberHistoryManager;
     }
 
     public function process(Message $campaign, ?OutputInterface $output = null): void
@@ -89,6 +93,14 @@ public function process(Message $campaign, ?OutputInterface $output = null): voi
                 $output?->writeln($this->translator->trans('Invalid email, marking unconfirmed: %email%', [
                     '%email%' => $subscriber->getEmail(),
                 ]));
+                $this->subscriberHistoryManager->addHistory(
+                    subscriber: $subscriber,
+                    message: $this->translator->trans('Subscriber marked unconfirmed for invalid email address'),
+                    details: $this->translator->trans(
+                        'Marked unconfirmed while sending campaign %message_id%',
+                        ['%message_id%' => $campaign->getId()]
+                    )
+                );
                 continue;
             }