fix vaults mover

controller/admincontroller.php

@@ -98,6 +98,7 @@ class AdminController extends ApiController {
 
 				$c = $credential->jsonSerialize();
 				$c['user_id'] = $destination_account;
+				$c['icon'] = json_encode($c['icon']);
 				$this->credentialService->updateCredential($c, true);
 			}
 			$vault->setUserId($destination_account);

css/admin.css

@@ -2,6 +2,10 @@
   padding: 5px; }
 #passwordSharingSettings #mover input[type="text"] {
   width: 350px; }
+#passwordSharingSettings #mover .account_mover_selector {
+  width: 350px; }
+  #passwordSharingSettings #mover .account_mover_selector .select2-choice {
+    height: 34px; }
 #passwordSharingSettings #requests-table {
   width: 100%; }
 

css/admin.css.map

@@ -1,6 +1,6 @@
 {
 "version": 3,
-"mappings": "AAGM,wCAAE;EACA,OAAO,EAAE,GAAG;AAGhB,kDAAkB;EAChB,KAAK,EAAE,KAAK;AAGhB,wCAAe;EACb,KAAK,EAAE,IAAI",
+"mappings": "AAGM,wCAAE;EACA,OAAO,EAAE,GAAG;AAGhB,kDAAkB;EAChB,KAAK,EAAE,KAAK;AAEd,uDAAwB;EACtB,KAAK,EAAE,KAAK;EACZ,uEAAgB;IACd,MAAM,EAAE,IAAI;AAIlB,wCAAe;EACb,KAAK,EAAE,IAAI",
 "sources": ["../sass/admin.scss"],
 "names": [],
 "file": "admin.css"

js/settings-admin.js

@@ -137,16 +137,38 @@ $(document).ready(function () {
 		'source_account': '',
 		'destination_account': ''
 	};
-	$(".username-autocomplete").autocomplete({
+	$('.account_mover_selector').select2({
-		source: OC.generateUrl('apps/passman/admin/search'),
+		ajax: {
-		minLength: 1,
+			url: OC.generateUrl('apps/passman/admin/search'),
-		select: function (event, ui) {
+			dataType: 'json',
-			accountMover[$(this).attr('id')] = ui.item.value;
+			delay: 50,
-		}
+			data: function (param) {
+				return {
+					term: param
+				};
+			},
+			results: function (data) {
+				var res = [];
+				for (var i = 0; i < data.length; i++) {
+					res.push({
+						id: i,
+						text: data[i].value
+					});
+				}
+				return {
+					results: res
+				};
+			},
+			cache: true
+		},
+		placeholder: 'Search for a user',
+		minimumInputLength: 1
 	});
 
 	$('#move_credentials').click(function () {
 		var self = this;
+		accountMover.source_account = $('#s2id_source_account a .select2-chosen').html();
+		accountMover.destination_account = $('#s2id_destination_account a .select2-chosen').html();
 		$('#moveStatus').hide();
 		$(self).attr('disabled', 'disabled');
 		$(self).html('<i class="fa fa-spinner fa-spin"></i> ' + OC.L10N.translate('passman', 'Moving') + '...');

sass/admin.scss

@@ -8,8 +8,14 @@
     input[type="text"]{
       width: 350px;
     }
+    .account_mover_selector {
+      width: 350px;
+      .select2-choice {
+        height: 34px;
+      }
+    }
   }
   #requests-table{
     width: 100%;
   }
-}
+}

templates/admin.php

@@ -123,11 +123,11 @@ style('passman', 'vendor/font-awesome/font-awesome.min');
 			<table class="table">
 				<tr>
 					<td><?php p($l->t('Source account')); ?> </td>
-					<td><input class="username-autocomplete" type="text" id="source_account" name="source_account"></td>
+					<td><input type="hidden" class="form-control account_mover_selector" id="source_account"></td>
 				</tr>
 				<tr>
 					<td><?php p($l->t('Destination account')); ?> </td>
-					<td><input class="username-autocomplete" type="text" id="destination_account" name="destination_account"></td>
+					<td><input type="hidden" class="form-control account_mover_selector" id="destination_account"></td>
 				</tr>
 			</table>
 			<button class="success" id="move_credentials">Move</button>