From 636e9379d8d02dd34f19df289a0d04248f1ff51a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Petr=20St=C5=99=C3=ADbn=C3=BD?= <petr@stribny.name>
Date: Fri, 20 Feb 2026 11:50:01 +0100
Subject: [PATCH] Fixed importing rows with 'period, comma' separated Number
 fields (#4789)

---
 .../contrib/database/fields/field_types.py    | 14 +----
 .../file_import/test_file_import_tasks.py     | 55 +++++++++++++++++++
 ...786_number_field_row_import_separator.json |  9 +++
 3 files changed, 65 insertions(+), 13 deletions(-)
 create mode 100644 changelog/entries/unreleased/bug/4786_number_field_row_import_separator.json

diff --git a/backend/src/baserow/contrib/database/fields/field_types.py b/backend/src/baserow/contrib/database/fields/field_types.py
index 8a2aa70f52..161b4a74ba 100755
--- a/backend/src/baserow/contrib/database/fields/field_types.py
+++ b/backend/src/baserow/contrib/database/fields/field_types.py
@@ -620,20 +620,8 @@ def prepare_value_for_db(self, instance: NumberField, value):
             return value
 
         if isinstance(value, str):
-            if instance.number_prefix is not None:
-                value = value.lstrip(instance.number_prefix)
-            if instance.number_suffix is not None:
-                value = value.rstrip(instance.number_suffix)
-
-            thousand_sep, decimal_sep = get_thousand_and_decimal_separator(
-                instance.number_separator
-            )
-
-            value = value.replace(thousand_sep, "").replace(decimal_sep, ".").strip()
-
-            if value in ["", "NaN"]:
+            if value == "":
                 return None
-
         try:
             value = Decimal(value)
         except (InvalidOperation, ValueError, TypeError):
diff --git a/backend/tests/baserow/contrib/database/file_import/test_file_import_tasks.py b/backend/tests/baserow/contrib/database/file_import/test_file_import_tasks.py
index f9fe9ad95a..f818731794 100644
--- a/backend/tests/baserow/contrib/database/file_import/test_file_import_tasks.py
+++ b/backend/tests/baserow/contrib/database/file_import/test_file_import_tasks.py
@@ -1,5 +1,6 @@
 import json
 from datetime import date, datetime, timedelta, timezone
+from decimal import Decimal
 from typing import NamedTuple
 
 from django.conf import settings
@@ -1714,3 +1715,57 @@ def test_run_file_import_task_with_upsert_and_none_skipped_fields(
 
     assert getattr(charlie, age_field.db_column) == 28
     assert getattr(charlie, email_field.db_column) == "charlie@example.com"
+
+
+@pytest.mark.django_db(transaction=True)
+def test_file_import_task_number_field(data_fixture, patch_filefield_storage):
+    user = data_fixture.create_user()
+    database = data_fixture.create_database_application(user=user)
+    table = data_fixture.create_database_table(database=database, user=user)
+    primary_field = data_fixture.create_text_field(
+        table=table, order=1, name="Item", primary=True
+    )
+    number_field = data_fixture.create_number_field(
+        table=table,
+        order=2,
+        name="Number",
+        number_negative=True,
+        number_decimal_places=2,
+        # number_separator should not affect the import as
+        # the data coming in are processed in the frontend
+        number_separator="PERIOD_COMMA",
+        # number_prefix should not affect the import
+        number_prefix="$",
+        # number_suffix should not affect the import
+        number_suffix="X",
+    )
+    data = [
+        ["Item", "Number"],
+        ["A", "127.30"],
+        ["B", "-30.00"],
+        ["C", "500"],
+        ["D", "1 000 000.1"],  # will be skipped due to spaces
+        ["E", "-3,33"],  # will be skipped due to comma
+        ["F", "$-3.33"],  # will be skipped due to prefix
+        ["G", "-3.33X"],  # will be skipped due to suffix
+        ["I", ""],
+        ["J", "NaN"],  # will be skipped, not a number
+    ]
+    job = data_fixture.create_file_import_job(
+        data={"data": data},
+        table=table,
+        user=user,
+        first_row_header=True,
+    )
+    run_async_job(job.id)
+    job.refresh_from_db()
+    assert job.state == JOB_FINISHED
+
+    model = table.get_model()
+    assert model.objects.count() == 4
+
+    rows = model.objects.all()
+    assert getattr(rows[0], number_field.db_column) == Decimal("127.30")
+    assert getattr(rows[1], number_field.db_column) == Decimal("-30.00")
+    assert getattr(rows[2], number_field.db_column) == Decimal("500")
+    assert getattr(rows[3], number_field.db_column) is None
diff --git a/changelog/entries/unreleased/bug/4786_number_field_row_import_separator.json b/changelog/entries/unreleased/bug/4786_number_field_row_import_separator.json
new file mode 100644
index 0000000000..d98af24ece
--- /dev/null
+++ b/changelog/entries/unreleased/bug/4786_number_field_row_import_separator.json
@@ -0,0 +1,9 @@
+{
+    "type": "bug",
+    "message": "Fixed Importing rows with 'period, comma' separated Number fields doesn't import numbers correctly.",
+    "issue_origin": "github",
+    "issue_number": 4786,
+    "domain": "database",
+    "bullet_points": [],
+    "created_at": "2026-02-17"
+}