fix(patch): set_task_status (#21130)

* fix(patch): set_task_status

* remove unnecessary loop, use tabs

Co-authored-by: Nabin Hait <nabinhait@gmail.com>
This commit is contained in:
Raffael Meyer
2020-04-08 05:54:34 +02:00
committed by GitHub
parent f93dc8dd88
commit 385a92d3f2

View File

@@ -1,16 +1,15 @@
import frappe import frappe
def execute(): def execute():
frappe.reload_doctype('Task') frappe.reload_doctype('Task')
# add "Completed" if customized # add "Completed" if customized
for doctype in ('Task'): property_setter_name = frappe.db.exists('Property Setter', dict(doc_type='Task', field_name = 'status', property = 'options'))
property_setter_name = frappe.db.exists('Property Setter', dict(doc_type = doctype, field_name = 'status', property = 'options')) if property_setter_name:
if property_setter_name: property_setter = frappe.get_doc('Property Setter', property_setter_name)
property_setter = frappe.get_doc('Property Setter', property_setter_name) if not "Completed" in property_setter.value:
if not "Completed" in property_setter.value: property_setter.value = property_setter.value + '\nCompleted'
property_setter.value = property_setter.value + '\nCompleted' property_setter.save()
property_setter.save()
# renamed default status to Completed as status "Closed" is ambiguous # renamed default status to Completed as status "Closed" is ambiguous
frappe.db.sql('update tabTask set status = "Completed" where status = "Closed"') frappe.db.sql('update tabTask set status = "Completed" where status = "Closed"')