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

Merge pull request #1130 from nabinhait/develop

Browse source 
[fix] update reports while field renaming
This commit is contained in:
Nabin Hait 2015-05-27 16:19:25 +05:30
commit a1c15c8a89
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
frappe/model/__init__.py
View file

@ -133,7 +133,7 @@ def update_reports(doctype, old_fieldname, new_fieldname):
# update filters
new_filters = []
for f in report_dict.get("filters"):
if f[0] == doctype and f[1] == old_fieldname:
if f and len(f) > 1 and f[0] == doctype and f[1] == old_fieldname:
new_filters.append([doctype, new_fieldname, f[2], f[3]])
report_dict["updated"] = True
else:
@ -142,7 +142,7 @@ def update_reports(doctype, old_fieldname, new_fieldname):
# update columns
new_columns = []
for c in report_dict.get("columns"):
if c[0] == old_fieldname and c[1] == doctype:
if c and len(c) > 1 and c[0] == old_fieldname and c[1] == doctype:
new_columns.append([new_fieldname, doctype])
report_dict["updated"] = True
else: