diff --git a/frappe/model/db_query.py b/frappe/model/db_query.py index 5f65ce50e4..81a9b982ad 100644 --- a/frappe/model/db_query.py +++ b/frappe/model/db_query.py @@ -878,7 +878,7 @@ from {tables} fallback = "''" elif f.fieldname == "name": - value = f.value + value = f.value if f.value is not None else "" fallback = "''" elif ( diff --git a/frappe/tests/test_db_query.py b/frappe/tests/test_db_query.py index 7b43201407..e4d68e9f80 100644 --- a/frappe/tests/test_db_query.py +++ b/frappe/tests/test_db_query.py @@ -1201,6 +1201,7 @@ class TestDBQuery(IntegrationTestCase): self.assertIn("''", query) self.assertNotIn("\\'", query) self.assertNotIn("ifnull", query) + self.assertFalse(frappe.get_all("DocField", {"name": None})) def test_ifnull_fallback_types(self): query = frappe.get_all("DocField", {"fieldname": ("!=", None)}, run=0)