commit f3a50dc1c58dd1b31e4bee051d47199b0574d1ee
author Rushabh Mehta <rmehta@gmail.com> Thu Jul 07 17:44:05 2016 +0530
committer GitHub <noreply@github.com> Thu Jul 07 17:44:05 2016 +0530
tree f0e1ef7f141e3eaead1fa33db46136eeb862ce26
parent 408d2c5db3ccb71faf698b3478820e62f6869615
parent 0d4a9c225f277b1353da233134d855d06aca5924

Merge pull request #5684 from rohitwaghchaure/v7_ts_issue

[Salary Structure] Make salary_component field mandatory if salary_slip_based_on_timesheet is true

erpnext/hr/doctype/salary_structure/salary_structure.js

diff --git a/erpnext/hr/doctype/salary_structure/salary_structure.js b/erpnext/hr/doctype/salary_structure/salary_structure.js
index aedd5ec..10b7544 100755
--- a/erpnext/hr/doctype/salary_structure/salary_structure.js
+++ b/erpnext/hr/doctype/salary_structure/salary_structure.js
@@ -32,7 +32,8 @@
 	},
 
 	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);
 	}
 })