mirror of
https://github.com/frappe/erpnext.git
synced 2026-02-13 09:43:49 +00:00
fix: prevent self in "Reports To" dropdown (UI-level check)
Ensures employee cannot select themselves in the "Reports To" field via UI.
This complements server-side validation by improving UX.
(cherry picked from commit 608d38a172)
This commit is contained in:
@@ -11,7 +11,13 @@ erpnext.setup.EmployeeController = class EmployeeController extends frappe.ui.fo
|
|||||||
};
|
};
|
||||||
};
|
};
|
||||||
this.frm.fields_dict.reports_to.get_query = function (doc, cdt, cdn) {
|
this.frm.fields_dict.reports_to.get_query = function (doc, cdt, cdn) {
|
||||||
return { query: "erpnext.controllers.queries.employee_query" };
|
// return { query: "erpnext.controllers.queries.employee_query" };
|
||||||
|
return {
|
||||||
|
filters: [
|
||||||
|
["status", "=", "Active"], // only active employees
|
||||||
|
["name", "!=", doc.name] // exclude self
|
||||||
|
]
|
||||||
|
};
|
||||||
};
|
};
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user