From 3fe7087271b5efa82c70fe8d473fb63e91036f40 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 14 Dec 2025 06:17:03 +0000
Subject: [PATCH 1/4] Initial plan


From 14cdc3814ae509cd64f0f6b35ec1c7375f9f1f6a Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 14 Dec 2025 06:24:45 +0000
Subject: [PATCH 2/4] Fix getDerivedStateFromProps to handle simultaneous
 allowNegative and format prop changes

Co-authored-by: ericblade <1451847+ericblade@users.noreply.github.com>
---
 src/index.tsx      | 36 +++++++++++++++++-------------------
 tests/base.spec.ts | 32 ++++++++++++++++++++++++++++++++
 2 files changed, 49 insertions(+), 19 deletions(-)

diff --git a/src/index.tsx b/src/index.tsx
index 202b507..2b0a62d 100644
--- a/src/index.tsx
+++ b/src/index.tsx
@@ -189,13 +189,17 @@ class CurrencyInput extends React.Component<CurrencyInputProps, CurrencyInputSta
         // Check if the VALUE prop itself changed (parent is controlling the input)
         const valueChanged = nextProps.value !== previousProps.value;
 
+        // Check if allowNegative changed (affects whether negative values are allowed)
+        const allowNegativeChanged = nextProps.allowNegative !== previousProps.allowNegative;
+
         // Check if separators or display formatting changed (these require reformatting the current value)
         const formatChanged =
             nextProps.decimalSeparator !== previousProps.decimalSeparator ||
             nextProps.thousandSeparator !== previousProps.thousandSeparator ||
             nextProps.precision !== previousProps.precision ||
             nextProps.prefix !== previousProps.prefix ||
