Shiloh/brotherton-erpnext
1
1
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix codacy

Browse Source
This commit is contained in:
Shreya 2018-06-20 16:09:16 +05:30
parent 5c6ade4bd8
commit 3fd21d4273
5 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
erpnext/hr/doctype/employee_onboarding/employee_onboarding.js
View File

@ -40,7 +40,7 @@ frappe.ui.form.on('Employee Onboarding', {
},
callback: function(r) {
if (r.message) {
frm.set_value('boarding_status', r.message)
frm.set_value('boarding_status', r.message);
}
refresh_field("boarding_status");
}

1
erpnext/hr/doctype/employee_onboarding/employee_onboarding.py
View File

@ -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

6
erpnext/hr/doctype/employee_onboarding/employee_onboarding_list.js
View File

@ -2,6 +2,6 @@ frappe.listview_settings['Employee Onboarding'] = {
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];
}
};

2
erpnext/hr/doctype/employee_separation/employee_separation.js
View File

@ -31,7 +31,7 @@ frappe.ui.form.on('Employee Separation', {
},
callback: function(r) {
if (r.message) {
frm.set_value('boarding_status', r.message)
frm.set_value('boarding_status', r.message);
}
refresh_field("boarding_status");
}

6
erpnext/hr/doctype/employee_separation/employee_separation_list.js
View File

@ -2,6 +2,6 @@ frappe.listview_settings['Employee Separation'] = {
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];
}
};