From 455e463916b64e66c1abbdd6217f2a20ed2debfe Mon Sep 17 00:00:00 2001 From: John Oleksowicz Date: Mon, 6 Jun 2022 10:53:53 -0500 Subject: [PATCH] Better naming MAILPOET-4264 --- .../Builder/CreateWorkflowFromTemplateController.php | 8 ++++---- .../Templates/{Templates.php => WorkflowBuilder.php} | 6 +++--- mailpoet/lib/DI/ContainerConfigurator.php | 2 +- 3 files changed, 8 insertions(+), 8 deletions(-) rename mailpoet/lib/Automation/Integrations/MailPoet/Templates/{Templates.php => WorkflowBuilder.php} (95%) diff --git a/mailpoet/lib/Automation/Engine/Builder/CreateWorkflowFromTemplateController.php b/mailpoet/lib/Automation/Engine/Builder/CreateWorkflowFromTemplateController.php index 99e7eb0aa8..4a43080f97 100644 --- a/mailpoet/lib/Automation/Engine/Builder/CreateWorkflowFromTemplateController.php +++ b/mailpoet/lib/Automation/Engine/Builder/CreateWorkflowFromTemplateController.php @@ -4,19 +4,19 @@ namespace MailPoet\Automation\Engine\Builder; use MailPoet\Automation\Engine\Storage\WorkflowStorage; use MailPoet\Automation\Engine\Workflows\Workflow; -use MailPoet\Automation\Integrations\MailPoet\Templates\Templates; +use MailPoet\Automation\Integrations\MailPoet\Templates\WorkflowBuilder; use MailPoet\UnexpectedValueException; class CreateWorkflowFromTemplateController { /** @var WorkflowStorage */ private $storage; - /** @var Templates */ + /** @var WorkflowBuilder */ private $templates; public function __construct( WorkflowStorage $storage, - Templates $templates + WorkflowBuilder $templates ) { $this->storage = $storage; $this->templates = $templates; @@ -28,7 +28,7 @@ class CreateWorkflowFromTemplateController { switch ($template) { case 'delayed-email-after-signup': - $workflow = $this->templates->delayedEmailAfterSignup($name); + $workflow = $this->templates->delayedEmailAfterSignupWorkflow($name); break; default: throw UnexpectedValueException::create()->withMessage('Template not found.'); diff --git a/mailpoet/lib/Automation/Integrations/MailPoet/Templates/Templates.php b/mailpoet/lib/Automation/Integrations/MailPoet/Templates/WorkflowBuilder.php similarity index 95% rename from mailpoet/lib/Automation/Integrations/MailPoet/Templates/Templates.php rename to mailpoet/lib/Automation/Integrations/MailPoet/Templates/WorkflowBuilder.php index a31820880a..e9e4f1f208 100644 --- a/mailpoet/lib/Automation/Integrations/MailPoet/Templates/Templates.php +++ b/mailpoet/lib/Automation/Integrations/MailPoet/Templates/WorkflowBuilder.php @@ -9,7 +9,7 @@ use MailPoet\Automation\Integrations\MailPoet\Actions\SendWelcomeEmailAction; use MailPoet\Automation\Integrations\MailPoet\Triggers\SegmentSubscribedTrigger; use MailPoet\Util\Security; -class Templates { +class WorkflowBuilder { /** @var WaitAction */ private $waitAction; @@ -30,7 +30,7 @@ class Templates { $this->sendWelcomeEmailAction = $sendWelcomeEmailAction; } - public function delayedEmailAfterSignup(string $name): Workflow { + public function delayedEmailAfterSignupWorkflow(string $name): Workflow { $triggerStep = $this->segmentSubscribedTriggerStep(); $waitStep = $this->waitStep(60 * 60); @@ -38,7 +38,7 @@ class Templates { $sendEmailStep = $this->sendEmailActionStep(); $waitStep->setNextStepId($sendEmailStep->getId()); - + $steps = [ $triggerStep, $waitStep, diff --git a/mailpoet/lib/DI/ContainerConfigurator.php b/mailpoet/lib/DI/ContainerConfigurator.php index 4c1cdb9ac6..51607e5186 100644 --- a/mailpoet/lib/DI/ContainerConfigurator.php +++ b/mailpoet/lib/DI/ContainerConfigurator.php @@ -135,7 +135,7 @@ class ContainerConfigurator implements IContainerConfigurator { $container->autowire(\MailPoet\Automation\Integrations\MailPoet\Subjects\SegmentSubject::class)->setPublic(true)->setShared(false); $container->autowire(\MailPoet\Automation\Integrations\MailPoet\Subjects\SubscriberSubject::class)->setPublic(true)->setShared(false); $container->autowire(\MailPoet\Automation\Integrations\MailPoet\Triggers\SegmentSubscribedTrigger::class)->setPublic(true)->setShared(false); - $container->autowire(\MailPoet\Automation\Integrations\MailPoet\Templates\Templates::class)->setPublic(true)->setShared(false); + $container->autowire(\MailPoet\Automation\Integrations\MailPoet\Templates\WorkflowBuilder::class)->setPublic(true)->setShared(false); $container->autowire(\MailPoet\Automation\Integrations\MailPoet\Actions\SendWelcomeEmailAction::class)->setPublic(true); // Config $container->autowire(\MailPoet\Config\AccessControl::class)->setPublic(true);