From 6915cb61aa458e34594543e5d51900e8406396dd Mon Sep 17 00:00:00 2001
From: Michael <michael@crossingpaths.be>
Date: Sat, 28 Oct 2023 11:12:10 +0200
Subject: [PATCH 1/2] Remove current user from list Display "no user" label in
 case there is no one to select DIsplay "you ned admin rihts" on non-admin
 users

---
 .../wwwroot/lang/da-DK.xml                    |  7 ++--
 .../wwwroot/lang/en-US.xml                    |  7 ++--
 .../wwwroot/user.controller.js                | 13 +++++---
 .../wwwroot/user.html                         | 32 ++++++++++++++-----
 4 files changed, 40 insertions(+), 19 deletions(-)

diff --git a/src/Our.Umbraco.Impersonator/wwwroot/lang/da-DK.xml b/src/Our.Umbraco.Impersonator/wwwroot/lang/da-DK.xml
index c75d43d..ea3d946 100644
--- a/src/Our.Umbraco.Impersonator/wwwroot/lang/da-DK.xml
+++ b/src/Our.Umbraco.Impersonator/wwwroot/lang/da-DK.xml
@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="utf-8" standalone="yes"?>
+<?xml version="1.0" encoding="utf-8" standalone="yes"?>
 <language alias="da" intName="Danish" localName="dansk" lcid="6" culture="da-DK">
 	<creator>
 		<name>Søren Kottal</name>
@@ -15,6 +15,7 @@
 		<key alias="notSignedIn">Du skal være logget ind i Umbraco for at kunne imitere en anden bruger</key>
 		<key alias="userNotFound">Den valgte bruger kunne ikke findes</key>
 		<key alias="invalidUserId">Det valgte bruger ID er ikke gyldigt</key>
-		<key alias="notAdministrator">Du skal have administratorrettigheder for at kunne imitere en anden bruger</key>
+		<key alias="notAdministrator">Du skal have brugeradministratorrettigheder for at kunne imitere en anden bruger</key>
+		<kay alias="noUser">Der er ingen aktive brugere at imiterer.</kay>
 	</area>
-</language>
\ No newline at end of file
+</language>
diff --git a/src/Our.Umbraco.Impersonator/wwwroot/lang/en-US.xml b/src/Our.Umbraco.Impersonator/wwwroot/lang/en-US.xml
index 070e241..8a3de72 100644
--- a/src/Our.Umbraco.Impersonator/wwwroot/lang/en-US.xml
+++ b/src/Our.Umbraco.Impersonator/wwwroot/lang/en-US.xml
@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="utf-8" standalone="yes"?>
+<?xml version="1.0" encoding="utf-8" standalone="yes"?>
 <language alias="en_us" intName="English (US)" localName="English (US)" lcid="" culture="en-US">
 	<creator>
 		<name>Søren Kottal</name>
@@ -15,6 +15,7 @@
 		<key alias="notSignedIn">You need to be signed in to Umbraco, to be able to impersonate another user.</key>
 		<key alias="userNotFound">The selected user could not be found.</key>
 		<key alias="invalidUserId">The selected user id is invalid.</key>
-		<key alias="notAdministrator">You need administrator rights, to be able to impersonate another user.</key>
+		<key alias="notAdministrator">You need users administrator rights, to be able to impersonate another user.</key>
+		<key alias="noUser">There are no active users to impersonate.</key>
 	</area>
