Merge pull request #17629 from nabinhait/multi-fixes-33

fix: Multiple small fixes
This commit is contained in:
Saurabh
2019-05-16 15:39:31 +05:30
committed by GitHub
3 changed files with 17 additions and 13 deletions

View File

@@ -1166,6 +1166,8 @@ class SalesInvoice(SellingController):
self.set_missing_values(for_validate = True) self.set_missing_values(for_validate = True)
def validate_inter_company_party(doctype, party, company, inter_company_invoice_reference): def validate_inter_company_party(doctype, party, company, inter_company_invoice_reference):
if not party:
return
if doctype == "Sales Invoice": if doctype == "Sales Invoice":
partytype, ref_partytype, internal = "Customer", "Supplier", "is_internal_customer" partytype, ref_partytype, internal = "Customer", "Supplier", "is_internal_customer"
ref_doc = "Purchase Invoice" ref_doc = "Purchase Invoice"

View File

@@ -172,6 +172,7 @@ frappe.ui.form.on('Loan', {
}, },
mode_of_payment: function (frm) { mode_of_payment: function (frm) {
if (frm.doc.mode_of_payment && frm.doc.company) {
frappe.call({ frappe.call({
method: "erpnext.accounts.doctype.sales_invoice.sales_invoice.get_bank_cash_account", method: "erpnext.accounts.doctype.sales_invoice.sales_invoice.get_bank_cash_account",
args: { args: {
@@ -184,6 +185,7 @@ frappe.ui.form.on('Loan', {
} }
} }
}); });
}
}, },
loan_application: function (frm) { loan_application: function (frm) {

View File

@@ -64,7 +64,7 @@ class Project(Document):
'name': ("not in", self.deleted_task_list) 'name': ("not in", self.deleted_task_list)
}) })
return frappe.get_all("Task", "*", filters, order_by="exp_start_date asc") return frappe.get_all("Task", "*", filters, order_by="exp_start_date asc, status asc")
def validate(self): def validate(self):
self.validate_project_name() self.validate_project_name()