-            nextProps.suffix !== previousProps.suffix;
+            nextProps.suffix !== previousProps.suffix ||
+            allowNegativeChanged;
 
         if (valueChanged) {
             // Parent changed the value prop - use the new value
@@ -204,19 +208,12 @@ class CurrencyInput extends React.Component<CurrencyInputProps, CurrencyInputSta
         }
 
         if (formatChanged) {
-            // Display formatting changed - reformat the current value with new formatting
-            const propsWithCurrentValue = { ...nextProps, value: prevState.value };
-            const newState = CurrencyInput.prepareProps(propsWithCurrentValue);
-            return { ...newState, previousProps: nextProps };
-        }
-
-        // Check if allowNegative changed
-        const allowNegativeChanged = nextProps.allowNegative !== previousProps.allowNegative;
-
-        if (allowNegativeChanged) {
-            if (!nextProps.allowNegative) {
-                // allowNegative was disabled
-                // If current value is negative, make it positive
+            // Display formatting or allowNegative changed - reformat the current value
+            // First, determine the value to use (convert negative to positive if needed)
+            let valueToFormat = prevState.value;
+            
+            if (allowNegativeChanged && !nextProps.allowNegative) {
+                // allowNegative was disabled - if current value is negative, make it positive
                 const parsedValue = typeof prevState.value === 'number'
                     ? prevState.value
                     : prevState.value === null
@@ -224,13 +221,14 @@ class CurrencyInput extends React.Component<CurrencyInputProps, CurrencyInputSta
                         : CurrencyInput.stringValueToFloat(String(prevState.value), nextProps.thousandSeparator, nextProps.decimalSeparator);
 
                 if (parsedValue < 0) {
-                    const propsWithPositiveValue = { ...nextProps, value: Math.abs(parsedValue) };
-                    const newState = CurrencyInput.prepareProps(propsWithPositiveValue);
-                    return { ...newState, previousProps: nextProps };
+                    valueToFormat = Math.abs(parsedValue);
                 }
             }
-            // allowNegative toggled on or no sign change needed: still advance previousProps
-            return { ...prevState, previousProps: nextProps };
+            
+            // Reformat with new formatting and potentially adjusted value
+            const propsWithCurrentValue = { ...nextProps, value: valueToFormat };
+            const newState = CurrencyInput.prepareProps(propsWithCurrentValue);
+            return { ...newState, previousProps: nextProps };
         }
 
         // Other props changed but value and display formatting didn't
diff --git a/tests/base.spec.ts b/tests/base.spec.ts
index e594649..3e4cad7 100644
--- a/tests/base.spec.ts
+++ b/tests/base.spec.ts
@@ -244,6 +244,38 @@ test.describe('component parameters', () => {
             value = await currencyInput.inputValue();
             expect(value).not.toContain('-');
         });
+
+        test('removes negative sign when allowNegative is disabled and format changes simultaneously', async ({ page }) => {
+            // First enable allowNegative
+            const allowNegativeCheckbox = page.locator('[name=allowNegative]');
+            const prefixInput = page.locator('[name=prefix]');
+            const applyBtn = page.locator('[name=apply]');
+            const currencyInput = page.locator('#currency-input');
+
+            await allowNegativeCheckbox.check();
+            await applyBtn.click();
+            await expect(currencyInput).toHaveValue('$0.00 USD');
+            await currencyInput.focus();
+            await currencyInput.selectText();
+
+            // Input a negative number
+            await currencyInput.pressSequentially('50');
+            await currencyInput.press('Minus');
+
+            let value = await currencyInput.inputValue();
+            expect(value).toContain('-');
+
+            // Now disable allowNegative AND change prefix at the same time
+            await allowNegativeCheckbox.uncheck();
+            await prefixInput.fill('€');
+            await applyBtn.click();
+
+            // Value should now be positive AND have the new prefix
+            value = await currencyInput.inputValue();
+            expect(value).not.toContain('-');
+            expect(value).toContain('€');
+            expect(value).toContain('50');
+        });
     });
 
     test.describe('allowEmpty', () => {

From d06dc55d8b55391de87cfbc49c95952491fa1fd8 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 14 Dec 2025 06:26:25 +0000
Subject: [PATCH 3/4] Add clarifying comment for valueToFormat initialization

Co-authored-by: ericblade <1451847+ericblade@users.noreply.github.com>
---
 src/index.tsx | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/index.tsx b/src/index.tsx
index 2b0a62d..9cf1374 100644
--- a/src/index.tsx
+++ b/src/index.tsx
@@ -209,7 +209,8 @@ class CurrencyInput extends React.Component<CurrencyInputProps, CurrencyInputSta
 
         if (formatChanged) {
             // Display formatting or allowNegative changed - reformat the current value
-            // First, determine the value to use (convert negative to positive if needed)
+            // First, determine the value to use. Start with the current state value,
+            // which may be adjusted below if allowNegative was disabled and value is negative.
             let valueToFormat = prevState.value;
             
             if (allowNegativeChanged && !nextProps.allowNegative) {

From 4a2122f9f1d0bd9cf2405d124a17b8a03144b9ad Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 14 Dec 2025 06:28:11 +0000
Subject: [PATCH 4/4] Fix whitespace for consistency with codebase style

Co-authored-by: ericblade <1451847+ericblade@users.noreply.github.com>
---
 src/index.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/index.tsx b/src/index.tsx
index 9cf1374..1cd9df6 100644
--- a/src/index.tsx
+++ b/src/index.tsx
@@ -212,7 +212,7 @@ class CurrencyInput extends React.Component<CurrencyInputProps, CurrencyInputSta
             // First, determine the value to use. Start with the current state value,
             // which may be adjusted below if allowNegative was disabled and value is negative.
             let valueToFormat = prevState.value;
-            
+
             if (allowNegativeChanged && !nextProps.allowNegative) {
                 // allowNegative was disabled - if current value is negative, make it positive
                 const parsedValue = typeof prevState.value === 'number'
@@ -225,7 +225,7 @@ class CurrencyInput extends React.Component<CurrencyInputProps, CurrencyInputSta
                     valueToFormat = Math.abs(parsedValue);
                 }
             }
-            
+
             // Reformat with new formatting and potentially adjusted value
             const propsWithCurrentValue = { ...nextProps, value: valueToFormat };
             const newState = CurrencyInput.prepareProps(propsWithCurrentValue);