Skip to content

Commit 9787777

Browse files
fix(trends): added validation for period_based_on filter (backport #53690) (#53692)
Co-authored-by: diptanilsaha <diptanil@frappe.io> fix(trends): added validation for `period_based_on` filter (#53690)
1 parent 9805745 commit 9787777

1 file changed

Lines changed: 8 additions & 0 deletions

File tree

erpnext/controllers/trends.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,14 @@ def validate_filters(filters):
5555
if filters.get("based_on") == filters.get("group_by"):
5656
frappe.throw(_("'Based On' and 'Group By' can not be same"))
5757

58+
if filters.get("period_based_on") and filters.period_based_on not in ["bill_date", "posting_date"]:
59+
frappe.throw(
60+
msg=_("{0} can be either {1} or {2}.").format(
61+
frappe.bold("Period based On"), frappe.bold("Posting Date"), frappe.bold("Billing Date")
62+
),
63+
title=_("Invalid Filter"),
64+
)
65+
5866

5967
def get_data(filters, conditions):
6068
data = []

0 commit comments

Comments
 (0)