Merge pull request #15567 from craftcms/bugfix/15565-money-value-can-…

…start-with-a-minus

money value can start with a minus

CHANGELOG.md

@@ -8,6 +8,7 @@
 - Fixed a bug where the “Default Values” Table field setting wasn’t escaping column headings. ([#15552](https://github.com/craftcms/cms/issues/15552))
 - Fixed a bug where Craft couldn’t be installed with existing project config files, if any plugins specified their schema version via `composer.json`. ([#15559](https://github.com/craftcms/cms/issues/15559))
 - Fixed a bug where Money fields’ min, max, and default values weren’t being set to the correct currency. ([#15565](https://github.com/craftcms/cms/issues/15565), [#15566](https://github.com/craftcms/cms/pull/15566))
+- Fixed a bug where Money fields weren’t handling negative values correctly. ([#15565](https://github.com/craftcms/cms/issues/15565), [#15567](https://github.com/craftcms/cms/pull/15567))
 - Fixed a bug where admin tables weren’t displaying disabled statuses. ([#15540](https://github.com/craftcms/cms/pull/15540))
 - Fixed a JavaScript error that occurred when adding a row to an editable table that didn’t allow reordering rows. ([#15543](https://github.com/craftcms/cms/issues/15543))
 - Fixed an error that occurred when editing an element with a Link field previously set to a URL value, if the field no longer allows URLs. ([#15542](https://github.com/craftcms/cms/issues/15542))

src/fields/Money.php

@@ -214,10 +214,10 @@ public function normalizeValue(mixed $value, ?ElementInterface $element): mixed
         }
 
         // Fail-safe if the value is not in the correct format
-        // Try to normalize the value if there are any non-numeric characters
-        if (is_string($value) && !preg_match('/^\d+$/', $value)) {
+        // Try to normalize the value if there are any non-numeric characters (except minus sign at the start)
+        if (is_string($value) && !preg_match('/^(-?)\d+$/', $value)) {
             try {
-                $value = MoneyHelper::normalizeString($value);
+                $value = MoneyHelper::normalizeString($value, new Currency($this->currency));
             } catch (ParserException) {
                 // Catch a parse and return appropriately
                 if (isset($this->defaultValue) && $this->isFresh($element)) {