BounceAction enum

tatevikg1 · tatevikg1 · commit 56af5dd4c259 · 2026-04-29T14:29:38.000+04:00
diff --git a/src/Bounce/Service/Handler/BlacklistEmailAndDeleteBounceHandler.php b/src/Bounce/Service/Handler/BlacklistEmailAndDeleteBounceHandler.php
@@ -6,6 +6,7 @@
 
 use PhpList\Core\Bounce\Service\Manager\BounceManager;
 use PhpList\Core\Bounce\Service\SubscriberBlacklistService;
+use PhpList\Core\Domain\Messaging\Model\BounceAction;
 use PhpList\Core\Domain\Subscription\Service\Manager\SubscriberHistoryManager;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
@@ -30,7 +31,7 @@ public function __construct(
 
     public function supports(string $action): bool
     {
-        return $action === 'blacklistemailanddeletebounce';
+        return $action === BounceAction::BlacklistEmailAndDeleteBounce->value;
     }
 
     public function handle(array $closureData): void
diff --git a/src/Bounce/Service/Handler/BlacklistEmailHandler.php b/src/Bounce/Service/Handler/BlacklistEmailHandler.php
@@ -5,6 +5,7 @@
 namespace PhpList\Core\Bounce\Service\Handler;
 
 use PhpList\Core\Bounce\Service\SubscriberBlacklistService;
+use PhpList\Core\Domain\Messaging\Model\BounceAction;
 use PhpList\Core\Domain\Subscription\Service\Manager\SubscriberHistoryManager;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
@@ -26,7 +27,7 @@ public function __construct(
 
     public function supports(string $action): bool
     {
-        return $action === 'blacklistemail';
+        return $action === BounceAction::BlacklistEmail->value;
     }
 
     public function handle(array $closureData): void
diff --git a/src/Bounce/Service/Handler/BlacklistUserAndDeleteBounceHandler.php b/src/Bounce/Service/Handler/BlacklistUserAndDeleteBounceHandler.php
@@ -6,6 +6,7 @@
 
 use PhpList\Core\Bounce\Service\Manager\BounceManager;
 use PhpList\Core\Bounce\Service\SubscriberBlacklistService;
+use PhpList\Core\Domain\Messaging\Model\BounceAction;
 use PhpList\Core\Domain\Subscription\Service\Manager\SubscriberHistoryManager;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
@@ -30,7 +31,7 @@ public function __construct(
 
     public function supports(string $action): bool
     {
-        return $action === 'blacklistuseranddeletebounce';
+        return $action === BounceAction::BlacklistUserAndDeleteBounce->value;
     }
 
     public function handle(array $closureData): void
diff --git a/src/Bounce/Service/Handler/BlacklistUserHandler.php b/src/Bounce/Service/Handler/BlacklistUserHandler.php
@@ -5,6 +5,7 @@
 namespace PhpList\Core\Bounce\Service\Handler;
 
 use PhpList\Core\Bounce\Service\SubscriberBlacklistService;
+use PhpList\Core\Domain\Messaging\Model\BounceAction;
 use PhpList\Core\Domain\Subscription\Service\Manager\SubscriberHistoryManager;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
@@ -26,7 +27,7 @@ public function __construct(
 
     public function supports(string $action): bool
     {
-        return $action === 'blacklistuser';
+        return $action === BounceAction::BlacklistUser->value;
     }
 
     public function handle(array $closureData): void
diff --git a/src/Bounce/Service/Handler/DecreaseCountConfirmUserAndDeleteBounceHandler.php b/src/Bounce/Service/Handler/DecreaseCountConfirmUserAndDeleteBounceHandler.php
@@ -5,6 +5,7 @@
 namespace PhpList\Core\Bounce\Service\Handler;
 
 use PhpList\Core\Bounce\Service\Manager\BounceManager;
+use PhpList\Core\Domain\Messaging\Model\BounceAction;
 use PhpList\Core\Domain\Subscription\Repository\SubscriberRepository;
 use PhpList\Core\Domain\Subscription\Service\Manager\SubscriberHistoryManager;
 use Symfony\Contracts\Translation\TranslatorInterface;
@@ -30,7 +31,7 @@ public function __construct(
 
     public function supports(string $action): bool
     {
-        return $action === 'decreasecountconfirmuseranddeletebounce';
+        return $action === BounceAction::DecreaseCountConfirmUserAndDeleteBounce->value;
     }
 
     public function handle(array $closureData): void
diff --git a/src/Bounce/Service/Handler/DeleteBounceHandler.php b/src/Bounce/Service/Handler/DeleteBounceHandler.php
@@ -5,6 +5,7 @@
 namespace PhpList\Core\Bounce\Service\Handler;
 
 use PhpList\Core\Bounce\Service\Manager\BounceManager;
+use PhpList\Core\Domain\Messaging\Model\BounceAction;
 
 class DeleteBounceHandler implements BounceActionHandlerInterface
 {
@@ -17,7 +18,7 @@ public function __construct(BounceManager $bounceManager)
 
     public function supports(string $action): bool
     {
-        return $action === 'deletebounce';
+        return $action === BounceAction::DeleteBounce->value;
     }
 
     public function handle(array $closureData): void
diff --git a/src/Bounce/Service/Handler/DeleteUserAndBounceHandler.php b/src/Bounce/Service/Handler/DeleteUserAndBounceHandler.php
@@ -6,6 +6,7 @@
 
 use PhpList\Core\Bounce\Service\Manager\BounceManager;
 use PhpList\Core\Domain\Subscription\Service\Manager\SubscriberManager;
+use PhpList\Core\Domain\Messaging\Model\BounceAction;
 
 class DeleteUserAndBounceHandler implements BounceActionHandlerInterface
 {
@@ -20,7 +21,7 @@ public function __construct(BounceManager $bounceManager, SubscriberManager $sub
 
     public function supports(string $action): bool
     {
-        return $action === 'deleteuserandbounce';
+        return $action === BounceAction::DeleteUserAndBounce->value;
     }
 
     public function handle(array $closureData): void
diff --git a/src/Bounce/Service/Handler/DeleteUserHandler.php b/src/Bounce/Service/Handler/DeleteUserHandler.php
@@ -4,6 +4,7 @@
 
 namespace PhpList\Core\Bounce\Service\Handler;
 
+use PhpList\Core\Domain\Messaging\Model\BounceAction;
 use PhpList\Core\Domain\Subscription\Service\Manager\SubscriberManager;
 use Psr\Log\LoggerInterface;
 
@@ -20,7 +21,7 @@ public function __construct(SubscriberManager $subscriberManager, LoggerInterfac
 
     public function supports(string $action): bool
     {
-        return $action === 'deleteuser';
+        return $action === BounceAction::DeleteUser->value;
     }
 
     public function handle(array $closureData): void
diff --git a/src/Bounce/Service/Handler/UnconfirmUserAndDeleteBounceHandler.php b/src/Bounce/Service/Handler/UnconfirmUserAndDeleteBounceHandler.php
@@ -5,6 +5,7 @@
 namespace PhpList\Core\Bounce\Service\Handler;
 
 use PhpList\Core\Bounce\Service\Manager\BounceManager;
+use PhpList\Core\Domain\Messaging\Model\BounceAction;
 use PhpList\Core\Domain\Subscription\Repository\SubscriberRepository;
 use PhpList\Core\Domain\Subscription\Service\Manager\SubscriberHistoryManager;
 use Symfony\Contracts\Translation\TranslatorInterface;
@@ -30,7 +31,7 @@ public function __construct(
 
     public function supports(string $action): bool
     {
-        return $action === 'unconfirmuseranddeletebounce';
+        return $action === BounceAction::UnconfirmUserAndDeleteBounce->value;
     }
 
     public function handle(array $closureData): void
diff --git a/src/Bounce/Service/Handler/UnconfirmUserHandler.php b/src/Bounce/Service/Handler/UnconfirmUserHandler.php
@@ -4,6 +4,7 @@
 
 namespace PhpList\Core\Bounce\Service\Handler;
 
+use PhpList\Core\Domain\Messaging\Model\BounceAction;
 use PhpList\Core\Domain\Subscription\Repository\SubscriberRepository;
 use PhpList\Core\Domain\Subscription\Service\Manager\SubscriberHistoryManager;
 use Symfony\Contracts\Translation\TranslatorInterface;
@@ -26,7 +27,7 @@ public function __construct(
 
     public function supports(string $action): bool
     {
-        return $action === 'unconfirmuser';
+        return $action === BounceAction::UnconfirmUser->value;
     }
 
     public function handle(array $closureData): void
diff --git a/src/Domain/Messaging/Model/BounceAction.php b/src/Domain/Messaging/Model/BounceAction.php
@@ -0,0 +1,24 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PhpList\Core\Domain\Messaging\Model;
+
+enum BounceAction: string
+{
+    case BlacklistUser = 'blacklistuser';
+    case BlacklistEmailAndDeleteBounce = 'blacklistemailanddeletebounce';
+    case BlacklistEmail = 'blacklistemail';
+    case BlacklistUserAndDeleteBounce = 'blacklistuseranddeletebounce';
+    case DecreaseCountConfirmUserAndDeleteBounce = 'decreasecountconfirmuseranddeletebounce';
+    case DeleteBounce = 'deletebounce';
+    case DeleteUserAndBounce = 'deleteuserandbounce';
+    case DeleteUser = 'deleteuser';
+    case UnconfirmUserAndDeleteBounce = 'unconfirmuseranddeletebounce';
+    case UnconfirmUser = 'unconfirmuser';
+
+    public static function values(): array
+    {
+        return array_column(self::cases(), 'value');
+    }
+}
diff --git a/src/Domain/Messaging/Service/Manager/BounceRuleManager.php b/src/Domain/Messaging/Service/Manager/BounceRuleManager.php
@@ -64,7 +64,6 @@ private function mapRows(array $rows): array
         return $result;
     }
 
-
     /**
      * @param array<string,BounceRegex> $rules
      */

Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,7 @@`
`6`	`6`
`7`	`7`	`use PhpList\Core\Bounce\Service\Manager\BounceManager;`
`8`	`8`	`use PhpList\Core\Bounce\Service\SubscriberBlacklistService;`
	`9`	`+use PhpList\Core\Domain\Messaging\Model\BounceAction;`
`9`	`10`	`use PhpList\Core\Domain\Subscription\Service\Manager\SubscriberHistoryManager;`
`10`	`11`	`use Symfony\Contracts\Translation\TranslatorInterface;`
`11`	`12`
`@@ -30,7 +31,7 @@ public function __construct(`
`30`	`31`
`31`	`32`	`public function supports(string $action): bool`
`32`	`33`	`{`
`33`		`- return $action === 'blacklistemailanddeletebounce';`
	`34`	`+ return $action === BounceAction::BlacklistEmailAndDeleteBounce->value;`
`34`	`35`	`}`
`35`	`36`
`36`	`37`	`public function handle(array $closureData): void`
Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,7 @@`
`5`	`5`	`namespace PhpList\Core\Bounce\Service\Handler;`
`6`	`6`
`7`	`7`	`use PhpList\Core\Bounce\Service\SubscriberBlacklistService;`
	`8`	`+use PhpList\Core\Domain\Messaging\Model\BounceAction;`
`8`	`9`	`use PhpList\Core\Domain\Subscription\Service\Manager\SubscriberHistoryManager;`
`9`	`10`	`use Symfony\Contracts\Translation\TranslatorInterface;`
`10`	`11`
`@@ -26,7 +27,7 @@ public function __construct(`
`26`	`27`
`27`	`28`	`public function supports(string $action): bool`
`28`	`29`	`{`
`29`		`- return $action === 'blacklistemail';`
	`30`	`+ return $action === BounceAction::BlacklistEmail->value;`
`30`	`31`	`}`
`31`	`32`
`32`	`33`	`public function handle(array $closureData): void`
Original file line number	Diff line number	Diff line change
`@@ -4,6 +4,7 @@`
`4`	`4`
`5`	`5`	`namespace PhpList\Core\Bounce\Service\Handler;`
`6`	`6`
	`7`	`+use PhpList\Core\Domain\Messaging\Model\BounceAction;`
`7`	`8`	`use PhpList\Core\Domain\Subscription\Service\Manager\SubscriberManager;`
`8`	`9`	`use Psr\Log\LoggerInterface;`
`9`	`10`
`@@ -20,7 +21,7 @@ public function __construct(SubscriberManager $subscriberManager, LoggerInterfac`
`20`	`21`
`21`	`22`	`public function supports(string $action): bool`
`22`	`23`	`{`
`23`		`- return $action === 'deleteuser';`
	`24`	`+ return $action === BounceAction::DeleteUser->value;`
`24`	`25`	`}`
`25`	`26`
`26`	`27`	`public function handle(array $closureData): void`