Merge pull request #6651 from shreyasp/del-company-txn
Delete Communications while Deleting Company Transactions
diff --git a/erpnext/setup/doctype/company/delete_company_transactions.py b/erpnext/setup/doctype/company/delete_company_transactions.py
index 8f058e8..eb5c043 100644
--- a/erpnext/setup/doctype/company/delete_company_transactions.py
+++ b/erpnext/setup/doctype/company/delete_company_transactions.py
@@ -14,7 +14,7 @@
doc = frappe.get_doc("Company", company_name)
if frappe.session.user != doc.owner:
- frappe.throw(_("Transactions can only be deleted by the creator of the Company"),
+ frappe.throw(_("Transactions can only be deleted by the creator of the Company"),
frappe.PermissionError)
delete_bins(company_name)
@@ -37,6 +37,9 @@
if not meta.issingle:
if not meta.istable:
+ # delete communication
+ delete_communications(doctype, company_name, company_fieldname)
+
# delete children
for df in meta.get_table_fields():
frappe.db.sql("""delete from `tab{0}` where parent in
@@ -64,7 +67,6 @@
frappe.db.sql("""update tabSeries set current = %s
where name=%s""", (last, prefix))
-
def delete_bins(company_name):
frappe.db.sql("""delete from tabBin where warehouse in
(select name from tabWarehouse where company=%s)""", company_name)
@@ -76,3 +78,9 @@
where lead=%s and (customer='' or customer is null) and (supplier='' or supplier is null)""", lead.name)
frappe.db.sql("""update `tabAddress` set lead=null, lead_name=null where lead=%s""", lead.name)
+
+def delete_communications(doctype, company_name, company_fieldname):
+ frappe.db.sql("""
+ DELETE FROM `tabCommunication` WHERE reference_doctype = %s AND
+ EXISTS (SELECT name FROM `tab{0}` WHERE {1} = %s AND `tabCommunication`.reference_name = name)
+ """.format(doctype, company_fieldname), (doctype, company_name))