diff --git a/frappe/core/doctype/user/user.js b/frappe/core/doctype/user/user.js
index c5ccde182c..40b5cd2efd 100644
--- a/frappe/core/doctype/user/user.js
+++ b/frappe/core/doctype/user/user.js
@@ -115,7 +115,7 @@ frappe.ModuleEditor = Class.extend({
 			var module = $(this).attr('data-module');
 			if($(this).prop("checked")) {
 				// remove from block_modules
-				me.frm.doc.block_modules = $.map(me.frm.doc.block_modules || [], function(d) { d.module != module });
+				me.frm.doc.block_modules = $.map(me.frm.doc.block_modules || [], function(d) { if(d.module != module){ return d } });
 			} else {
 				me.frm.add_child("block_modules", {"module": module});
 			}