From 672d7d2b6c3e3f84dd09610fb50d650dc22cf966 Mon Sep 17 00:00:00 2001
From: mmtr <miguel.torres@automattic.com>
Date: Wed, 5 Aug 2020 13:08:16 +0200
Subject: [PATCH] Keep in sync the donate buttons labels across all intervals

---
 extensions/blocks/donations/tab.js | 56 ++++++++++++++++--------------
 1 file changed, 29 insertions(+), 27 deletions(-)

diff --git a/extensions/blocks/donations/tab.js b/extensions/blocks/donations/tab.js
index 948ef136823f3..6ef7ea42da3be 100644
--- a/extensions/blocks/donations/tab.js
+++ b/extensions/blocks/donations/tab.js
@@ -22,14 +22,6 @@ const Tab = ( { activeTab, attributes, setAttributes } ) => {
 		customAmountText,
 	} = attributes;
 
-	const [ previousCurrency, setPreviousCurrency ] = useState( currency );
-	const minAmount = minimumTransactionAmountForCurrency( currency );
-	const [ defaultAmounts, setDefaultAmounts ] = useState( [
-		minAmount * 10, // 1st tier (USD 5)
-		minAmount * 30, // 2nd tier (USD 15)
-		minAmount * 200, // 3rd tier (USD 100)
-	] );
-
 	const donationAttributes = {
 		'one-time': 'oneTimeDonation',
 		'1 month': 'monthlyDonation',
@@ -49,42 +41,52 @@ const Tab = ( { activeTab, attributes, setAttributes } ) => {
 		} );
 	};
 
-	const amounts = getDonationValue( 'amounts' );
-
-	const setAmount = ( amount, tier ) => {
-		const newAmounts = [ ...amounts ];
-		newAmounts[ tier ] = amount;
-		setDonationValue( 'amounts', newAmounts );
-	};
-
 	// Updates the amounts whenever there are new defaults due to a currency change.
+	const [ previousCurrency, setPreviousCurrency ] = useState( currency );
+	const minAmount = minimumTransactionAmountForCurrency( currency );
+	const defaultAmounts = [
+		minAmount * 10, // 1st tier (USD 5)
+		minAmount * 30, // 2nd tier (USD 15)
+		minAmount * 200, // 3rd tier (USD 100)
+	];
 	useEffect( () => {
 		if ( previousCurrency === currency ) {
 			return;
 		}
 		setPreviousCurrency( currency );
 
-		const newDefaultAmounts = [
-			minAmount * 10, // 1st tier (USD 5)
-			minAmount * 30, // 2nd tier (USD 15)
-			minAmount * 200, // 3rd tier (USD 100)
-		];
-		setDefaultAmounts( newDefaultAmounts );
 		setAttributes( {
-			oneTimeDonation: { ...oneTimeDonation, amounts: newDefaultAmounts },
-			monthlyDonation: { ...monthlyDonation, amounts: newDefaultAmounts },
-			annualDonation: { ...annualDonation, amounts: newDefaultAmounts },
+			oneTimeDonation: { ...oneTimeDonation, amounts: defaultAmounts },
+			monthlyDonation: { ...monthlyDonation, amounts: defaultAmounts },
+			annualDonation: { ...annualDonation, amounts: defaultAmounts },
 		} );
 	}, [
 		currency,
 		previousCurrency,
-		minAmount,
+		defaultAmounts,
 		oneTimeDonation,
 		monthlyDonation,
 		annualDonation,
 		setAttributes,
 	] );
 
+	const amounts = getDonationValue( 'amounts' );
+
+	const setAmount = ( amount, tier ) => {
+		const newAmounts = [ ...amounts ];
+		newAmounts[ tier ] = amount;
+		setDonationValue( 'amounts', newAmounts );
+	};
+
+	// Keeps in sync the donate buttons labels across all intervals once the default value is overridden in one of them.
+	const setButtonText = buttonText => {
+		setAttributes( {
+			oneTimeDonation: { ...oneTimeDonation, buttonText: buttonText },
+			monthlyDonation: { ...monthlyDonation, buttonText: buttonText },
+			annualDonation: { ...annualDonation, buttonText: buttonText },
+		} );
+	};
+
 	return (
 		<div className="donations__tab">
 			<RichText
@@ -144,7 +146,7 @@ const Tab = ( { activeTab, attributes, setAttributes } ) => {
 					className="wp-block-button__link"
 					placeholder={ __( 'Write a message…', 'jetpack' ) }
 					value={ getDonationValue( 'buttonText' ) }
-					onChange={ value => setDonationValue( 'buttonText', value ) }
+					onChange={ value => setButtonText( value ) }
 				/>
 			</div>
 		</div>