-</language>
\ No newline at end of file
+</language>
diff --git a/src/Our.Umbraco.Impersonator/wwwroot/user.controller.js b/src/Our.Umbraco.Impersonator/wwwroot/user.controller.js
index 221f617..6da182f 100644
--- a/src/Our.Umbraco.Impersonator/wwwroot/user.controller.js
+++ b/src/Our.Umbraco.Impersonator/wwwroot/user.controller.js
@@ -1,4 +1,4 @@
-(function () {
+(function () {
   "use strict";
 
   function ImpersonatorUserController(
@@ -12,7 +12,6 @@
     var vm = this;
 
     vm.isImpersonating = false;
-    vm.isLoading = false;
     vm.users = [];
     vm.loadingUsers = true;
     vm.impersonateUserButtonState = "init";
@@ -25,12 +24,12 @@
         // Get users
         usersResource
           .getPagedResults({
-              pageSize: 2147483647,
-              userStates: ["Active"],
+            pageSize: 2147483647,
+            userStates: ["Active"],
           })
           .then(
             function (data) {
-              vm.users = data.items;
+              vm.users = data.items.filter(function (u) { return u.id != user.id }); // remove current user from impersonation list
               vm.loadingUsers = false;
             },
             function (error) {
@@ -38,6 +37,10 @@
             }
           );
       }
+      else {
+        // No users to load, we're done
+        vm.loadingUsers = false;
+      }
     });
 
     vm.impersonateUser = function () {
diff --git a/src/Our.Umbraco.Impersonator/wwwroot/user.html b/src/Our.Umbraco.Impersonator/wwwroot/user.html
index dc5c048..3209fac 100644
--- a/src/Our.Umbraco.Impersonator/wwwroot/user.html
+++ b/src/Our.Umbraco.Impersonator/wwwroot/user.html
@@ -1,10 +1,9 @@
-<div ng-controller="Impersonator.User.Controller as vm">
-  <umb-box ng-if="vm.isImpersonating && !vm.isLoading">
+<div ng-controller="Impersonator.User.Controller as vm">
+  <umb-box ng-if="vm.isImpersonating && !vm.loadingUsers">
     <umb-box-content>
       <div class="mb2">
         <localize key="impersonator_impersonatingDescription"
-          >You are impersonating the current user.</localize
-        >
+          >You are impersonating the current user.</localize>
       </div>
       <div class="text-right">
         <umb-button
@@ -13,14 +12,14 @@
           label-key="impersonator_endImpersonation"
           button-style="primary"
           state="vm.endImpersonationButtonState"
-        ></umb-button>
-      </div>
+          ></umb-button>
+        </div>
     </umb-box-content>
   </umb-box>
   <umb-box
-    ng-if="vm.canImpersonateUsers && !vm.isImpersonating && !vm.isLoading"
+    ng-if="vm.canImpersonateUsers && !vm.isImpersonating && !vm.loadingUsers"
   >
-    <umb-box-content>
+    <umb-box-content ng-if="vm.users.length > 0">
       <div class="flex">
         <select
           class="umb-dropdown flx-g1 mb0"
@@ -33,8 +32,25 @@
           label-key="impersonator_impersonate"
           button-style="primary"
           state="vm.impersonateUserButtonState"
+          disabled="!vm.userToImpersonate"
         ></umb-button>
       </div>
     </umb-box-content>
+    <umb-box-content ng-if="vm.users.length <= 0">
+      <div class="mb2">
+        <localize key="impersonator_noUser">
+          There are no active users to impersonate
+        </localize>
+      </div>
+    </umb-box-content>
+  </umb-box>
+  <umb-box ng-if="!vm.canImpersonateUsers && !vm.isImpersonating && !vm.loadingUsers">
+    <umb-box-content>
+      <div class="mb2">
+        <localize key="impersonator_notAdministrator">
+          You need users administrator rights, to be able to impersonate another user.
+        </localize>
+      </div>
+    </umb-box-content>
   </umb-box>
 </div>

From 1174bef00ae837af7a0fa7a91f94959a8c2483fb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=B8ren=20Kottal?= <snielsen43@gmail.com>
Date: Sun, 29 Oct 2023 08:58:27 +0100
Subject: [PATCH 2/2] Update da-DK.xml

---
 src/Our.Umbraco.Impersonator/wwwroot/lang/da-DK.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Our.Umbraco.Impersonator/wwwroot/lang/da-DK.xml b/src/Our.Umbraco.Impersonator/wwwroot/lang/da-DK.xml
index ea3d946..63552b7 100644
--- a/src/Our.Umbraco.Impersonator/wwwroot/lang/da-DK.xml
+++ b/src/Our.Umbraco.Impersonator/wwwroot/lang/da-DK.xml
@@ -16,6 +16,6 @@
 		<key alias="userNotFound">Den valgte bruger kunne ikke findes</key>
 		<key alias="invalidUserId">Det valgte bruger ID er ikke gyldigt</key>
 		<key alias="notAdministrator">Du skal have brugeradministratorrettigheder for at kunne imitere en anden bruger</key>
-		<kay alias="noUser">Der er ingen aktive brugere at imiterer.</kay>
+		<kay alias="noUser">Der er ingen aktive brugere at imitere.</kay>
 	</area>
 </language>