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

fix: check for numeric arg first

Browse source
This commit is contained in:
Sagar Vora 2026-03-26 17:25:13 +05:30
parent 0d415afdd5
commit 17f9ca9819
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
frappe/database/query.py
View file

@ -2420,14 +2420,15 @@ class SQLFunctionParser:
).format(arg), ).format(arg),
frappe.ValidationError, frappe.ValidationError,
) )
elif self._is_valid_field_name(arg):
self._check_function_field_permission(arg)
return self.engine.table[arg]
# Check if it's a numeric string like "1" (for COUNT(1), etc.) # Check if it's a numeric string like "1" (for COUNT(1), etc.)
elif arg.isdigit(): elif arg.isdigit():
return int(arg) return int(arg)
elif self._is_valid_field_name(arg):
self._check_function_field_permission(arg)
return self.engine.table[arg]
else: else:
frappe.throw( frappe.throw(
_( _(