chore: mass trailing whitespace and EOF fixes

erpnext/hr/doctype/appraisal/appraisal.js

@@ -15,7 +15,7 @@ frappe.ui.form.on('Appraisal', {
 			frm.set_value('status', 'Draft');
 		}
 	},
-	
+
 	kra_template: function(frm) {
 		frm.doc.goals = [];
 		erpnext.utils.map_current_doc({

erpnext/hr/doctype/appraisal/test_appraisal.js

@@ -55,4 +55,3 @@ QUnit.test("Test: Expense Claim [HR]", function (assert) {
 		() => done()
 	]);
 });
-

erpnext/hr/doctype/appraisal_goal/appraisal_goal.py

@@ -7,4 +7,4 @@ import frappe
 from frappe.model.document import Document
 
 class AppraisalGoal(Document):
-	pass
+	pass

erpnext/hr/doctype/appraisal_template/appraisal_template_dashboard.py

@@ -9,4 +9,4 @@ def get_data():
                 'items': ['Appraisal']
             },
         ],
-    }
+    }

erpnext/hr/doctype/appraisal_template/test_appraisal_template.js

@@ -27,4 +27,3 @@ QUnit.test("Test: Appraisal Template [HR]", function (assert) {
 		() => done()
 	]);
 });
-

erpnext/hr/doctype/appraisal_template_goal/appraisal_template_goal.py

@@ -7,4 +7,4 @@ import frappe
 from frappe.model.document import Document
 
 class AppraisalTemplateGoal(Document):
-	pass
+	pass

erpnext/hr/doctype/attendance/attendance_calendar.js

@@ -9,4 +9,4 @@ frappe.views.calendar["Attendance"] = {
 		}
 	},
 	get_events_method: "erpnext.hr.doctype.attendance.attendance.get_events"
-};
+};

erpnext/hr/doctype/attendance/test_attendance.js

@@ -36,4 +36,4 @@ QUnit.test("Test: Attendance [HR]", function (assert) {
 			"attendance for Present day is marked"),
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/attendance_request/attendance_request_dashboard.py

@@ -8,4 +8,4 @@ def get_data():
 				'items': ['Attendance']
 			}
 		]
-	}
+	}

erpnext/hr/doctype/branch/branch.py

@@ -7,4 +7,4 @@ import frappe
 from frappe.model.document import Document
 
 class Branch(Document):
-	pass
+	pass

erpnext/hr/doctype/branch/test_branch.js

@@ -20,4 +20,4 @@ QUnit.test("Test: Branch [HR]", function (assert) {
 			'name of branch correctly saved'),
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/branch/test_branch.py

@@ -4,4 +4,4 @@ from __future__ import unicode_literals
 
 
 import frappe
-test_records = frappe.get_test_records('Branch')
+test_records = frappe.get_test_records('Branch')

erpnext/hr/doctype/daily_work_summary/test_daily_work_summary.js

@@ -20,4 +20,4 @@ QUnit.test("test: Daily Work Summary", function (assert) {
 		() => done()
 	]);
 
-});
+});

erpnext/hr/doctype/department/department_tree.js

@@ -25,4 +25,4 @@ frappe.treeview_settings["Department"] = {
 	onload: function(treeview) {
 		treeview.make_tree();
 	}
-};
+};

erpnext/hr/doctype/department/test_department.js

@@ -20,4 +20,4 @@ QUnit.test("Test: Department [HR]", function (assert) {
 			'name of department correctly saved'),
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/department/test_department.py

@@ -21,4 +21,4 @@ def create_department(department_name, parent_department=None):
 
     return doc
 
-test_records = frappe.get_test_records('Department')
+test_records = frappe.get_test_records('Department')

erpnext/hr/doctype/designation/designation.py

@@ -7,4 +7,4 @@ import frappe
 from frappe.model.document import Document
 
 class Designation(Document):
-	pass
+	pass

erpnext/hr/doctype/designation/test_designation.js

@@ -20,4 +20,4 @@ QUnit.test("Test: Designation [HR]", function (assert) {
 			'name of designation correctly saved'),
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/designation/test_designation.py

@@ -17,4 +17,4 @@ def create_designation(**args):
         "description": args.description or "_Test description"
     })
     designation.save()
-    return designation
+    return designation

erpnext/hr/doctype/employee/employee.py

@@ -520,4 +520,4 @@ def has_upload_permission(doc, ptype='read', user=None):
 		user = frappe.session.user
 	if get_doc_permissions(doc, user=user, ptype=ptype).get(ptype):
 		return True
