refactor: sanitize code

Author: RishadAlam

frontend-dev/src/components/AllIntegrations/MailerPress/EditMailerPress.jsx

@@ -4,10 +4,9 @@ import { useRecoilState, useRecoilValue } from 'recoil'
 import { $actionConf, $formFields, $newFlow } from '../../../GlobalStates'
 import { __ } from '../../../Utils/i18nwrap'
 import SnackMsg from '../../Utilities/SnackMsg'
-import EditFormInteg from '../EditFormInteg'
-import SetEditIntegComponents from '../IntegrationHelpers/SetEditIntegComponents'
 import { saveActionConf } from '../IntegrationHelpers/IntegrationHelpers'
 import IntegrationStepThree from '../IntegrationHelpers/IntegrationStepThree'
+import SetEditIntegComponents from '../IntegrationHelpers/SetEditIntegComponents'
 import { checkMappedFields, handleInput } from './MailerPressCommonFunc'
 import MailerPressIntegLayout from './MailerPressIntegLayout'
 

frontend-dev/src/components/AllIntegrations/MailerPress/MailerPress.jsx

@@ -104,3 +104,4 @@ export default function MailerPress({ formFields, setFlow, flow, allIntegURL })
     </div>
   )
 }
+

frontend-dev/src/components/AllIntegrations/MailerPress/MailerPressAuthorization.jsx

@@ -2,13 +2,8 @@ import { useState } from 'react'
 import BackIcn from '../../../Icons/BackIcn'
 import bitsFetch from '../../../Utils/bitsFetch'
 import { __ } from '../../../Utils/i18nwrap'
-import LoaderSm from '../../Loaders/LoaderSm'
-import {
-  refreshMailerPressLists,
-  refreshMailerPressTags,
-  mailerPressListHeaders
-} from './MailerPressCommonFunc'
 import tutorialLinks from '../../../Utils/StaticData/tutorialLinks'
+import LoaderSm from '../../Loaders/LoaderSm'
 import TutorialLink from '../../Utilities/TutorialLink'
 
 export default function MailerPressAuthorization({

frontend-dev/src/components/AllIntegrations/MailerPress/MailerPressCommonFunc.js

@@ -1,7 +1,7 @@
+import { create } from 'mutative'
 import toast from 'react-hot-toast'
 import bitsFetch from '../../../Utils/bitsFetch'
-import { __, sprintf } from '../../../Utils/i18nwrap'
-import { create } from 'mutative'
+import { __ } from '../../../Utils/i18nwrap'
 
 export const handleInput = (e, mailerPressConf, setMailerPressConf) => {
   const { name, value } = e.target
@@ -55,35 +55,13 @@ export const refreshMailerPressTags = (setMailerPressConf, setIsLoading, setSnac
     .catch(() => setIsLoading(false))
 }
 
-export const mailerPressListHeaders = (setMailerPressConf, setIsLoading, setSnackbar) => {
-  setIsLoading(true)
-  bitsFetch(null, 'mailer_press_list_headers')
-    .then(result => {
-      if (result && result.success) {
-        setMailerPressConf(oldConf => {
-          const newConf = { ...oldConf }
-          if (result.data.mailerPressFields) {
-            newConf.mailerPressFields = result.data.mailerPressFields
-          }
-          return newConf
-        })
-        setIsLoading(false)
-        toast.success(__('Fields fetched successfully', 'bit-integrations'))
-        return
-      }
-      setIsLoading(false)
-      toast.error(__('MailerPress field fetch failed. Please try again', 'bit-integrations'))
-    })
-    .catch(() => setIsLoading(false))
-}
-
 export const checkMappedFields = mailerPressConf => {
   const mappedFields = mailerPressConf?.field_map
     ? mailerPressConf.field_map.filter(
         mappedField =>
           !mappedField.formField ||
           !mappedField.mailerPressField ||
-          (!mappedField.formField === 'custom' && !mappedField.customValue)
+          (mappedField.formField === 'custom' && !mappedField.customValue)
       )
     : []
   if (mappedFields.length > 0) {

frontend-dev/src/components/AllIntegrations/MailerPress/MailerPressFieldMap.jsx

@@ -1,6 +1,6 @@
 import { useRecoilValue } from 'recoil'
 import { $btcbi } from '../../../GlobalStates'
-import { __ } from '../../../Utils/i18nwrap'
+import { __, sprintf } from '../../../Utils/i18nwrap'
 import { SmartTagField } from '../../../Utils/StaticData/SmartTagField'
 import TagifyInput from '../../Utilities/TagifyInput'
 import {

frontend-dev/src/components/AllIntegrations/MailerPress/MailerPressIntegLayout.jsx

@@ -1,19 +1,18 @@
+import { create } from 'mutative'
 import MultiSelect from 'react-multiple-select-dropdown-lite'
+import { useRecoilValue } from 'recoil'
+import { $btcbi } from '../../../GlobalStates'
 import { __ } from '../../../Utils/i18nwrap'
 import Loader from '../../Loaders/Loader'
+import { checkIsPro, getProLabel } from '../../Utilities/ProUtilHelpers'
 import { addFieldMap } from '../IntegrationHelpers/IntegrationHelpers'
 import {
   generateMappedField,
   refreshMailerPressLists,
   refreshMailerPressTags
 } from './MailerPressCommonFunc'
 import MailerPressFieldMap from './MailerPressFieldMap'
-import { create } from 'mutative'
 import { ContactFields, emailField, modules } from './staticData'
-import { generate } from 'nth-check'
-import { checkIsPro, getProLabel } from '../../Utilities/ProUtilHelpers'
-import { useRecoilValue } from 'recoil'
-import { $btcbi } from '../../../GlobalStates'
 
 export default function MailerPressIntegLayout({
   formID,

includes/Actions/MailerPress/MailerPressController.php

@@ -150,3 +150,4 @@ private static function convertStringToArray($value, $separator = ',')
         return array_filter($array);
     }
 }
+

includes/Actions/MailerPress/RecordApiHelper.php

@@ -178,6 +178,11 @@ function ($id) {
                 'message' => $isUpdate ? __('Contact updated successfully', 'bit-integrations') : __('Contact created successfully', 'bit-integrations')
             ];
         }
+
+        return [
+            'success' => false,
+            'message' => $result['message'] ?? __('Failed to create or update contact', 'bit-integrations')
+        ];
     }
 
     /**

includes/Actions/MailerPress/Routes.php

@@ -10,5 +10,3 @@
 Route::post('mailer_press_authorize', [MailerPressController::class, 'mailerPressAuthorize']);
 Route::post('refresh_mailer_press_lists', [MailerPressController::class, 'refreshLists']);
 Route::post('refresh_mailer_press_tags', [MailerPressController::class, 'refreshTags']);
-Route::post('mailer_press_list_headers', [MailerPressController::class, 'mailerPressListHeaders']);
-