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

fix: remove unnecessary conditional

Browse source 
both code branches exactly same, so why bother
This commit is contained in:
Ankush Menat 2023-09-20 12:21:40 +05:30
parent d7f4518de3
commit 9a36a09bee
1 changed files with 0 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
frappe/public/js/frappe/ui/group_by/group_by.html
View file

@ -10,22 +10,12 @@
<option value="" disabled selected>{{ __("Select Group By...") }}</option>
{% for (var parent_doctype in group_by_conditions) { %}
{% for (var val in group_by_conditions[parent_doctype]) { %}
{% if (parent_doctype !== doctype) { %}
<option
data-doctype="{{parent_doctype}}"
value="{{group_by_conditions[parent_doctype][val].fieldname}}"
>
{{ __(group_by_conditions[parent_doctype][val].label) }}
({{ __(parent_doctype) }})
</option>
{% } else { %}
<option
data-doctype="{{parent_doctype}}"
value="{{group_by_conditions[parent_doctype][val].fieldname}}"
>
{{ __(group_by_conditions[parent_doctype][val].label || group_by_conditions[parent_doctype][val].fieldname) }}
</option>
{% } %}
{% } %}
{% } %}
</select>