-	return doc.user_id == user
+	return doc.user_id == user

erpnext/hr/doctype/employee/employee_tree.js

@@ -33,4 +33,4 @@ frappe.treeview_settings['Employee'] = {
 			condition: 'frappe.boot.user.can_create.indexOf("Employee") !== -1'
 		}
 	],
-};
+};

erpnext/hr/doctype/employee/test_employee.js

@@ -37,4 +37,4 @@ QUnit.test("Test: Employee [HR]", function (assert) {
 		() => frappe.timeout(10),
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/employee_advance/test_employee_advance.py

@@ -48,4 +48,4 @@ def make_employee_advance(employee_name):
 	doc.insert()
 	doc.submit()
 
-	return doc
+	return doc

erpnext/hr/doctype/employee_attendance_tool/employee_attendance_tool.css

@@ -18,4 +18,4 @@
 
 .checkbox{
 	margin-top: -3px;
-}
+}

erpnext/hr/doctype/employee_attendance_tool/employee_attendance_tool.js

@@ -267,5 +267,3 @@ erpnext.EmployeeSelector = class EmployeeSelector {
 		mark_employee_toolbar.appendTo($(this.wrapper));
 	}
 };
-
-

erpnext/hr/doctype/employee_attendance_tool/test_employee_attendance_tool.js

@@ -58,4 +58,4 @@ QUnit.test("Test: Employee attendance tool [HR]", function (assert) {
 		},
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/employee_checkin/employee_checkin.py

@@ -176,4 +176,3 @@ def time_diff_in_hours(start, end):
 
 def find_index_in_dict(dict_list, key, value):
 	return next((index for (index, d) in enumerate(dict_list) if d[key] == value), None)
-

erpnext/hr/doctype/employee_checkin/test_employee_checkin.py

@@ -42,11 +42,11 @@ class TestEmployeeCheckin(unittest.TestCase):
 		self.assertEqual(logs_count, 4)
 		attendance_count = frappe.db.count('Attendance', {'status':'Present', 'working_hours':8.2,
 			'employee':employee, 'attendance_date':now_date})
-		self.assertEqual(attendance_count, 1)		
+		self.assertEqual(attendance_count, 1)
 
 	def test_calculate_working_hours(self):
 		check_in_out_type = ['Alternating entries as IN and OUT during the same shift',
-			'Strictly based on Log Type in Employee Checkin'] 
+			'Strictly based on Log Type in Employee Checkin']
 		working_hours_calc_type = ['First Check-in and Last Check-out',
 			'Every Valid Check-in and Check-out']
 		logs_type_1 = [

erpnext/hr/doctype/employee_education/employee_education.py

@@ -7,4 +7,4 @@ import frappe
 from frappe.model.document import Document
 
 class EmployeeEducation(Document):
-	pass
+	pass

erpnext/hr/doctype/employee_external_work_history/employee_external_work_history.py

@@ -7,4 +7,4 @@ import frappe
 from frappe.model.document import Document
 
 class EmployeeExternalWorkHistory(Document):
-	pass
+	pass

erpnext/hr/doctype/employee_grade/employee_grade_dashboard.py

@@ -10,4 +10,4 @@ def get_data():
 				'items': ['Employee Onboarding Template', 'Employee Separation Template']
 			}
 		]
-	}
+	}

erpnext/hr/doctype/employee_grievance/employee_grievance.py

@@ -12,4 +12,3 @@ class EmployeeGrievance(Document):
 				bold("Invalid"),
 				bold("Resolved"))
 			)
-

erpnext/hr/doctype/employee_grievance/employee_grievance_list.js

@@ -9,4 +9,4 @@ frappe.listview_settings["Employee Grievance"] = {
 		};
 		return [__(doc.status), colors[doc.status], "status,=," + doc.status];
 	}
-};
+};

erpnext/hr/doctype/employee_grievance/test_employee_grievance.py

@@ -48,4 +48,3 @@ def create_grievance_type():
 	grievance_type.save()
 
 	return grievance_type.name
-

erpnext/hr/doctype/employee_group/test_employee_group.py

@@ -29,4 +29,4 @@ def make_employee_group():
 
 def get_employee_group():
 	employee_group = frappe.db.exists("Employee Group", "_Test Employee Group")
-	return employee_group
+	return employee_group

erpnext/hr/doctype/employee_internal_work_history/employee_internal_work_history.py

@@ -7,4 +7,4 @@ import frappe
 from frappe.model.document import Document
 
 class EmployeeInternalWorkHistory(Document):
-	pass
+	pass

erpnext/hr/doctype/employee_onboarding/employee_onboarding.py

@@ -57,4 +57,3 @@ def make_employee(source_name, target_doc=None):
 				}}
 		}, target_doc, set_missing_values)
 	return doc
-

erpnext/hr/doctype/employee_onboarding/test_employee_onboarding.py

@@ -101,4 +101,4 @@ def create_employee_onboarding():
 	onboarding.insert()
 	onboarding.submit()
 
-	return onboarding
+	return onboarding

erpnext/hr/doctype/employee_onboarding_template/employee_onboarding_template_dashboard.py

@@ -9,4 +9,4 @@ def get_data():
                 'items': ['Employee Onboarding']
             },
         ],
-    }
+    }

erpnext/hr/doctype/employee_referral/employee_referral.py

@@ -70,4 +70,3 @@ def create_additional_salary(doc):
 		additional_salary.ref_docname = doc.name
 
 	return additional_salary
-

erpnext/hr/doctype/employee_referral/employee_referral_dashboard.py

@@ -12,4 +12,4 @@ def get_data():
 			},
 
 		]
-	}
+	}

erpnext/hr/doctype/employee_referral/employee_referral_list.js

@@ -11,4 +11,4 @@ frappe.listview_settings['Employee Referral'] = {
 			return [__(doc.status), "red", "status,=," + doc.status];
 		}
 	},
-};
+};

erpnext/hr/doctype/employee_referral/test_employee_referral.py

@@ -57,4 +57,4 @@ def create_employee_referral():
 	emp_ref.save()
 	emp_ref.submit()
 
-	return emp_ref
+	return emp_ref

erpnext/hr/doctype/employee_separation/test_employee_separation.py

@@ -45,4 +45,4 @@ def create_employee_separation():
 	separation.boarding_status = 'Pending'
 	separation.insert()
 	separation.submit()
-	return separation
+	return separation

erpnext/hr/doctype/employee_separation_template/employee_separation_template_dashboard.py

@@ -9,4 +9,4 @@ def get_data():
                 'items': ['Employee Separation']
             },
         ],
-    }
+    }

erpnext/hr/doctype/employment_type/employment_type.py

@@ -7,4 +7,4 @@ import frappe
 from frappe.model.document import Document
 
 class EmploymentType(Document):
-	pass
+	pass

erpnext/hr/doctype/employment_type/test_employment_type.js

@@ -19,4 +19,4 @@ QUnit.test("Test: Employment type [HR]", function (assert) {
 			'name of employment type correctly saved'),
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/employment_type/test_employment_type.py

@@ -4,4 +4,4 @@ from __future__ import unicode_literals
 
 
 import frappe
-test_records = frappe.get_test_records('Employment Type')
+test_records = frappe.get_test_records('Employment Type')

erpnext/hr/doctype/expense_claim/expense_claim.js

@@ -442,4 +442,4 @@ frappe.ui.form.on("Expense Taxes and Charges", {
 	tax_amount: function(frm, cdt, cdn) {
 		frm.trigger("calculate_total_tax", cdt, cdn);
 	}
-});
+});

erpnext/hr/doctype/expense_claim/expense_claim_dashboard.py

@@ -17,4 +17,4 @@ def get_data():
 				'items': ['Employee Advance']
 			},
 		]
-	}
+	}

erpnext/hr/doctype/expense_claim/test_expense_claim.js

@@ -42,4 +42,3 @@ QUnit.test("Test: Expense Claim [HR]", function (assert) {
 		() => done()
 	]);
 });
-

erpnext/hr/doctype/expense_claim/test_expense_claim.py

@@ -72,7 +72,7 @@ class TestExpenseClaim(unittest.TestCase):
 	def test_expense_claim_gl_entry(self):
 		payable_account = get_payable_account(company_name)
 		taxes = generate_taxes()
-		expense_claim = make_expense_claim(payable_account, 300, 200, company_name, "Travel Expenses - _TC4", 
+		expense_claim = make_expense_claim(payable_account, 300, 200, company_name, "Travel Expenses - _TC4",
 			do_not_submit=True, taxes=taxes)
 		expense_claim.submit()
 

erpnext/hr/doctype/expense_claim_detail/expense_claim_detail.py

@@ -7,4 +7,4 @@ import frappe
 from frappe.model.document import Document
 
 class ExpenseClaimDetail(Document):
-	pass
+	pass

erpnext/hr/doctype/expense_claim_type/expense_claim_type.py

@@ -25,4 +25,4 @@ class ExpenseClaimType(Document):
 			"""Error when Company of Ledger account doesn't match with Company Selected"""
 			if frappe.db.get_value("Account", entry.default_account, "company") != entry.company:
 				frappe.throw(_("Account {0} does not match with Company {1}"
-					).format(entry.default_account, entry.company))
+					).format(entry.default_account, entry.company))

erpnext/hr/doctype/expense_claim_type/test_expense_claim_type.js

@@ -27,4 +27,3 @@ QUnit.test("Test: Expense Claim Type [HR]", function (assert) {
 		() => done()
 	]);
 });
-

erpnext/hr/doctype/holiday/holiday.py

@@ -7,4 +7,4 @@ import frappe
 from frappe.model.document import Document
 
 class Holiday(Document):
-	pass
+	pass

erpnext/hr/doctype/holiday_list/holiday_list_dashboard.py

@@ -18,4 +18,4 @@ def get_data():
 				'items': ['Service Level', 'Service Level Agreement']
 			}
 		]
-	}
+	}

erpnext/hr/doctype/holiday_list/test_holiday_list.js

@@ -39,4 +39,4 @@ QUnit.test("Test: Holiday list [HR]", function (assert) {
 		},
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/hr_settings/hr_settings.js

@@ -5,4 +5,4 @@ frappe.ui.form.on('HR Settings', {
 	restrict_backdated_leave_application: function(frm) {
 		frm.toggle_reqd("role_allowed_to_create_backdated_leave_application", frm.doc.restrict_backdated_leave_application);
 	}
-});
+});

erpnext/hr/doctype/hr_settings/hr_settings.py

@@ -15,4 +15,3 @@ class HRSettings(Document):
 		from erpnext.setup.doctype.naming_series.naming_series import set_by_naming_series
 		set_by_naming_series("Employee", "employee_number",
 			self.get("emp_created_by")=="Naming Series", hide_name_field=True)
-

erpnext/hr/doctype/job_applicant/job_applicant.js

@@ -38,4 +38,4 @@ frappe.ui.form.on("Job Applicant", {
 		});
 
 	}
-});
+});

erpnext/hr/doctype/job_applicant/job_applicant.py

@@ -50,4 +50,3 @@ class JobApplicant(Document):
 
 			if names:
 				frappe.throw(_("Email Address must be unique, already exists for {0}").format(comma_and(names)), frappe.DuplicateEntryError)
-

erpnext/hr/doctype/job_applicant/job_applicant_dashboard.py

@@ -12,4 +12,4 @@ def get_data():
                 'items': ['Job Offer']
             },
         ],
-    }
+    }

erpnext/hr/doctype/job_applicant/test_job_applicant.js

@@ -26,4 +26,3 @@ QUnit.test("Test: Job Opening [HR]", function (assert) {
 		() => done()
 	]);
 });
-

erpnext/hr/doctype/job_offer/test_job_offer.js

@@ -48,4 +48,4 @@ QUnit.test("Test: Job Offer [HR]", function (assert) {
 		() => frappe.timeout(2),
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/job_offer/test_job_offer.py

@@ -79,4 +79,4 @@ def create_staffing_plan(**args):
 	})
 	staffing_plan.insert()
 	staffing_plan.submit()
-	return staffing_plan
+	return staffing_plan

erpnext/hr/doctype/job_opening/job_opening_dashboard.py

@@ -9,4 +9,4 @@ def get_data():
                 'items': ['Job Applicant']
             }
         ],
-    }
+    }

erpnext/hr/doctype/job_opening/templates/job_opening_row.html

@@ -1,16 +1,16 @@
 <div class="my-5">
 	<h3>{{ doc.job_title }}</h3>
 	<p>{{ doc.description }}</p>
-	{%- if doc.publish_salary_range -%} 
+	{%- if doc.publish_salary_range -%}
 	<p><b>{{_("Salary range per month")}}: </b>{{ frappe.format_value(frappe.utils.flt(doc.lower_range), currency=doc.currency) }} - {{ frappe.format_value(frappe.utils.flt(doc.upper_range), currency=doc.currency) }}</p>
 	{% endif %}
 	<div>
 		{%- if doc.job_application_route -%}
