Merge pull request #2474 from nabinhait/fix1
Added leave approver name field in leave application
diff --git a/erpnext/hr/doctype/leave_application/leave_application.js b/erpnext/hr/doctype/leave_application/leave_application.js
index ecaac4a..9ba988e 100755
--- a/erpnext/hr/doctype/leave_application/leave_application.js
+++ b/erpnext/hr/doctype/leave_application/leave_application.js
@@ -118,3 +118,18 @@
}
cur_frm.fields_dict.employee.get_query = erpnext.queries.employee;
+
+frappe.ui.form.on("Leave Application", "leave_approver", function(frm) {
+ frappe.call({
+ "method": "frappe.client.get",
+ args: {
+ doctype: "User",
+ name: frm.doc.leave_approver
+ },
+ callback: function (data) {
+ frappe.model.set_value(frm.doctype, frm.docname, "leave_approver_name",
+ data.message.first_name
+ + (data.message.last_name ? (" " + data.message.last_name) : ""))
+ }
+ })
+})
diff --git a/erpnext/hr/doctype/leave_application/leave_application.json b/erpnext/hr/doctype/leave_application/leave_application.json
index 9e2ad53..8beed8b 100644
--- a/erpnext/hr/doctype/leave_application/leave_application.json
+++ b/erpnext/hr/doctype/leave_application/leave_application.json
@@ -25,6 +25,13 @@
"permlevel": 0
},
{
+ "fieldname": "leave_approver_name",
+ "fieldtype": "Read Only",
+ "label": "Leave Approver Name",
+ "permlevel": 0,
+ "precision": ""
+ },
+ {
"fieldname": "leave_type",
"fieldtype": "Link",
"ignore_user_permissions": 1,
@@ -184,7 +191,7 @@
"idx": 1,
"is_submittable": 1,
"max_attachments": 3,
- "modified": "2014-09-09 05:35:31.531651",
+ "modified": "2014-12-09 16:33:29.626849",
"modified_by": "Administrator",
"module": "HR",
"name": "Leave Application",