From a4b8a362ae72e73e56d9a39774b83f63ecbebd7e Mon Sep 17 00:00:00 2001
From: Rushabh Mehta <rmehta@gmail.com>
Date: Mon, 23 Jun 2014 12:31:06 +0530
Subject: [PATCH] user permissions, show buttons only for system manager

---
 .../page/user_permissions/user_permissions.js | 26 ++++++++++---------
 1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/frappe/core/page/user_permissions/user_permissions.js b/frappe/core/page/user_permissions/user_permissions.js
index 38d6c3618c..7d1392736b 100644
--- a/frappe/core/page/user_permissions/user_permissions.js
+++ b/frappe/core/page/user_permissions/user_permissions.js
@@ -73,19 +73,21 @@ frappe.UserPermissions = Class.extend({
 					options: "[Select]"
 				});
 
-				me.download = me.wrapper.appframe.add_field({
-					fieldname: "download",
-					label: __("Download"),
-					fieldtype: "Button",
-					icon: "icon-download"
-				});
+				if(user_roles.indexOf("System Manager")!==-1) {
+					me.download = me.wrapper.appframe.add_field({
+						fieldname: "download",
+						label: __("Download"),
+						fieldtype: "Button",
+						icon: "icon-download"
+					});
 
-				me.upload = me.wrapper.appframe.add_field({
-					fieldname: "upload",
-					label: __("Upload"),
-					fieldtype: "Button",
-					icon: "icon-upload"
-				});
+					me.upload = me.wrapper.appframe.add_field({
+						fieldname: "upload",
+						label: __("Upload"),
+						fieldtype: "Button",
+						icon: "icon-upload"
+					});
+				}
 
 				// bind change event
 				$.each(me.filters, function(k, f) {