fix: mailerlite change state issue fixed

Author: RishadAlam

frontend-dev/src/components/AllIntegrations/MailerLite/MailerLiteCommonFunc.js

@@ -7,15 +7,15 @@ import { create } from 'mutative'
 export const handleInput = (e, mailerLiteConf, setMailerLiteConf, loading, setLoading) => {
   const { name, value } = e.target
 
-  setMailerLiteConf(prev =>
-    create(prev, draftConf => {
-      draftConf[name] = value
+  const updatedConf = create(mailerLiteConf, draftConf => {
+    draftConf[name] = value
+  })
 
-      if (name === 'action' && value !== '') {
-        mailerliteRefreshFields(draftConf, setMailerLiteConf, loading, setLoading)
-      }
-    })
-  )
+  setMailerLiteConf(updatedConf)
+
+  if (name === 'action' && value !== '') {
+    mailerliteRefreshFields(updatedConf, setMailerLiteConf, loading, setLoading)
+  }
 }
 
 export const generateMappedField = mailerLiteConf => {

frontend-dev/src/components/AllIntegrations/MailerLite/MailerLiteIntegLayout.jsx

@@ -5,6 +5,7 @@ import MailerLiteFieldMap from './MailerLiteFieldMap'
 import MailerLiteActions from './MailerLiteActions'
 import { mailerliteRefreshFields } from './MailerLiteCommonFunc'
 import { useState } from 'react'
+import Note from '../../Utilities/Note'
 
 export default function MailerLiteIntegLayout({
   formFields,
@@ -102,6 +103,8 @@ export default function MailerLiteIntegLayout({
         </>
       )}
 
+      {mailerLiteConf?.action && mailerLiteConf?.action !== 'add_subscriber' && <Note note={note} />}
+
       {mailerLiteConf?.action && mailerLiteConf?.action === 'add_subscriber' && (
         <>
           <div className="mt-4">
@@ -120,3 +123,7 @@ export default function MailerLiteIntegLayout({
     </>
   )
 }
+
+const note = `
+    <p>${__('This action requires a MailerLite New account. It isn’t supported with Classic accounts.', 'bit-integrations')}</p>
+  `

includes/Actions/MailerLite/RecordApiHelper.php

@@ -25,10 +25,14 @@ class RecordApiHelper
 
     private $_actions;
 
+    private $_isMailerLiteV2;
+
     public function __construct($auth_token, $integrationDetails, $integId, $actions, $version)
     {
         $this->_integrationDetails = $integrationDetails;
         $this->_integrationID = $integId;
+        $this->_isMailerLiteV2 = (bool) ('v2' === $version);
+
         if ('v2' === $version) {
             $this->_baseUrl = 'https://connect.mailerlite.com/api/';
             $this->_defaultHeader = [
@@ -94,10 +98,9 @@ public function addSubscriber($auth_token, $groupIds, $type, $finalData)
 
         $email = $finalData['email'];
         $splitGroupIds = !empty($groupIds) ? explode(',', $groupIds) : [];
-        $isMailerLiteV2 = $this->_baseUrl === 'https://connect.mailerlite.com/api/';
         $apiEndpoint = $this->_baseUrl . 'subscribers';
 
-        $requestParams = self::prepareRequestParams($finalData, $type, $isMailerLiteV2);
+        $requestParams = self::prepareRequestParams($finalData, $type, $this->_isMailerLiteV2);
 
         $isExist = $this->existSubscriber($auth_token, $email);
         $response = null;
@@ -110,10 +113,10 @@ public function addSubscriber($auth_token, $groupIds, $type, $finalData)
             ];
         }
 
-        self::handleDoubleOptIn($this, $auth_token, $requestParams, $isMailerLiteV2);
+        self::handleDoubleOptIn($this, $auth_token, $requestParams, $this->_isMailerLiteV2);
 
         if (!empty($splitGroupIds)) {
-            return self::sendToGroups($this, $splitGroupIds, $requestParams, $isMailerLiteV2);
+            return self::sendToGroups($this, $splitGroupIds, $requestParams, $this->_isMailerLiteV2);
         }
 
         if ($isExist) {
@@ -128,6 +131,14 @@ public function addSubscriber($auth_token, $groupIds, $type, $finalData)
 
     public function deleteSubscriber($auth_token, $finalData)
     {
+        if (!$this->_isMailerLiteV2) {
+            return [
+                'success' => false,
+                'message' => __('This action is not supported for Classic accounts.', 'bit-integrations'),
+                'code'    => 400
+            ];
+        }
+
         if (empty($finalData['email'])) {
             return [
                 'success' => false,
@@ -136,7 +147,6 @@ public function deleteSubscriber($auth_token, $finalData)
             ];
         }
 
-        // $isMailerLiteV2 = $this->_baseUrl === 'https://connect.mailerlite.com/api/';
         $subscriberId = $this->existSubscriber($auth_token, $finalData['email']);
 
         if (empty($subscriberId)) {