From 5e65cc9202c970802b642bde01c3aba169165b87 Mon Sep 17 00:00:00 2001 From: "mergify[bot]" <37929162+mergify[bot]@users.noreply.github.com> Date: Fri, 22 Mar 2024 11:58:32 +0530 Subject: [PATCH] perf: remove useless sorting on docstatus (#25571) (#25590) (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 173314c2d6f..3febf625898 100644 --- a/frappe/model/db_query.py +++ b/frappe/model/db_query.py @@ -1079,11 +1079,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: