diff --git a/erpnext/stock/doctype/quality_inspection/quality_inspection.py b/erpnext/stock/doctype/quality_inspection/quality_inspection.py index c7a315d2298..faef473a7fa 100644 --- a/erpnext/stock/doctype/quality_inspection/quality_inspection.py +++ b/erpnext/stock/doctype/quality_inspection/quality_inspection.py @@ -283,9 +283,11 @@ class QualityInspection(Document): def min_max_criteria_passed(self, reading): """Determine whether all readings fall in the acceptable range.""" + has_reading = False for i in range(1, 11): reading_value = reading.get("reading_" + str(i)) if reading_value is not None and reading_value.strip(): + has_reading = True result = ( flt(reading.get("min_value")) <= parse_float(reading_value) @@ -293,7 +295,7 @@ class QualityInspection(Document): ) if not result: return False - return True + return has_reading def set_status_based_on_acceptance_formula(self, reading): if not reading.acceptance_formula: diff --git a/erpnext/stock/doctype/quality_inspection/test_quality_inspection.py b/erpnext/stock/doctype/quality_inspection/test_quality_inspection.py index f59d7c3216b..b0b67d8c8ea 100644 --- a/erpnext/stock/doctype/quality_inspection/test_quality_inspection.py +++ b/erpnext/stock/doctype/quality_inspection/test_quality_inspection.py @@ -290,7 +290,7 @@ def create_quality_inspection(**args): if not args.readings: create_quality_inspection_parameter("Size") - readings = {"specification": "Size", "min_value": 0, "max_value": 10} + readings = {"specification": "Size", "min_value": 0, "max_value": 10, "reading_1": "5"} if args.status == "Rejected": readings["reading_1"] = "12" # status is auto set in child on save else: