diff --git a/frappe/public/js/frappe/ui/filters/filters.js b/frappe/public/js/frappe/ui/filters/filters.js
index 87caef9941..f39ed4907f 100644
--- a/frappe/public/js/frappe/ui/filters/filters.js
+++ b/frappe/public/js/frappe/ui/filters/filters.js
@@ -211,10 +211,9 @@ frappe.ui.Filter = Class.extend({
 
 		var df = copy_dict(me.fieldselect.fields_by_name[doctype][fieldname]);
 
-		// all fields shown in filters
-		if(df.hidden) {
-			df.hidden = 0;
-		}
+		// filter field shouldn't be read only or hidden
+		df.read_only = 0;
+		df.hidden = 0;
 
 		this.set_fieldtype(df, fieldtype);
 
@@ -236,6 +235,7 @@ frappe.ui.Filter = Class.extend({
 		}
 
 		var field_area = me.$w.find('.filter_field').empty().get(0);
+
 		var f = frappe.ui.form.make_control({
 			df: df,
 			parent: field_area,