Prevent high numbers from breaking currency inputs (woocommerce#49489)

Neosate · Jul 16, 2024 · 4dbe1c9 · 4dbe1c9
1 parent 8d23ac1
commit 4dbe1c9
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/packages/js/product-editor/changelog/fix-high-number-currency-round b/packages/js/product-editor/changelog/fix-high-number-currency-round
@@ -0,0 +1,5 @@
+Significance: patch
+Type: fix
+Comment: Prevent high numbers from breaking currency inputs
+
+
diff --git a/packages/js/product-editor/src/hooks/use-currency-input-props.ts b/packages/js/product-editor/src/hooks/use-currency-input-props.ts
@@ -27,6 +27,8 @@ type Props = {
 	onKeyUp?: ( event: React.KeyboardEvent< HTMLInputElement > ) => void;
 };
 
+const CURRENCY_INPUT_MAX = 1_000_000_000_000_000_000.0;
+
 export const useCurrencyInputProps = ( {
 	value,
 	onChange,
@@ -72,7 +74,11 @@ export const useCurrencyInputProps = ( {
 		onChange( newValue: string ) {
 			const sanitizeValue = sanitizePrice( newValue );
 			if ( onChange ) {
-				onChange( sanitizeValue );
+				onChange(
+					Number( sanitizeValue ) <= CURRENCY_INPUT_MAX
+						? sanitizeValue
+						: String( CURRENCY_INPUT_MAX )
+				);
 			}
 		},
 	};