From a8dd55431acbfa8908819c067fdb703229450c83 Mon Sep 17 00:00:00 2001
From: Jamie Curnow <jcurnow@vision6.com.au>
Date: Fri, 24 Aug 2018 16:04:27 +1000
Subject: [PATCH] Fix some ui bugs

---
 src/frontend/js/app/user/form.ejs       | 4 ++--
 src/frontend/js/app/user/form.js        | 4 ++++
 src/frontend/js/app/user/password.ejs   | 2 +-
 src/frontend/js/app/users/list/item.ejs | 2 ++
 4 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/frontend/js/app/user/form.ejs b/src/frontend/js/app/user/form.ejs
index 1e88b44..7169dbf 100644
--- a/src/frontend/js/app/user/form.ejs
+++ b/src/frontend/js/app/user/form.ejs
@@ -25,14 +25,14 @@
                         <div class="invalid-feedback secret-error"></div>
                     </div>
                 </div>
-                <% if (isAdmin()) { %>
+                <% if (isAdmin() && !isSelf()) { %>
                 <div class="col-sm-12 col-md-12">
                     <div class="form-label"><%- i18n('roles', 'title') %></div>
                 </div>
                 <div class="col-sm-6 col-md-6">
                     <div class="form-group">
                         <label class="custom-switch">
-                            <input type="checkbox" class="custom-switch-input" name="is_admin" value="1"<%- isAdmin() ? ' checked' : '' %><%- isSelf() ? ' disabled' : '' %>>
+                            <input type="checkbox" class="custom-switch-input" name="is_admin" value="1"<%- isAdminUser() ? ' checked' : '' %><%- isSelf() ? ' disabled' : '' %>>
                             <span class="custom-switch-indicator"></span>
                             <span class="custom-switch-description"><%- i18n('roles', 'admin') %></span>
                         </label>
diff --git a/src/frontend/js/app/user/form.js b/src/frontend/js/app/user/form.js
index dca797c..6fe88f0 100644
--- a/src/frontend/js/app/user/form.js
+++ b/src/frontend/js/app/user/form.js
@@ -89,6 +89,10 @@ module.exports = Mn.View.extend({
             },
 
             isAdmin: function () {
+                return App.Cache.User.isAdmin();
+            },
+
+            isAdminUser: function () {
                 return view.model.isAdmin();
             },
 
diff --git a/src/frontend/js/app/user/password.ejs b/src/frontend/js/app/user/password.ejs
index 5b74a53..7dd497d 100644
--- a/src/frontend/js/app/user/password.ejs
+++ b/src/frontend/js/app/user/password.ejs
@@ -1,6 +1,6 @@
 <div class="modal-content">
     <div class="modal-header">
-        <h5 class="modal-title"><%- i18n('users', 'form-title', {self: isSelf(), name: name}) %></h5>
+        <h5 class="modal-title"><%- i18n('users', 'password-title', {self: isSelf(), name: name}) %></h5>
         <button type="button" class="close cancel" aria-label="Close" data-dismiss="modal">&nbsp;</button>
     </div>
     <div class="modal-body">
diff --git a/src/frontend/js/app/users/list/item.ejs b/src/frontend/js/app/users/list/item.ejs
index 9cd2feb..e8699b5 100644
--- a/src/frontend/js/app/users/list/item.ejs
+++ b/src/frontend/js/app/users/list/item.ejs
@@ -33,7 +33,9 @@
             <a href="#" class="edit-permissions dropdown-item"><i class="dropdown-icon fe fe-shield"></i> <%- i18n('users', 'edit-permissions') %></a>
             <a href="#" class="set-password dropdown-item"><i class="dropdown-icon fe fe-lock"></i> <%- i18n('users', 'change-password') %></a>
             <% if (!isSelf()) { %>
+            <% if (!is_disabled) { %>
             <a href="#" class="login dropdown-item"><i class="dropdown-icon fe fe-log-in"></i> <%- i18n('users', 'sign-in-as') %></a>
+            <% } %>
             <div class="dropdown-divider"></div>
             <a href="#" class="delete-user dropdown-item"><i class="dropdown-icon fe fe-trash-2"></i> <%- i18n('users', 'delete', {name: name}) %></a>
             <% } %>