Merge branch 'develop' into as-fix-ece-shipping-subs-with-free-trial-…

…and-sign-up-fee

changelog.txt

@@ -1,5 +1,9 @@
 *** WooPayments Changelog ***
 
+= 8.6.1 - 2024-12-17 =
+* Fix - Checkout: Fix error when wc_address_i18n_params does not have data for a given country
+* Fix - Skip mysqlcheck SSL Requirement during E2E environment setup
+
 = 8.6.0 - 2024-12-04 =
 * Add - Add Bank reference key column in Payout reports. This will help reconcile WooPayments Payouts with bank statements.
 * Add - Display credit card brand icons on order received page.

changelog/fix-9736-remove-temporary-payment-activity-transaction-search-css

@@ -0,0 +1,4 @@
+Significance: patch
+Type: fix
+
+Fix inconsistent alignment of the download button across transactions, payouts, and disputes reporting views for a more cohesive user interface.

changelog/fix-9794-refresh-page-when-ece-dismissed

@@ -0,0 +1,4 @@
+Significance: patch
+Type: add
+
+Refresh the cart and checkout pages when ECE is dismissed and the shipping options were modified in the payment sheet.

changelog/fix-allow-addresses-from-woo-supported-countries

@@ -0,0 +1,4 @@
+Significance: patch
+Type: fix
+
+Checkout: Fix error when wc_address_i18n_params does not have data for a given country

changelog/fix-unhandled-promises

@@ -0,0 +1,4 @@
+Significance: minor
+Type: fix
+
+Fix payment method filtering when billing country changes in Blocks checkout.

changelog/update-pass-capabilities-to-onboarding-as-get-params

@@ -0,0 +1,4 @@
+Significance: minor
+Type: dev
+
+Add support for utilizing NOX capabilities as URL parameters during account creation.

client/checkout/utils/test/upe.test.js

@@ -12,6 +12,7 @@ import {
 	isUsingSavedPaymentMethod,
 	dispatchChangeEventFor,
 	togglePaymentMethodForCountry,
+	isBillingInformationMissing,
 } from '../upe';
 
 import { getPaymentMethodsConstants } from '../../constants';
@@ -22,11 +23,134 @@ jest.mock( 'wcpay/utils/checkout' );
 
 jest.mock( '../../constants', () => {
 	return {
+		...jest.requireActual( '../../constants' ),
 		getPaymentMethodsConstants: jest.fn(),
 	};
 } );
 
