From 79d0bc76a32e4f665d3a245ad4ee4ea68bd5e74d Mon Sep 17 00:00:00 2001
From: brantje <brantje@gmail.com>
Date: Sun, 2 Oct 2016 22:09:45 +0200
Subject: [PATCH] Fix add multiple users to shared credential

---
 js/app/controllers/share.js | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

diff --git a/js/app/controllers/share.js b/js/app/controllers/share.js
index fa7f9a14..21f3c782 100644
--- a/js/app/controllers/share.js
+++ b/js/app/controllers/share.js
@@ -100,24 +100,17 @@ angular.module('passmanApp')
             //@TODO Improve this so we can add, edit and remove users and permissions.
 			$scope.inputSharedWith = [];
 			if(shareWith.length > 0) {
-			    shareWithMainLoop:
 				for (var i = 0; i < shareWith.length; i++) {
 			        // Avoid adding twice the same user.
-			        for (var x = 0; x < $scope.share_settings.credentialSharedWithUserAndGroup.length; x++){
-			            if (shareWith[x].uid == $scope.share_settings.credentialSharedWithUserAndGroup[i].userId){
-			                continue shareWithMainLoop;
-                        }
-                    }
 					var obj = {
 						userId: shareWith[i].uid,
 						displayName: shareWith[i].text,
 						type: shareWith[i].type,
 						accessLevel: selectedAccessLevel
 					};
-					// if (obj.type == 'group') obj.users = shareWith[i].users;
-					$scope.share_settings.credentialSharedWithUserAndGroup.push(
-						obj
-					)
+					if($scope.share_settings.credentialSharedWithUserAndGroup.indexOf(obj) === -1){
+						$scope.share_settings.credentialSharedWithUserAndGroup.push(obj)
+					}
 				}
 			}
 		};