From 3cd0ad4c98a9f8d6b132fa104ed648888c1ac43b Mon Sep 17 00:00:00 2001 From: Jamsheer Date: Wed, 16 May 2018 15:17:49 +0530 Subject: [PATCH] Codacy fixes --- .../employee_benefit_application.js | 10 +++++----- erpnext/hr/doctype/payroll_period/payroll_period.py | 1 - 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/erpnext/hr/doctype/employee_benefit_application/employee_benefit_application.js b/erpnext/hr/doctype/employee_benefit_application/employee_benefit_application.js index 65619b3777..624a3d5eb0 100644 --- a/erpnext/hr/doctype/employee_benefit_application/employee_benefit_application.js +++ b/erpnext/hr/doctype/employee_benefit_application/employee_benefit_application.js @@ -10,8 +10,8 @@ frappe.ui.form.on('Employee Benefit Application', { is_flexible_benefit: true, disabled: false } - } - }) + }; + }); }, employee: function(frm) { frappe.call({ @@ -31,8 +31,8 @@ frappe.ui.form.on('Employee Benefit Application', { frappe.ui.form.on("Employee Benefit Application Detail",{ amount: function(frm, cdt, cdn) { calculate_all(frm.doc, cdt, cdn); - }, -}) + } +}); var calculate_all = function(doc, dt, dn) { var tbl = doc.employee_benefits || []; @@ -50,4 +50,4 @@ var calculate_all = function(doc, dt, dn) { doc.remainig_benefits = doc.max_benefits - total_amount; doc.pro_rata_dispensed_amount = pro_rata_dispensed_amount; refresh_many(['pro_rata_dispensed_amount', 'total_amount','remainig_benefits']); -} +}; diff --git a/erpnext/hr/doctype/payroll_period/payroll_period.py b/erpnext/hr/doctype/payroll_period/payroll_period.py index c95c7bdbaf..a818bc4276 100644 --- a/erpnext/hr/doctype/payroll_period/payroll_period.py +++ b/erpnext/hr/doctype/payroll_period/payroll_period.py @@ -26,5 +26,4 @@ def get_payroll_period_days(start_date, end_date, company): }) if len(payroll_period_dates) > 0: - print payroll_period_dates[0][0], payroll_period_dates[0][1] return date_diff(getdate(payroll_period_dates[0][1]), getdate(payroll_period_dates[0][0])) + 1