magento#8952 Fixed subscribe to newsletter if you already have an acc…

…ount issue update changes as per suggestion
ravi-chandra3197 · Oct 29, 2018 · 6de5586 · 6de5586
1 parent a2d02fe
commit 6de5586
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/app/code/Magento/Newsletter/Controller/Subscriber/NewAction.php b/app/code/Magento/Newsletter/Controller/Subscriber/NewAction.php
@@ -77,14 +77,12 @@ public function __construct(
     protected function validateEmailAvailable($email)
     {
         $websiteId = $this->_storeManager->getStore()->getWebsiteId();
-        if (isset($this->_customerSession->getCustomerDataObject()->getEmail())) {
-            if ($this->_customerSession->getCustomerDataObject()->getEmail() !== $email
-                && !$this->customerAccountManagement->isEmailAvailable($email, $websiteId)
-            ) {
-                throw new LocalizedException(
-                    __('This email address is already assigned to another user.')
-                );
-            }
+        if (isset($this->_customerSession->getCustomerDataObject()->getEmail()) && ($this->_customerSession->getCustomerDataObject()->getEmail() !== $email
+            && !$this->customerAccountManagement->isEmailAvailable($email, $websiteId)
+        )) {
+            throw new \Magento\Framework\Exception\LocalizedException(
+                __('This email address is already assigned to another user.')
+            );
         }
     }