From 01912910c81a332010bc6a3d51ab3442ee0ff94d Mon Sep 17 00:00:00 2001 From: Ejaaz Khan Date: Mon, 17 Mar 2025 12:15:04 +0530 Subject: [PATCH] fix: give prority to user setting on sorting --- frappe/public/js/frappe/ui/sort_selector.js | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/frappe/public/js/frappe/ui/sort_selector.js b/frappe/public/js/frappe/ui/sort_selector.js index 33cdb28f04..0ce84c1d5f 100644 --- a/frappe/public/js/frappe/ui/sort_selector.js +++ b/frappe/public/js/frappe/ui/sort_selector.js @@ -103,13 +103,15 @@ frappe.ui.SortSelector = class SortSelector { var { meta_sort_field, meta_sort_order } = this.get_meta_sort_field(); - if (meta_sort_field) { - this.args.sort_by = meta_sort_field; - this.args.sort_order = meta_sort_order; - } else { - // default - this.args.sort_by = "creation"; - this.args.sort_order = "desc"; + if (!this.args.sort_by) { + if (meta_sort_field) { + this.args.sort_by = meta_sort_field; + this.args.sort_order = meta_sort_order; + } else { + // default + this.args.sort_by = "creation"; + this.args.sort_order = "desc"; + } } if (!this.args.sort_by_label) {