From 5e5853eb836aa5800b4734c3b68afcd469ec0813 Mon Sep 17 00:00:00 2001 From: "mergify[bot]" <37929162+mergify[bot]@users.noreply.github.com> Date: Fri, 22 Mar 2024 11:58:35 +0530 Subject: [PATCH] perf: remove useless sorting on docstatus (#25571) (#25591) (cherry picked from commit a12fc118f4b4ad85e28e7c1a866614d0e50f2c89) Co-authored-by: Ankush Menat --- frappe/model/db_query.py | 5 ----- 1 file changed, 5 deletions(-) diff --git a/frappe/model/db_query.py b/frappe/model/db_query.py index 26e1a33714a..d6c8e290d2c 100644 --- a/frappe/model/db_query.py +++ b/frappe/model/db_query.py @@ -1097,11 +1097,6 @@ def set_order_by(self, args): f"`tab{self.doctype}`.`{sort_field or 'modified'}` {sort_order or 'desc'}" ) - # draft docs always on top - if hasattr(self.doctype_meta, "is_submittable") and self.doctype_meta.is_submittable: - if self.order_by: - args.order_by = f"`tab{self.doctype}`.docstatus asc, {args.order_by}" - def validate_order_by_and_group_by(self, parameters: str): """Check order by, group by so that atleast one column is selected and does not have subquery""" if not parameters: