33040 add is subscription enabled checking to new action controller

Author: silinmykola

app/code/Magento/Newsletter/Controller/Subscriber/NewAction.php

@@ -4,6 +4,8 @@
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
+declare(strict_types=1);
+
 namespace Magento\Newsletter\Controller\Subscriber;
 
 use Magento\Customer\Api\AccountManagementInterface as CustomerAccountManagement;
@@ -23,6 +25,7 @@
 use Magento\Newsletter\Model\SubscriptionManagerInterface;
 use Magento\Store\Model\ScopeInterface;
 use Magento\Store\Model\StoreManagerInterface;
+use Magento\Newsletter\Model\Config as NewsletterConfig;
 use Magento\Newsletter\Model\SubscriberFactory;
 
 /**
@@ -42,6 +45,11 @@ class NewAction extends SubscriberController implements HttpPostActionInterface
      */
     private $emailValidator;
 
+    /**
+     * @var NewsletterConfig
+     */
+    private $newsletterConfig;
+
     /**
      * @var SubscriptionManagerInterface
      */
@@ -57,7 +65,8 @@ class NewAction extends SubscriberController implements HttpPostActionInterface
      * @param CustomerUrl $customerUrl
      * @param CustomerAccountManagement $customerAccountManagement
      * @param SubscriptionManagerInterface $subscriptionManager
-     * @param EmailValidator $emailValidator
+     * @param EmailValidator|null $emailValidator
+     * @param NewsletterConfig|null $newsletterConfig
      */
     public function __construct(
         Context $context,
@@ -67,11 +76,13 @@ public function __construct(
         CustomerUrl $customerUrl,
         CustomerAccountManagement $customerAccountManagement,
         SubscriptionManagerInterface $subscriptionManager,
-        EmailValidator $emailValidator = null
+        EmailValidator $emailValidator = null,
+        NewsletterConfig $newsletterConfig = null
     ) {
         $this->customerAccountManagement = $customerAccountManagement;
         $this->subscriptionManager = $subscriptionManager;
         $this->emailValidator = $emailValidator ?: ObjectManager::getInstance()->get(EmailValidator::class);
+        $this->newsletterConfig = $newsletterConfig?: ObjectManager::getInstance()->get(NewsletterConfig::class);
         parent::__construct(
             $context,
             $subscriberFactory,
@@ -85,8 +96,9 @@ public function __construct(
      * Validates that the email address isn't being used by a different account.
      *
      * @param string $email
-     * @throws LocalizedException
+     *
      * @return void
+     * @throws LocalizedException
      */
     protected function validateEmailAvailable($email)
     {
@@ -129,8 +141,9 @@ protected function validateGuestSubscription()
      * Validates the format of the email address
      *
      * @param string $email
-     * @throws LocalizedException
+     *
      * @return void
+     * @throws LocalizedException
      */
     protected function validateEmailFormat($email)
     {
@@ -146,7 +159,10 @@ protected function validateEmailFormat($email)
      */
     public function execute()
     {
-        if ($this->getRequest()->isPost() && $this->getRequest()->getPost('email')) {
+        if ($this->getRequest()->isPost()
+            && $this->getRequest()->getPost('email')
+            && $this->newsletterConfig->isActive()
+        ) {
             $email = (string)$this->getRequest()->getPost('email');
 
             try {
@@ -183,13 +199,15 @@ public function execute()
         /** @var Redirect $redirect */
         $redirect = $this->resultFactory->create(ResultFactory::TYPE_REDIRECT);
         $redirectUrl = $this->_redirect->getRedirectUrl();
+
         return $redirect->setUrl($redirectUrl);
     }
 
     /**
      * Get customer id from session if he is owner of the email
      *
      * @param string $email
+     *
      * @return int|null
      */
     private function getSessionCustomerId(string $email): ?int
@@ -210,6 +228,7 @@ private function getSessionCustomerId(string $email): ?int
      * Get success message
      *
      * @param int $status
+     *
      * @return Phrase
      */
     private function getSuccessMessage(int $status): Phrase

dev/tests/integration/testsuite/Magento/Newsletter/Controller/Subscriber/NewActionTest.php

@@ -7,14 +7,15 @@
 
 namespace Magento\Newsletter\Controller\Subscriber;
 
+use Exception;
+use Laminas\Stdlib\Parameters;
 use Magento\Customer\Api\CustomerRepositoryInterface;
 use Magento\Customer\Model\Session;
 use Magento\Customer\Model\Url;
 use Magento\Framework\App\Request\Http as HttpRequest;
 use Magento\Newsletter\Model\ResourceModel\Subscriber as SubscriberResource;
 use Magento\Newsletter\Model\ResourceModel\Subscriber\CollectionFactory;
 use Magento\TestFramework\TestCase\AbstractController;
-use Laminas\Stdlib\Parameters;
 
 /**
  * Class checks subscription behaviour from frontend
@@ -24,6 +25,9 @@
  */
 class NewActionTest extends AbstractController
 {
+    /** @var CustomerRepositoryInterface */
+    private $customerRepository;
+
     /** @var Session */
     private $session;
 
@@ -36,9 +40,6 @@ class NewActionTest extends AbstractController
     /** @var string|null */
     private $subscriberToDelete;
 
-    /** @var CustomerRepositoryInterface */
-    private $customerRepository;
-
     /** @var Url */
     private $customerUrl;
 
@@ -84,6 +85,38 @@ public function testNewAction(string $email, string $expectedMessage): void
         $this->performAsserts($expectedMessage);
     }
 
+    /**
+     * @magentoConfigFixture newsletter/general/active 1
+     *
+     * @return void
+     */
+    public function testNewActionWithSubscriptionConfigEnabled(): void
+    {
+        $email = 'good_subscription@example.com';
+        $this->subscriberToDelete = $email;
+        $this->prepareRequest($email);
+        $this->dispatch('newsletter/subscriber/new');
+        $subscriberCollection = $this->subscriberCollectionFactory->create();
+        $subscriberCollection->addFieldToFilter('subscriber_email', $email)->setPageSize(1);
+        $this->assertEquals(1, count($subscriberCollection));
+        $this->assertEquals($email, $subscriberCollection->getFirstItem()->getEmail());
+    }
+
+    /**
+     * @magentoConfigFixture newsletter/general/active 0
+     *
+     * @return void
+     */
+    public function testNewActionWithSubscriptionConfigDisabled(): void
+    {
+        $email = 'bad_subscription@example.com';
+        $this->prepareRequest($email);
+        $this->dispatch('newsletter/subscriber/new');
+        $subscriberCollection = $this->subscriberCollectionFactory->create();
+        $subscriberCollection->addFieldToFilter('subscriber_email', $email)->setPageSize(1);
+        $this->assertEquals(0, count($subscriberCollection));
+    }
+
     /**
      * @return array
      */
@@ -242,7 +275,9 @@ private function performAsserts(string $message): void
      * Delete subscribers by email
      *
      * @param string $email
+     *
      * @return void
+     * @throws Exception
      */
     private function deleteSubscriber(string $email): void
     {