+function buildForm( fields ) {
+	const form = document.createElement( 'form' );
+	fields.forEach( ( field ) => {
+		const input = document.createElement( 'input' );
+		input.id = field.id;
+		input.value = field.value;
+		form.appendChild( input );
+	} );
+	return form;
+}
+
 describe( 'UPE checkout utils', () => {
+	describe( 'isBillingInformationMissing', () => {
+		beforeAll( () => {
+			window.wc_address_i18n_params = {
+				locale: {
+					US: {},
+					HK: {
+						postcode: { required: false },
+					},
+					default: {
+						address_1: { required: true },
+						postcode: { required: true },
+					},
+				},
+			};
+		} );
+
+		beforeEach( () => {
+			getUPEConfig.mockImplementation( ( argument ) => {
+				if ( argument === 'enabledBillingFields' ) {
+					return {
+						billing_first_name: {
+							required: true,
+						},
+						billing_last_name: {
+							required: true,
+						},
+						billing_company: {
+							required: false,
+						},
+						billing_country: {
+							required: true,
+						},
+						billing_address_1: {
+							required: true,
+						},
+						billing_address_2: {
+							required: false,
+						},
+						billing_city: {
+							required: true,
+						},
+						billing_state: {
+							required: true,
+						},
+						billing_postcode: {
+							required: true,
+						},
+						billing_phone: {
+							required: true,
+						},
+						billing_email: {
+							required: true,
+						},
+					};
+				}
+			} );
+		} );
+
+		it( 'should return false when the billing information is not missing', () => {
+			const form = buildForm( [
+				{ id: 'billing_first_name', value: 'Test' },
+				{ id: 'billing_last_name', value: 'User' },
+				{ id: 'billing_email', value: '[email protected]' },
+				{ id: 'billing_country', value: 'US' },
+				{ id: 'billing_address_1', value: '123 Main St' },
+				{ id: 'billing_city', value: 'Anytown' },
+				{ id: 'billing_postcode', value: '12345' },
+			] );
+			expect( isBillingInformationMissing( form ) ).toBe( false );
+		} );
+
+		it( 'should return true when the billing information is missing', () => {
+			const form = buildForm( [
+				{ id: 'billing_first_name', value: 'Test' },
+				{ id: 'billing_last_name', value: 'User' },
+				{ id: 'billing_email', value: '[email protected]' },
+				{ id: 'billing_country', value: 'US' },
+				{ id: 'billing_address_1', value: '123 Main St' },
+				{ id: 'billing_city', value: 'Anytown' },
+				{ id: 'billing_postcode', value: '' },
+			] );
+			expect( isBillingInformationMissing( form ) ).toBe( true );
+		} );
+
+		it( 'should use the defaults when there is no specific locale data for a country', () => {
+			const form = buildForm( [
+				{ id: 'billing_first_name', value: 'Test' },
+				{ id: 'billing_last_name', value: 'User' },
+				{ id: 'billing_email', value: '[email protected]' },
+				{ id: 'billing_country', value: 'MX' },
+				{ id: 'billing_address_1', value: '123 Main St' },
+				{ id: 'billing_city', value: 'Anytown' },
+				{ id: 'billing_postcode', value: '' },
+			] );
+			expect( isBillingInformationMissing( form ) ).toBe( true );
+		} );
+
+		it( 'should return false when the locale data for a country has no required fields', () => {
+			const form = buildForm( [
+				{ id: 'billing_first_name', value: 'Test' },
+				{ id: 'billing_last_name', value: 'User' },
+				{ id: 'billing_email', value: '[email protected]' },
+				{ id: 'billing_country', value: 'HK' },
+				{ id: 'billing_address_1', value: '123 Main St' },
+				{ id: 'billing_city', value: 'Anytown' },
+				{ id: 'billing_postcode', value: '' },
+			] );
+			expect( isBillingInformationMissing( form ) ).toBe( true );
+		} );
+	} );
+
 	describe( 'getSelectedUPEGatewayPaymentMethod', () => {
 		let container;
 
@@ -54,7 +178,7 @@ describe( 'UPE checkout utils', () => {
 		} );
 
 		test( 'Selected UPE Payment Method is card', () => {
-			container.innerHTML = `<input 
+			container.innerHTML = `<input
 				id="payment_method_woocommerce_payments"
 				value="woocommerce_payments"
 				name="payment_method"
@@ -67,12 +191,12 @@ describe( 'UPE checkout utils', () => {
 
 		test( 'Selected UPE Payment Method is bancontact', () => {
 			container.innerHTML = `
-				<input 
-					id="payment_method_woocommerce_payments_bancontact" 
-					value="woocommerce_payments_bancontact" 
-					name="payment_method" 
-					type="radio" 
-					class="input-radio" 
+				<input
+					id="payment_method_woocommerce_payments_bancontact"
+					value="woocommerce_payments_bancontact"
+					name="payment_method"
+					type="radio"
+					class="input-radio"
 					checked
 				></input>
 			`;

client/checkout/utils/upe.js

@@ -449,8 +449,8 @@ export const isBillingInformationMissing = ( form ) => {
 			if ( country && locale && fieldName !== 'billing_email' ) {
 				const key = fieldName.replace( 'billing_', '' );
 				isRequired =
-					locale[ country ][ key ]?.required ??
-					locale.default[ key ]?.required;
+					locale[ country ]?.[ key ]?.required ??
+					locale.default?.[ key ]?.required;
 			}
 
 			const hasValue = field?.value;

client/connect-account-page/index.tsx

@@ -215,6 +215,7 @@ const ConnectAccountPage: React.FC = () => {
 
 		const customizedConnectUrl = addQueryArgs( connectUrl, {
 			test_drive: 'true',
+			capabilities: urlParams.get( 'capabilities' ) || '',
 		} );
 
 		const updateProgress = setInterval( updateLoaderProgress, 2500, 40, 5 );

client/express-checkout/blocks/index.js

@@ -39,9 +39,7 @@ const expressCheckoutElementApplePay = ( api ) => ( {
 			return false;
 		}
 
-		return new Promise( ( resolve ) => {
-			checkPaymentMethodIsAvailable( 'applePay', cart, resolve );
-		} );
+		return checkPaymentMethodIsAvailable( 'applePay', cart );
 	},
 } );
 
@@ -77,9 +75,7 @@ const expressCheckoutElementGooglePay = ( api ) => {
 				return false;
 			}
 
-			return new Promise( ( resolve ) => {
-				checkPaymentMethodIsAvailable( 'googlePay', cart, resolve );
-			} );
+			return checkPaymentMethodIsAvailable( 'googlePay', cart );
 		},
 	};
 };

client/express-checkout/event-handlers.js

@@ -13,12 +13,15 @@ import {
 	normalizeShippingAddress,
 	normalizeLineItems,
 	getExpressCheckoutData,
+	updateShippingAddressUI,
 } from './utils';
 import {
 	trackExpressCheckoutButtonClick,
 	trackExpressCheckoutButtonLoad,
 } from './tracking';
 
+let lastSelectedAddress = null;
+
 export const shippingAddressChangeHandler = async ( api, event, elements ) => {
 	try {
 		const response = await api.expressCheckoutECECalculateShippingOptions(
@@ -29,6 +32,9 @@ export const shippingAddressChangeHandler = async ( api, event, elements ) => {
 			elements.update( {
 				amount: response.total.amount,
 			} );
+
+			lastSelectedAddress = event.address;
+
 			event.resolve( {
 				shippingRates: response.shipping_options,
 				lineItems: normalizeLineItems( response.displayItems ),
@@ -171,5 +177,9 @@ export const onCompletePaymentHandler = () => {
 };
 
 export const onCancelHandler = () => {
+	if ( lastSelectedAddress ) {
+		updateShippingAddressUI( lastSelectedAddress );
+	}
+	lastSelectedAddress = null;
 	unblockUI();
 };