From 5e789237ed542a5ce0d6a5a513925ee19a26912c Mon Sep 17 00:00:00 2001
From: Hadi Nategh <hn@stylite.de>
Date: Thu, 6 Aug 2015 09:27:10 +0000
Subject: [PATCH] Fix custom rights option messes selected mail acl rights

---
 mail/js/app.js | 24 ++++++++++++++++++------
 1 file changed, 18 insertions(+), 6 deletions(-)

diff --git a/mail/js/app.js b/mail/js/app.js
index a7a7e34836..74c3753646 100644
--- a/mail/js/app.js
+++ b/mail/js/app.js
@@ -3524,13 +3524,25 @@ app.classes.mail = AppJS.extend(
 	acl_common_rights_selector: function(event,widget)
 	{
 		var rowId = widget.id.replace(/[^0-9.]+/g, '');
-		var rights = (widget.get_value() == "custom"?[]:widget.get_value() == "aeiklprstwx")?
-						widget.get_value().replace(/[k,x,t,e]/g,"cd").split(""):widget.get_value().split("");
-
-		for (var i=0;i<this.aclRights.length;i++)
+		var rights = [];
+		
+		switch (widget.get_value())
 		{
-			var rightsWidget = this.et2.getWidgetById(rowId+'[acl_' + this.aclRights[i]+ ']');
-			rightsWidget.set_value((jQuery.inArray(this.aclRights[i],rights) != -1 )?true:false);
+			case 'custom':
+				break;
+			case 'aeiklprstwx':
+				rights = widget.get_value().replace(/[k,x,t,e]/g,"cd").split("");
+				break;
+			default:
+				rights = widget.get_value().split("");
+		}
+		if (rights.length > 0)
+		{
+			for (var i=0;i<this.aclRights.length;i++)
+			{
+				var rightsWidget = this.et2.getWidgetById(rowId+'[acl_' + this.aclRights[i]+ ']');
+				rightsWidget.set_value((jQuery.inArray(this.aclRights[i],rights) != -1 )?true:false);
+			}
 		}
 	},