From bc0bc6677e7fa6312a1d773cd9b5777b23104508 Mon Sep 17 00:00:00 2001 From: Himanshu Warekar Date: Fri, 28 Jun 2019 19:41:29 +0530 Subject: [PATCH] fix: use db sql to delete property setter --- erpnext/patches.txt | 1 + erpnext/patches/v12_0/delete_priority_property_setter.py | 9 +++++++++ erpnext/patches/v12_0/set_priority_for_support.py | 2 -- 3 files changed, 10 insertions(+), 2 deletions(-) create mode 100644 erpnext/patches/v12_0/delete_priority_property_setter.py diff --git a/erpnext/patches.txt b/erpnext/patches.txt index fa51638605..36a31cfadc 100644 --- a/erpnext/patches.txt +++ b/erpnext/patches.txt @@ -607,3 +607,4 @@ execute:frappe.delete_doc_if_exists("Page", "support-analytics") erpnext.patches.v12_0.make_item_manufacturer erpnext.patches.v12_0.set_quotation_status erpnext.patches.v12_0.set_priority_for_support +erpnext.patches.v12_0.delete_priority_property_setter diff --git a/erpnext/patches/v12_0/delete_priority_property_setter.py b/erpnext/patches/v12_0/delete_priority_property_setter.py new file mode 100644 index 0000000000..5927267543 --- /dev/null +++ b/erpnext/patches/v12_0/delete_priority_property_setter.py @@ -0,0 +1,9 @@ +import frappe + +def execute(): + frappe.db.sql(""" + DELETE FROM `tabProperty Setter` + WHERE `tabProperty Setter`.doc_type='Issue' + AND `tabProperty Setter`.field_name='priority' + AND `tabProperty Setter`.property='options' + """) \ No newline at end of file diff --git a/erpnext/patches/v12_0/set_priority_for_support.py b/erpnext/patches/v12_0/set_priority_for_support.py index 4eae0654e8..cc290396f8 100644 --- a/erpnext/patches/v12_0/set_priority_for_support.py +++ b/erpnext/patches/v12_0/set_priority_for_support.py @@ -18,8 +18,6 @@ def set_issue_priority(): "name": priority }).insert(ignore_permissions=True) - frappe.delete_doc_if_exists("Property Setter", {"field_name": "priority", "property": "options"}) - def set_priority_for_issue(): # Sets priority for Issues as Select field is changed to Link field. issue_priority = frappe.get_list("Issue", fields=["name", "priority"])