From a79132e92245b2e83eb7b982b6b27f50a701b282 Mon Sep 17 00:00:00 2001
From: Tim Geyssens <tg@umbraco.com>
Date: Thu, 31 Oct 2013 09:29:06 +0100
Subject: [PATCH] Member groups picker prop editor is working

---
 .../membergrouppicker.controller.js           | 29 +++++++++----------
 1 file changed, 14 insertions(+), 15 deletions(-)

diff --git a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/membergrouppicker/membergrouppicker.controller.js b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/membergrouppicker/membergrouppicker.controller.js
index e1b892379b..4e236bd668 100644
--- a/src/Umbraco.Web.UI.Client/src/views/propertyeditors/membergrouppicker/membergrouppicker.controller.js
+++ b/src/Umbraco.Web.UI.Client/src/views/propertyeditors/membergrouppicker/membergrouppicker.controller.js
@@ -3,11 +3,22 @@
 angular.module('umbraco')
 .controller("Umbraco.PropertyEditors.MemberGroupPickerController",
 	
-	function($scope, dialogService, entityResource, $log, iconHelper){
+	function($scope, dialogService){
 		$scope.renderModel = [];
-		$scope.ids = $scope.model.value.split(',');
+		$scope.ids = [];
 
-		$scope.cfg = {multiPicker: true, entityType: "MemberGroup", type: "membergroup", treeAlias: "memberGroup", filter: ""};
+		
+	    
+	    if ($scope.model.value) {
+	        $scope.ids = $scope.model.value.split(',');
+
+	        $($scope.ids).each(function (i, item) {
+	           
+	            $scope.renderModel.push({ name: item, id: item, icon: 'icon-users' });
+	        });
+	    }
+	    
+	    $scope.cfg = {multiPicker: true, entityType: "MemberGroup", type: "membergroup", treeAlias: "memberGroup", filter: ""};
 		if($scope.model.config){
 			$scope.cfg = angular.extend($scope.cfg, $scope.model.config);
 		}
@@ -49,18 +60,6 @@ angular.module('umbraco')
 	    };
 	   
 
-	    $scope.sortableOptions = {
-	        update: function(e, ui) {
-	        	var r = [];
-	        	angular.forEach($scope.renderModel, function(value, key){
-	        		r.push(value.id);
-	        	});
-
-	        	$scope.ids = r;
-	        	$scope.model.value = trim($scope.ids.join(), ",");
-	        }
-	    };
-
 
 	    $scope.$on("formSubmitting", function (ev, args) {
 			$scope.model.value = trim($scope.ids.join(), ",");