From 0d4a9c225f277b1353da233134d855d06aca5924 Mon Sep 17 00:00:00 2001 From: Rohit Waghchaure Date: Thu, 7 Jul 2016 17:04:54 +0530 Subject: [PATCH] [Salary Structure] Make salary_component field mandatory if salary_slip_based_on_timesheet is true --- erpnext/hr/doctype/salary_structure/salary_structure.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/erpnext/hr/doctype/salary_structure/salary_structure.js b/erpnext/hr/doctype/salary_structure/salary_structure.js index aedd5ec555..10b7544c42 100755 --- a/erpnext/hr/doctype/salary_structure/salary_structure.js +++ b/erpnext/hr/doctype/salary_structure/salary_structure.js @@ -32,7 +32,8 @@ frappe.ui.form.on('Salary Structure', { }, toggle_fields: function(frm) { - frm.toggle_display('time_sheet_earning_detail', cint(frm.doc.salary_slip_based_on_timesheet)==1); + frm.toggle_display('time_sheet_earning_detail', frm.doc.salary_slip_based_on_timesheet); + frm.toggle_reqd('salary_component', frm.doc.salary_slip_based_on_timesheet); } })