Do not send OptInType as a data field

dotmailer · Jul 23, 2024 · 11ed803 · 11ed803
1 parent 44c5bd1
commit 11ed803
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/Model/Sync/Subscriber/SingleSubscriberSyncer.php b/Model/Sync/Subscriber/SingleSubscriberSyncer.php
@@ -46,6 +46,8 @@ public function pushContactToSubscriberAddressBook(Contact $contact)
         $websiteId = (int) $contact->getWebsiteId();
         $subscriberSyncEnabled = $this->helper->isSubscriberSyncEnabled($websiteId);
         $subscriberAddressBookId = $this->helper->getSubscriberAddressBook($websiteId);
+        $optInType = null;
+
         if (!$subscriberSyncEnabled || !$subscriberAddressBookId) {
             return null;
         }
@@ -59,11 +61,19 @@ public function pushContactToSubscriberAddressBook(Contact $contact)
             )
         );
 
-        // optInType will be set in $subscriberDataFields if it is 'Double'
+        // optInType is not a data field - this will be refactored in a future release
+        foreach ($subscriberDataFields as $i => $field) {
+            if ($field['Key'] === 'OptInType') {
+                $optInType = $field['Value'];
+                unset($subscriberDataFields[$i]);
+                break;
+            }
+        }
+
         return $client->addContactToAddressBook(
             $contact->getEmail(),
             $subscriberAddressBookId,
-            null,
+            $optInType,
             $subscriberDataFields
         );
     }