From ccdcb7dfcc29eb68d1cad38f3426e6fb575528eb Mon Sep 17 00:00:00 2001 From: s-aga-r Date: Mon, 4 Dec 2023 11:58:44 +0530 Subject: [PATCH 1/3] fix(ux): stock-item filter for Item Code field --- erpnext/stock/report/stock_analytics/stock_analytics.js | 1 + 1 file changed, 1 insertion(+) diff --git a/erpnext/stock/report/stock_analytics/stock_analytics.js b/erpnext/stock/report/stock_analytics/stock_analytics.js index ea7bf5688e..e033fd9a9d 100644 --- a/erpnext/stock/report/stock_analytics/stock_analytics.js +++ b/erpnext/stock/report/stock_analytics/stock_analytics.js @@ -17,6 +17,7 @@ frappe.query_reports["Stock Analytics"] = { fieldtype: "Link", options:"Item", default: "", + get_query: () => ({filters: { 'is_stock_item': 1 }}), }, { fieldname: "value_quantity", From 01aadbef85e2570399f054027e57d6bb01af9433 Mon Sep 17 00:00:00 2001 From: s-aga-r Date: Mon, 4 Dec 2023 12:00:36 +0530 Subject: [PATCH 2/3] fix: don't show non-stock items in Stock Analytics report --- erpnext/stock/report/stock_analytics/stock_analytics.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/erpnext/stock/report/stock_analytics/stock_analytics.py b/erpnext/stock/report/stock_analytics/stock_analytics.py index 6c5b58c6e4..ab48181c48 100644 --- a/erpnext/stock/report/stock_analytics/stock_analytics.py +++ b/erpnext/stock/report/stock_analytics/stock_analytics.py @@ -270,7 +270,7 @@ def get_items(filters): if item_code := filters.get("item_code"): return [item_code] else: - item_filters = {} + item_filters = {"is_stock_item": 1} if item_group := filters.get("item_group"): children = get_descendants_of("Item Group", item_group, ignore_permissions=True) item_filters["item_group"] = ("in", children + [item_group]) From 15fff84bb5cc78507aa24771fde3e9a9fbb926bc Mon Sep 17 00:00:00 2001 From: s-aga-r Date: Mon, 4 Dec 2023 12:05:27 +0530 Subject: [PATCH 3/3] fix: `linter` --- .../accounts/report/accounts_receivable/accounts_receivable.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/erpnext/accounts/report/accounts_receivable/accounts_receivable.py b/erpnext/accounts/report/accounts_receivable/accounts_receivable.py index 7948e5f465..eaf9f421e1 100644 --- a/erpnext/accounts/report/accounts_receivable/accounts_receivable.py +++ b/erpnext/accounts/report/accounts_receivable/accounts_receivable.py @@ -1087,7 +1087,7 @@ class ReceivablePayableReport(object): ) if self.filters.show_remarks: - self.add_column(label=_("Remarks"), fieldname="remarks", fieldtype="Text", width=200), + self.add_column(label=_("Remarks"), fieldname="remarks", fieldtype="Text", width=200) def add_column(self, label, fieldname=None, fieldtype="Currency", options=None, width=120): if not fieldname: