Merge pull request #1586 from frappe/version-14-hotfix

chore: release v14

Author: ruchamahabal

hrms/hr/doctype/vehicle_log/vehicle_log.js

@@ -2,6 +2,15 @@
 // For license information, please see license.txt
 
 frappe.ui.form.on("Vehicle Log", {
+	setup: function (frm) {
+		frm.set_query("employee", function() {
+			return {
+				filters: {
+					status: "Active",
+				},
+			};
+		});
+	},
 	refresh: function(frm) {
 		if(frm.doc.docstatus == 1) {
 			frm.add_custom_button(__('Expense Claim'), function() {

hrms/hr/report/monthly_attendance_sheet/monthly_attendance_sheet.js

@@ -83,6 +83,10 @@ frappe.query_reports["Monthly Attendance Sheet"] = {
 		const summarized_view = frappe.query_report.get_filter_value('summarized_view');
 		const group_by = frappe.query_report.get_filter_value('group_by');
 
+		if (group_by && column.colIndex === 1) {
+			value = "<strong>" + value + "</strong>";
+		}
+
 		if (!summarized_view) {
 			if ((group_by && column.colIndex > 3) || (!group_by && column.colIndex > 2)) {
 				if (value == 'P' || value == 'WFH')

hrms/hr/report/monthly_attendance_sheet/monthly_attendance_sheet.py

@@ -72,12 +72,19 @@ def get_columns(filters: Filters) -> List[Dict]:
 	columns = []
 
 	if filters.group_by:
+		options_mapping = {
+			"Branch": "Branch",
+			"Grade": "Employee Grade",
+			"Department": "Department",
+			"Designation": "Designation",
+		}
+		options = options_mapping.get(filters.group_by)
 		columns.append(
 			{
 				"label": _(filters.group_by),
 				"fieldname": frappe.scrub(filters.group_by),
 				"fieldtype": "Link",
-				"options": "Branch",
+				"options": options,
 				"width": 120,
 			}
 		)
@@ -191,7 +198,7 @@ def get_data(filters: Filters, attendance_map: Dict) -> List[Dict]:
 			records = get_rows(employee_details[value], filters, holiday_map, attendance_map)
 
 			if records:
-				data.append({group_by_column: frappe.bold(value)})
+				data.append({group_by_column: value})
 				data.extend(records)
 	else:
 		data = get_rows(employee_details, filters, holiday_map, attendance_map)

hrms/payroll/report/bank_remittance/bank_remittance.py

@@ -22,7 +22,12 @@ def execute(filters=None):
 			"hidden": 1,
 			"width": 200,
 		},
-		{"label": _("Payment Date"), "fieldtype": "Data", "fieldname": "payment_date", "width": 100},
+		{
+			"label": _("Payment Date"),
+			"fieldtype": "Data",
+			"fieldname": "payment_date",
+			"width": 100,
+		},
 		{
 			"label": _("Employee Name"),
 			"fieldtype": "Link",
@@ -146,12 +151,10 @@ def get_emp_bank_ifsc_code(salary_slips):
 	emp_names = [d.employee for d in salary_slips]
 	ifsc_codes = get_all("Employee", [("name", "IN", emp_names)], ["ifsc_code", "name"])
 
-	ifsc_codes_map = {}
-	for code in ifsc_codes:
-		ifsc_codes_map[code.name] = code
+	ifsc_codes_map = {code.name: code.ifsc_code for code in ifsc_codes}
 
 	for slip in salary_slips:
-		slip["ifsc_code"] = ifsc_codes_map[code.name]["ifsc_code"]
+		slip["ifsc_code"] = ifsc_codes_map[slip.employee]
 
 	return salary_slips