Merge pull request #28740 from frappe/quick_filters_fix

fix: in case of owner, always include owner in count data
This commit is contained in:
Sumit Bhanushali 2024-12-12 01:21:30 +05:30 committed by GitHub
commit 6d034bb92b
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -71,9 +71,23 @@ def get_group_by_count(doctype: str, current_filters: str, field: str) -> list[d
group_by=f"`tab{doctype}`.{field}",
fields=["count(*) as count", f"`{field}` as name"],
order_by="count desc",
limit=50,
)
if field == "owner":
owner_idx = None
for idx, item in enumerate(data):
if item.name == frappe.session.user:
owner_idx = idx
break
if owner_idx:
data = [data.pop(owner_idx)] + data[0:49]
else:
data = data[0:50]
else:
data = data[0:50]
# Add in title if it's a link field and `show_title_field_in_link` is set
if (field_meta := meta.get_field(field)) and field_meta.fieldtype == "Link":
link_meta = frappe.get_meta(field_meta.options)