vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix(query): allow AggregateFunction as well in apply_field_permissions

Browse source 
Without this `fields=[{"COUNT": "name"}]` didn't work, although fields=[{"COUNT": "NAME"}] did.

Signed-off-by: Akhil Narang <me@akhilnarang.dev>
This commit is contained in:
Akhil Narang 2025-11-11 22:12:01 +05:30
parent 1d2a73f659
commit 8e03924356
No known key found for this signature in database
GPG key ID: 9DCC61E211BF645F
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
frappe/database/query.py
View file

@ -973,7 +973,7 @@ class Engine:
elif hasattr(field, "alias") and field.alias and field.name in permitted_fields_set:
allowed_fields.append(field)
elif isinstance(field, PseudoColumnMapper):
elif isinstance(field, AggregateFunction | PseudoColumnMapper):
# Typically functions or complex terms
allowed_fields.append(field)