-		<a class='btn btn-primary' 
+		<a class='btn btn-primary'
 		href='/{{doc.job_application_route}}?new=1&job_title={{ doc.name }}'>
 		{{ _("Apply Now") }}</a>
 		{% else %}
-		<a class='btn btn-primary' 
+		<a class='btn btn-primary'
 		href='/job_application?new=1&job_title={{ doc.name }}'>
 		{{ _("Apply Now") }}</a>
 		{% endif %}

erpnext/hr/doctype/job_opening/test_job_opening.js

@@ -24,4 +24,3 @@ QUnit.test("Test: Job Opening [HR]", function (assert) {
 		() => done()
 	]);
 });
-

erpnext/hr/doctype/leave_allocation/leave_allocation.js

@@ -100,4 +100,4 @@ frappe.ui.form.on("Leave Allocation", {
 			frm.set_value("total_leaves_allocated", flt(frm.doc.new_leaves_allocated));
 		}
 	}
-});
+});

erpnext/hr/doctype/leave_allocation/leave_allocation_dashboard.py

@@ -17,4 +17,4 @@ def get_data():
 				'items': ['Employee Leave Balance']
 			}
 		]
-    }
+    }

erpnext/hr/doctype/leave_allocation/test_leave_allocation.js

@@ -38,4 +38,4 @@ QUnit.test("Test: Leave allocation [HR]", function (assert) {
 			"total leave calculation is correctly set"),
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/leave_application/leave_application_calendar.js

@@ -17,4 +17,4 @@ frappe.views.calendar["Leave Application"] = {
 		}
 	},
 	get_events_method: "erpnext.hr.doctype.leave_application.leave_application.get_events"
-}
+}

erpnext/hr/doctype/leave_application/leave_application_dashboard.py

@@ -17,4 +17,4 @@ def get_data():
 				'items': ['Employee Leave Balance']
 			}
 		]
-    }
+    }

erpnext/hr/doctype/leave_application/leave_application_email_template.html

@@ -21,5 +21,5 @@
 		<tr>
 			<td>Status</td>
 			<td>{{status}}</td>
-		</tr> 
+		</tr>
 	</table>

erpnext/hr/doctype/leave_application/test_leave_application.js

@@ -39,4 +39,4 @@ QUnit.test("Test: Leave application [HR]", function (assert) {
 		// 	"leave for correct employee is submitted"),
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/leave_block_list/leave_block_list_dashboard.py

@@ -8,4 +8,4 @@ def get_data():
 				'items': ['Department']
 			}
 		]
-	}
+	}

erpnext/hr/doctype/leave_block_list/test_leave_block_list.js

@@ -24,4 +24,4 @@ QUnit.test("Test: Leave block list [HR]", function (assert) {
 			'name of blocked leave list correctly saved'),
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/leave_block_list_allow/leave_block_list_allow.py

@@ -9,4 +9,4 @@ import frappe
 from frappe.model.document import Document
 
 class LeaveBlockListAllow(Document):
-	pass
+	pass

erpnext/hr/doctype/leave_block_list_date/leave_block_list_date.py

@@ -9,4 +9,4 @@ import frappe
 from frappe.model.document import Document
 
 class LeaveBlockListDate(Document):
-	pass
+	pass

erpnext/hr/doctype/leave_control_panel/leave_control_panel.js

@@ -21,4 +21,4 @@ frappe.ui.form.on("Leave Control Panel", {
 			});
 		}
 	}
-});
+});

erpnext/hr/doctype/leave_control_panel/test_leave_control_panel.js

@@ -47,4 +47,4 @@ QUnit.test("Test: Leave control panel [HR]", function (assert) {
 		},
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/leave_encashment/leave_encashment.py

@@ -134,4 +134,4 @@ def create_leave_encashment(leave_allocation):
 			leave_type=allocation.leave_type,
 			encashment_date=allocation.to_date
 		))
-		leave_encashment.insert(ignore_permissions=True)
+		leave_encashment.insert(ignore_permissions=True)

erpnext/hr/doctype/leave_ledger_entry/leave_ledger_entry.py

@@ -185,4 +185,4 @@ def expire_carried_forward_allocation(allocation):
 			from_date=allocation.to_date,
 			to_date=allocation.to_date
 		)
-		create_leave_ledger_entry(allocation, args)
+		create_leave_ledger_entry(allocation, args)

erpnext/hr/doctype/leave_period/leave_period_dashboard.py

@@ -10,4 +10,4 @@ def get_data():
 				'items': ['Leave Allocation']
 			}
 		]
-	}
+	}

