Shiloh/brotherton-erpnext
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge patch and delete fields from subscription (#14941)

Browse Source 
* Delete subscription records after migrating all records to auto-repeat

* delete old fields from subscription
This commit is contained in:
Saurabh 2018-07-19 14:33:31 +05:30 committed by Nabin Hait
parent 139d165258
commit a8e24cba46
3 changed files with 29 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
erpnext/patches.txt
View File

@ -524,7 +524,6 @@ erpnext.patches.v11_0.uom_conversion_data #30-06-2018
erpnext.patches.v10_0.taxes_issue_with_pos erpnext.patches.v10_0.taxes_issue_with_pos
erpnext.patches.v11_0.update_account_type_in_party_type erpnext.patches.v11_0.update_account_type_in_party_type
erpnext.patches.v10_1.transfer_subscription_to_auto_repeat erpnext.patches.v10_1.transfer_subscription_to_auto_repeat
erpnext.patches.v10_1.drop_old_subscription_records
erpnext.patches.v11_0.update_brand_in_item_price erpnext.patches.v11_0.update_brand_in_item_price
erpnext.patches.v11_0.create_default_success_action erpnext.patches.v11_0.create_default_success_action
erpnext.patches.v11_0.add_healthcare_service_unit_tree_root erpnext.patches.v11_0.add_healthcare_service_unit_tree_root

6
erpnext/patches/v10_1/drop_old_subscription_records.py
View File

@ -1,6 +0,0 @@
from __future__ import unicode_literals
import frappe
def execute():
frappe.db.sql('DELETE from `tabSubscription`')

42
erpnext/patches/v10_1/transfer_subscription_to_auto_repeat.py
View File

@ -4,23 +4,19 @@ from frappe.model.utils.rename_field import rename_field
def execute(): def execute():
to_rename = ['Purchase Order', 'Journal Entry', 'Sales Invoice', 'Payment Entry',
'Delivery Note', 'Purchase Invoice', 'Quotation', 'Sales Order',
'Purchase Receipt', 'Supplier Quotation']
frappe.reload_doc('accounts', 'doctype', 'sales_invoice')
frappe.reload_doc('accounts', 'doctype', 'purchase_invoice')
frappe.reload_doc('accounts', 'doctype', 'payment_entry')
frappe.reload_doc('accounts', 'doctype', 'journal_entry')
frappe.reload_doc('buying', 'doctype', 'purchase_order')
frappe.reload_doc('buying', 'doctype', 'supplier_quotation')
frappe.reload_doc('desk', 'doctype', 'auto_repeat') frappe.reload_doc('desk', 'doctype', 'auto_repeat')
frappe.reload_doc('selling', 'doctype', 'quotation')
frappe.reload_doc('selling', 'doctype', 'sales_order')
frappe.reload_doc('stock', 'doctype', 'purchase_receipt')
frappe.reload_doc('stock', 'doctype', 'delivery_note')
for doctype in to_rename: doctypes_to_rename = {
'accounts': ['Journal Entry', 'Payment Entry', 'Purchase Invoice', 'Sales Invoice'],
'buying': ['Purchase Order', 'Supplier Quotation'],
'selling': ['Quotation', 'Sales Order'],
'stock': ['Delivery Note', 'Purchase Receipt']
}
for module, doctypes in doctypes_to_rename.items():
for doctype in doctypes:
frappe.reload_doc(module, 'doctype', frappe.scrub(doctype))
if frappe.db.has_column(doctype, 'subscription'): if frappe.db.has_column(doctype, 'subscription'):
rename_field(doctype, 'subscription', 'auto_repeat') rename_field(doctype, 'subscription', 'auto_repeat')
@ -30,3 +26,19 @@ def execute():
doc['doctype'] = 'Auto Repeat' doc['doctype'] = 'Auto Repeat'
auto_repeat = frappe.get_doc(doc) auto_repeat = frappe.get_doc(doc)
auto_repeat.db_insert() auto_repeat.db_insert()
frappe.db.sql('delete from `tabSubscription`')
frappe.db.commit()
drop_columns_from_subscription()
def drop_columns_from_subscription():
fields_to_drop = {'Subscription': []}
for field in ['naming_series', 'reference_doctype', 'reference_document', 'start_date',
'end_date', 'submit_on_creation', 'disabled', 'frequency', 'repeat_on_day',
'next_schedule_date', 'notify_by_email', 'subject', 'recipients', 'print_format',
'message', 'status', 'amended_from']:
if field in frappe.db.get_table_columns("Subscription"):
fields_to_drop['Subscription'].append(field)
frappe.model.delete_fields(fields_to_drop, delete=1)