Fix codacy
diff --git a/erpnext/hr/doctype/employee_onboarding/employee_onboarding.js b/erpnext/hr/doctype/employee_onboarding/employee_onboarding.js
index ffba940..0861588 100644
--- a/erpnext/hr/doctype/employee_onboarding/employee_onboarding.js
+++ b/erpnext/hr/doctype/employee_onboarding/employee_onboarding.js
@@ -40,7 +40,7 @@
},
callback: function(r) {
if (r.message) {
- frm.set_value('boarding_status', r.message)
+ frm.set_value('boarding_status', r.message);
}
refresh_field("boarding_status");
}
diff --git a/erpnext/hr/doctype/employee_onboarding/employee_onboarding.py b/erpnext/hr/doctype/employee_onboarding/employee_onboarding.py
index 29124fc..9262a06 100644
--- a/erpnext/hr/doctype/employee_onboarding/employee_onboarding.py
+++ b/erpnext/hr/doctype/employee_onboarding/employee_onboarding.py
@@ -5,7 +5,6 @@
from __future__ import unicode_literals
import frappe
from frappe import _
-from frappe.utils import flt
from erpnext.hr.utils import EmployeeBoardingController
from frappe.model.mapper import get_mapped_doc
diff --git a/erpnext/hr/doctype/employee_onboarding/employee_onboarding_list.js b/erpnext/hr/doctype/employee_onboarding/employee_onboarding_list.js
index c14d7a5..a33619b 100644
--- a/erpnext/hr/doctype/employee_onboarding/employee_onboarding_list.js
+++ b/erpnext/hr/doctype/employee_onboarding/employee_onboarding_list.js
@@ -2,6 +2,6 @@
add_fields: ["boarding_status", "employee_name", "date_of_joining", "department"],
filters:[["boarding_status","=", "Pending"]],
get_indicator: function(doc) {
- return [__(doc.boarding_status), frappe.utils.guess_colour(doc.boarding_status), "status,=," + doc.boarding_status];
- }
- };
+ return [__(doc.boarding_status), frappe.utils.guess_colour(doc.boarding_status), "status,=," + doc.boarding_status];
+ }
+};
diff --git a/erpnext/hr/doctype/employee_separation/employee_separation.js b/erpnext/hr/doctype/employee_separation/employee_separation.js
index 6ef73eb..9a75c16 100644
--- a/erpnext/hr/doctype/employee_separation/employee_separation.js
+++ b/erpnext/hr/doctype/employee_separation/employee_separation.js
@@ -31,7 +31,7 @@
},
callback: function(r) {
if (r.message) {
- frm.set_value('boarding_status', r.message)
+ frm.set_value('boarding_status', r.message);
}
refresh_field("boarding_status");
}
diff --git a/erpnext/hr/doctype/employee_separation/employee_separation_list.js b/erpnext/hr/doctype/employee_separation/employee_separation_list.js
index d23802c..11487cc 100644
--- a/erpnext/hr/doctype/employee_separation/employee_separation_list.js
+++ b/erpnext/hr/doctype/employee_separation/employee_separation_list.js
@@ -2,6 +2,6 @@
add_fields: ["boarding_status", "employee_name", "date_of_joining", "department"],
filters:[["boarding_status","=", "Pending"]],
get_indicator: function(doc) {
- return [__(doc.boarding_status), frappe.utils.guess_colour(doc.boarding_status), "status,=," + doc.boarding_status];
- }
- };
+ return [__(doc.boarding_status), frappe.utils.guess_colour(doc.boarding_status), "status,=," + doc.boarding_status];
+ }
+};