From 97b1b7f8423d9c485c68a1b1cb60da17d8e84dfc Mon Sep 17 00:00:00 2001 From: "mergify[bot]" <37929162+mergify[bot]@users.noreply.github.com> Date: Tue, 4 Jun 2024 12:21:16 +0530 Subject: [PATCH] fix: filtered out data having none values in date field (backport #41457) (#41765) * fix: filtered out data having none values in date field (cherry picked from commit 3bd455ac856c5a073e7ec44bb7e855c64e7789cb) * style: code optimization (cherry picked from commit a36b7fb95a4b1e42f0a0da3e73d9e380f171d5b5) --------- Co-authored-by: Khushi Rawat <142375893+khushi8112@users.noreply.github.com> --- .../report/fixed_asset_register/fixed_asset_register.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/erpnext/assets/report/fixed_asset_register/fixed_asset_register.py b/erpnext/assets/report/fixed_asset_register/fixed_asset_register.py index 4ff04c80434..e86ca15d00a 100644 --- a/erpnext/assets/report/fixed_asset_register/fixed_asset_register.py +++ b/erpnext/assets/report/fixed_asset_register/fixed_asset_register.py @@ -159,8 +159,9 @@ def prepare_chart_data(data, filters): if filters.filter_based_on not in ("Date Range", "Fiscal Year"): filters_filter_based_on = "Date Range" date_field = "purchase_date" - filters_from_date = min(data, key=lambda a: a.get(date_field)).get(date_field) - filters_to_date = max(data, key=lambda a: a.get(date_field)).get(date_field) + filtered_data = [d for d in data if not d.get(date_field)] + filters_from_date = min(filtered_data, key=lambda a: a.get(date_field)).get(date_field) + filters_to_date = max(filtered_data, key=lambda a: a.get(date_field)).get(date_field) else: filters_filter_based_on = filters.filter_based_on date_field = frappe.scrub(filters.date_based_on)