Merge pull request #1 from outeredge/main

Change data to comma separator
outeredge · Feb 10, 2022 · 1d5aafe · 1d5aafe
2 parents 92da502 + cd17247
commit 1d5aafe
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/Block/Form/Options.php b/Block/Form/Options.php
@@ -66,12 +66,8 @@ public function getNewsletterOptionsByCustomer($newsletterOption)
         $customerNewsOpt = $customer->getCustomAttribute('newsletter_options')->getValue();
 
         if (!empty($customerNewsOpt)) {
-            $result = json_decode($customerNewsOpt, true);
-
-            if (isset($result[$newsletterOption])) {
-                if ($result[$newsletterOption] == 1) {
-                    return true;
-                }
+            if (strpos($customerNewsOpt, $newsletterOption) !== false) {
+                return true;
             }
         }
         return false;

diff --git a/Observer/UpdateOptions.php b/Observer/UpdateOptions.php
@@ -47,7 +47,7 @@ public function execute(Observer $observer) : void
     {
         $newsOptions = $observer->getRequest()->getParam('newsletter_options', false);
 
-        $dataToSave = json_encode($newsOptions);
+        $dataToSave = implode(',', array_keys($newsOptions));
 
         $storeId = (int)$this->storeManager->getStore()->getId();
         $websiteId = (int)$this->storeManager->getStore($storeId)->getWebsiteId();