Patch fixed for legacy migration
This commit is contained in:
parent
4ed0653c7f
commit
d03087d3ab
@ -51,6 +51,9 @@ class Budget(Document):
|
|||||||
|
|
||||||
def validate_expense_against_budget(args):
|
def validate_expense_against_budget(args):
|
||||||
args = frappe._dict(args)
|
args = frappe._dict(args)
|
||||||
|
if not args.cost_center:
|
||||||
|
return
|
||||||
|
|
||||||
if frappe.db.get_value("Account", {"name": args.account, "root_type": "Expense"}):
|
if frappe.db.get_value("Account", {"name": args.account, "root_type": "Expense"}):
|
||||||
cc_lft, cc_rgt = frappe.db.get_value("Cost Center", args.cost_center, ["lft", "rgt"])
|
cc_lft, cc_rgt = frappe.db.get_value("Cost Center", args.cost_center, ["lft", "rgt"])
|
||||||
|
|
||||||
|
|||||||
@ -27,6 +27,7 @@ def execute():
|
|||||||
time_sheet.update_cost()
|
time_sheet.update_cost()
|
||||||
time_sheet.calculate_total_amounts()
|
time_sheet.calculate_total_amounts()
|
||||||
time_sheet.flags.ignore_validate = True
|
time_sheet.flags.ignore_validate = True
|
||||||
|
time_sheet.flags.ignore_links = True
|
||||||
time_sheet.save(ignore_permissions=True)
|
time_sheet.save(ignore_permissions=True)
|
||||||
|
|
||||||
# To ignore validate_mandatory_fields function
|
# To ignore validate_mandatory_fields function
|
||||||
|
|||||||
@ -17,10 +17,10 @@ def execute():
|
|||||||
add_timesheet_detail(time_sheet, args)
|
add_timesheet_detail(time_sheet, args)
|
||||||
|
|
||||||
time_sheet.docstatus = tlb.docstatus
|
time_sheet.docstatus = tlb.docstatus
|
||||||
|
time_sheet.flags.ignore_links = True
|
||||||
time_sheet.save(ignore_permissions=True)
|
time_sheet.save(ignore_permissions=True)
|
||||||
|
|
||||||
def get_timesheet_data(data):
|
def get_timesheet_data(data):
|
||||||
time_log = frappe.get_all('Time Log', fields=["*"],
|
time_log = frappe.get_all('Time Log', fields=["*"], filters = {'name': data.time_log})
|
||||||
filters = {'name': data.time_log})[0]
|
if time_log:
|
||||||
|
return get_timelog_data(time_log[0])
|
||||||
return get_timelog_data(time_log)
|
|
||||||
@ -5,6 +5,8 @@ def execute():
|
|||||||
frappe.reload_doc('accounts', 'doctype', 'sales_invoice_payment')
|
frappe.reload_doc('accounts', 'doctype', 'sales_invoice_payment')
|
||||||
for time_sheet in frappe.db.sql(""" select sales_invoice, name, total_billing_amount from `tabTimesheet`
|
for time_sheet in frappe.db.sql(""" select sales_invoice, name, total_billing_amount from `tabTimesheet`
|
||||||
where sales_invoice is not null and docstatus < 2""", as_dict=True):
|
where sales_invoice is not null and docstatus < 2""", as_dict=True):
|
||||||
|
if not frappe.db.exists('Sales Invoice', time_sheet.sales_invoice):
|
||||||
|
continue
|
||||||
si_doc = frappe.get_doc('Sales Invoice', time_sheet.sales_invoice)
|
si_doc = frappe.get_doc('Sales Invoice', time_sheet.sales_invoice)
|
||||||
ts = si_doc.append('timesheets',{})
|
ts = si_doc.append('timesheets',{})
|
||||||
ts.time_sheet = time_sheet.name
|
ts.time_sheet = time_sheet.name
|
||||||
|
|||||||
@ -7,6 +7,8 @@ def execute():
|
|||||||
frappe.reload_doctype('Stock Settings')
|
frappe.reload_doctype('Stock Settings')
|
||||||
stock_settings = frappe.get_doc('Stock Settings', 'Stock Settings')
|
stock_settings = frappe.get_doc('Stock Settings', 'Stock Settings')
|
||||||
stock_settings.show_barcode_field = cint(frappe.db.get_value("Features Setup", None, "fs_item_barcode"))
|
stock_settings.show_barcode_field = cint(frappe.db.get_value("Features Setup", None, "fs_item_barcode"))
|
||||||
|
if not frappe.db.exists("UOM", stock_settings.stock_uom):
|
||||||
|
stock_settings.stock_uom = None
|
||||||
stock_settings.save()
|
stock_settings.save()
|
||||||
|
|
||||||
create_compact_item_print_custom_field()
|
create_compact_item_print_custom_field()
|
||||||
|
|||||||
@ -16,4 +16,5 @@ def execute():
|
|||||||
"company": frappe.db.get_value("Account", expense_claim_type.default_account, "company"),
|
"company": frappe.db.get_value("Account", expense_claim_type.default_account, "company"),
|
||||||
"default_account": expense_claim_type.default_account,
|
"default_account": expense_claim_type.default_account,
|
||||||
})
|
})
|
||||||
|
doc.flags.ignore_mandatory = True
|
||||||
doc.save(ignore_permissions=True)
|
doc.save(ignore_permissions=True)
|
||||||
@ -71,7 +71,7 @@ class Warehouse(NestedSet):
|
|||||||
"freeze_account": "No"
|
"freeze_account": "No"
|
||||||
})
|
})
|
||||||
ac_doc.flags.ignore_permissions = True
|
ac_doc.flags.ignore_permissions = True
|
||||||
|
ac_doc.flags.ignore_mandatory = True
|
||||||
try:
|
try:
|
||||||
ac_doc.insert()
|
ac_doc.insert()
|
||||||
msgprint(_("Account head {0} created").format(ac_doc.name))
|
msgprint(_("Account head {0} created").format(ac_doc.name))
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user