style: change type comparison for boolean expression

This commit is contained in:
Mangesh-Khairnar
2019-04-09 12:26:52 +05:30
parent 16c4fd6733
commit b70b048fa2

View File

@@ -33,7 +33,7 @@ frappe.ui.form.on("Sales Order", {
}) })
}, },
refresh: function(frm) { refresh: function(frm) {
if(frm.doc.docstatus == 1 && frm.doc.status != 'Closed' if(frm.doc.docstatus === 1 && frm.doc.status !== 'Closed'
&& flt(frm.doc.per_delivered) < 100 && flt(frm.doc.per_billed) < 100) { && flt(frm.doc.per_delivered) < 100 && flt(frm.doc.per_billed) < 100) {
frm.add_custom_button(__('Update Items'), () => { frm.add_custom_button(__('Update Items'), () => {
erpnext.utils.update_child_items({ erpnext.utils.update_child_items({
@@ -135,8 +135,8 @@ erpnext.selling.SalesOrderController = erpnext.selling.SellingController.extend(
}, __("Status")); }, __("Status"));
} }
} }
if(doc.status != 'Closed') { if(doc.status !== 'Closed') {
if(doc.status != 'On Hold') { if(doc.status !== 'On Hold') {
for (var i in this.frm.doc.items) { for (var i in this.frm.doc.items) {
var item = this.frm.doc.items[i]; var item = this.frm.doc.items[i];
if(item.delivered_by_supplier === 1 || item.supplier){ if(item.delivered_by_supplier === 1 || item.supplier){