From b05a3e933fb433f1e40f79814966e7d7632afe5d Mon Sep 17 00:00:00 2001 From: Philipp Stracker Date: Mon, 16 Sep 2024 17:56:00 +0200 Subject: [PATCH] =?UTF-8?q?=F0=9F=94=A5=20Remove=20useless=20state=20value?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- modules/ppcp-axo-block/resources/js/hooks/useAxoSetup.js | 5 ++--- .../ppcp-axo-block/resources/js/hooks/usePhoneSyncHandler.js | 4 +--- modules/ppcp-axo-block/resources/js/index.js | 4 +--- 3 files changed, 4 insertions(+), 9 deletions(-) diff --git a/modules/ppcp-axo-block/resources/js/hooks/useAxoSetup.js b/modules/ppcp-axo-block/resources/js/hooks/useAxoSetup.js index 721aca97d..3fd77cf48 100644 --- a/modules/ppcp-axo-block/resources/js/hooks/useAxoSetup.js +++ b/modules/ppcp-axo-block/resources/js/hooks/useAxoSetup.js @@ -17,8 +17,7 @@ const useAxoSetup = ( paymentComponent, onChangeCardButtonClick, setShippingAddress, - setCard, - setWooPhone + setCard ) => { const { setIsAxoActive, setIsAxoScriptLoaded } = useDispatch( STORE_NAME ); const paypalLoaded = usePayPalScript( ppcpConfig ); @@ -35,7 +34,7 @@ const useAxoSetup = ( setBillingAddress: setWooBillingAddress, } = useCustomerData(); - usePhoneSyncHandler( paymentComponent, setWooPhone ); + usePhoneSyncHandler( paymentComponent ); useEffect( () => { console.log( 'Initializing class toggles' ); diff --git a/modules/ppcp-axo-block/resources/js/hooks/usePhoneSyncHandler.js b/modules/ppcp-axo-block/resources/js/hooks/usePhoneSyncHandler.js index 50baddc23..d293a2c30 100644 --- a/modules/ppcp-axo-block/resources/js/hooks/usePhoneSyncHandler.js +++ b/modules/ppcp-axo-block/resources/js/hooks/usePhoneSyncHandler.js @@ -46,10 +46,9 @@ const updatePrefills = ( paymentComponent, phoneNumber ) => { /** * Custom hook to synchronize the WooCommerce phone number with a React component state. * - * @param {Function} setWooPhone - The state setter function for the phone number. * @param {Object} paymentComponent - The CardField component from Fastlane. */ -export const usePhoneSyncHandler = ( paymentComponent, setWooPhone ) => { +export const usePhoneSyncHandler = ( paymentComponent ) => { // Fetch and sanitize phone number from WooCommerce. const phoneNumber = useSelect( ( select ) => getSanitizedPhoneNumber( select ) @@ -58,7 +57,6 @@ export const usePhoneSyncHandler = ( paymentComponent, setWooPhone ) => { // Create a debounced function that updates the prefilled phone-number. const debouncedUpdatePhone = useRef( debounce( ( number, component ) => { - setWooPhone( number ); updatePrefills( component, number ); }, PHONE_DEBOUNCE_DELAY ) ).current; diff --git a/modules/ppcp-axo-block/resources/js/index.js b/modules/ppcp-axo-block/resources/js/index.js index 8457be7c3..82acff78b 100644 --- a/modules/ppcp-axo-block/resources/js/index.js +++ b/modules/ppcp-axo-block/resources/js/index.js @@ -27,7 +27,6 @@ const Axo = ( props ) => { const { onPaymentSetup } = eventRegistration; const [ shippingAddress, setShippingAddress ] = useState( null ); const [ card, setCard ] = useState( null ); - const [ wooPhone, setWooPhone ] = useState( '' ); const [ paymentComponent, setPaymentComponent ] = useState( null ); const fastlaneSdk = useFastlaneSdk( axoConfig, ppcpConfig ); @@ -46,8 +45,7 @@ const Axo = ( props ) => { paymentComponent, onChangeCardButtonClick, setShippingAddress, - setCard, - setWooPhone + setCard ); const { handlePaymentLoad } = usePaymentSetupEffect(