code review: use `delete from` to delete records
diff --git a/erpnext/patches/v10_1/drop_old_subscription_records.py b/erpnext/patches/v10_1/drop_old_subscription_records.py
index 1183326..7573f15 100644
--- a/erpnext/patches/v10_1/drop_old_subscription_records.py
+++ b/erpnext/patches/v10_1/drop_old_subscription_records.py
@@ -3,10 +3,4 @@
def execute():
- subscriptions = frappe.db.sql('SELECT name from `tabSubscription`', as_dict=True)
-
- for subscription in subscriptions:
- doc = frappe.get_doc('Subscription', subscription['name'])
- if doc.docstatus == 1:
- doc.cancel()
- frappe.delete_doc('Subscription', subscription['name'], force=True, ignore_permissions=True)
+ frappe.db.sql('DELETE from `tabSubscription`')