Merge pull request #2187 from KanchanChauhan/hide-disabled-reports

Disables reports will not show up in list view

frappe/public/js/frappe/list/list_sidebar.js

@@ -69,7 +69,7 @@ frappe.views.ListSidebar = Class.extend({
 
 		var add_reports = function(reports) {
 			$.each(reports, function(name, r) {
-				if(!r.ref_doctype || r.ref_doctype==me.doctype) {
+				if(!r.ref_doctype || r.ref_doctype==me.doctype && !r.disabled) {
 					var report_type = r.report_type==='Report Builder'
 						? 'Report/' + r.ref_doctype : 'query-report';
 					var route = r.route || report_type + '/' + r.name;

frappe/utils/user.py

@@ -205,7 +205,7 @@ class UserPermissions:
 		return d
 
 	def get_all_reports(self):
-		reports =  frappe.db.sql("""select name, report_type, ref_doctype from tabReport
+		reports =  frappe.db.sql("""select name, report_type, ref_doctype, disabled from tabReport
 		    where ref_doctype in ('{0}')""".format("', '".join(self.can_get_report)), as_dict=1)
 
 		return frappe._dict((d.name, d) for d in reports)