erpnext/hr/doctype/leave_period/test_leave_period.py

@@ -27,4 +27,4 @@ def create_leave_period(from_date, to_date, company=None):
 		"to_date": to_date,
 		"is_active": 1
 	}).insert()
-	return leave_period
+	return leave_period

erpnext/hr/doctype/leave_policy/leave_policy_dashboard.py

@@ -10,4 +10,4 @@ def get_data():
 				'items': ['Leave Policy Assignment', 'Leave Allocation']
 			},
 		]
-	}
+	}

erpnext/hr/doctype/leave_policy/test_leave_policy.py

@@ -28,4 +28,4 @@ def create_leave_policy(**args):
 			"leave_type": args.leave_type or "_Test Leave Type",
 			"annual_allocation": args.annual_allocation or 10
 		}]
-	})
+	})

erpnext/hr/doctype/leave_policy_assignment/leave_policy_assignment_dashboard.py

@@ -10,4 +10,4 @@ def get_data():
 				'items': ['Leave Allocation']
 			},
 		]
-	}
+	}

erpnext/hr/doctype/leave_policy_assignment/leave_policy_assignment_list.js

@@ -105,4 +105,4 @@ frappe.listview_settings['Leave Policy Assignment'] = {
 			});
 		}
 	}
-};
+};

erpnext/hr/doctype/leave_policy_assignment/test_leave_policy_assignment.py

@@ -99,5 +99,3 @@ class TestLeavePolicyAssignment(unittest.TestCase):
 	def tearDown(self):
 		for doctype in ["Leave Application", "Leave Allocation", "Leave Policy Assignment", "Leave Ledger Entry"]:
 			frappe.db.sql("delete from `tab{0}`".format(doctype)) #nosec
-
-

erpnext/hr/doctype/leave_type/leave_type_dashboard.py

@@ -11,4 +11,4 @@ def get_data():
 				'items': ['Attendance', 'Leave Encashment']
 			}
 		]
-	}
+	}

erpnext/hr/doctype/leave_type/test_leave_type.js

@@ -19,4 +19,4 @@ QUnit.test("Test: Leave type [HR]", function (assert) {
 			'leave type correctly saved'),
 		() => done()
 	]);
-});
+});

erpnext/hr/doctype/leave_type/test_leave_type.py

@@ -28,4 +28,4 @@ def create_leave_type(**args):
     if leave_type.is_ppl:
         leave_type.fraction_of_daily_salary_per_leave = args.fraction_of_daily_salary_per_leave or 0.5
 
-    return leave_type
+    return leave_type

erpnext/hr/doctype/shift_assignment/shift_assignment_calendar.js

@@ -10,4 +10,4 @@ frappe.views.calendar["Shift Assignment"] = {
 		"allDay": "allDay",
 	},
 	get_events_method: "erpnext.hr.doctype.shift_assignment.shift_assignment.get_events"
-}
+}

erpnext/hr/doctype/shift_assignment/test_shift_assignment.py

@@ -77,4 +77,4 @@ class TestShiftAssignment(unittest.TestCase):
 				"status": 'Active'
 			})
 
-			self.assertRaises(frappe.ValidationError, shift_assignment_3.save)
+			self.assertRaises(frappe.ValidationError, shift_assignment_3.save)

erpnext/hr/doctype/shift_request/shift_request.py

@@ -94,4 +94,4 @@ class ShiftRequest(Document):
 		msg = _("Employee {0} has already applied for {1} between {2} and {3} : ").format(self.employee,
 			d['shift_type'], formatdate(d['from_date']), formatdate(d['to_date'])) \
 			+ """ <b><a href="/app/Form/Shift Request/{0}">{0}</a></b>""".format(d["name"])
-		frappe.throw(msg, OverlapError)
+		frappe.throw(msg, OverlapError)

erpnext/hr/doctype/shift_request/shift_request_dashboard.py

@@ -9,4 +9,4 @@ def get_data():
                 'items': ['Shift Assignment']
             },
         ],
-    }
+    }

erpnext/hr/doctype/shift_request/test_shift_request.py

@@ -106,4 +106,4 @@ def make_shift_request(approver, do_not_submit=0):
 		return shift_request
 
 	shift_request.submit()
-	return shift_request
+	return shift_request

erpnext/hr/doctype/staffing_plan/staffing_plan_dashboard.py

@@ -9,4 +9,4 @@ def get_data():
                 'items': ['Job Opening']
             }
         ],
-    }
+    }