Merge branch 'next-22891/fix-newsletter-route-does-not-handle-double-…

…optin-setting-correctly' into '6.4.18.1'

NEXT-22891 - Fix newsletter route does not handle double optin setting correctly

See merge request shopware/6/product/platform!8819

Content/Newsletter/SalesChannel/NewsletterSubscribeRoute.php

@@ -175,7 +175,7 @@ public function subscribe(RequestDataBag $dataBag, SalesChannelContext $context,
 
         $recipient = $this->getNewsletterRecipient('email', $data['email'], $context->getContext());
 
-        if ($data['status'] === self::STATUS_DIRECT) {
+        if (!$this->systemConfigService->getBool('core.newsletter.doubleOptIn')) {
             $event = new NewsletterConfirmEvent($context->getContext(), $recipient, $context->getSalesChannel()->getId());
             $this->eventDispatcher->dispatch($event);
 
@@ -262,8 +262,8 @@ private function getNewsletterRecipientId(string $email, SalesChannelContext $co
     private function getOptionSelection(): array
     {
         return [
-            self::OPTION_DIRECT => self::STATUS_DIRECT,
-            self::OPTION_SUBSCRIBE => self::STATUS_NOT_SET,
+            self::OPTION_DIRECT => $this->systemConfigService->getBool('core.newsletter.doubleOptIn') ? self::STATUS_NOT_SET : self::STATUS_DIRECT,
+            self::OPTION_SUBSCRIBE => $this->systemConfigService->getBool('core.newsletter.doubleOptIn') ? self::STATUS_NOT_SET : self::STATUS_DIRECT,
             self::OPTION_CONFIRM_SUBSCRIBE => self::STATUS_OPT_IN,
             self::OPTION_UNSUBSCRIBE => self::STATUS_OPT_OUT,
         ];