From a5ab8ceb47d78fcff33d025cb75a70bc911f6848 Mon Sep 17 00:00:00 2001
From: Angelina Filippova <linakirsanova@gmail.com>
Date: Thu, 20 Aug 2020 20:29:58 +0300
Subject: [PATCH 1/6] Add select for actor type, remove default select styles

---
 .../users/components/ModerationDropdown.vue   | 47 +++++++++++++++++++
 1 file changed, 47 insertions(+)

diff --git a/src/views/users/components/ModerationDropdown.vue b/src/views/users/components/ModerationDropdown.vue
index 4ae3d7f2..6aa939d0 100644
--- a/src/views/users/components/ModerationDropdown.vue
+++ b/src/views/users/components/ModerationDropdown.vue
@@ -16,8 +16,17 @@
       </el-button>
     </div>
     <el-dropdown-menu slot="dropdown">
+      <el-dropdown-item
+        class="actor-type-dropdown"
+        @click.native="getPasswordResetToken(user.nickname)">
+        <el-select v-model="actorType" placeholder="Actor Type" class="actor-type-select">
+          <el-option label="Bot" value="bot"/>
+          <el-option label="Person" value="person"/>
+        </el-select>
+      </el-dropdown-item>
       <el-dropdown-item
         v-if="showAdminAction(user)"
+        divided
         @click.native="toggleUserRight(user, 'admin')">
         {{ user.roles.admin ? $t('users.revokeAdmin') : $t('users.grantAdmin') }}
       </el-dropdown-item>
@@ -137,6 +146,11 @@ export default {
       default: ''
     }
   },
+  data() {
+    return {
+      actorType: ''
+    }
+  },
   computed: {
     isDesktop() {
       return this.$store.state.app.device === 'desktop'
@@ -225,6 +239,39 @@ export default {
 </script>
 
 <style rel='stylesheet/scss' lang='scss'>
+  .el-dropdown-menu--small  .el-dropdown-menu__item.el-dropdown-menu__item--divided.actor-type-dropdown:before {
+    margin: 0 0;
+    height: 0;
+  }
+  .el-dropdown-menu--small .actor-type-dropdown {
+    padding: 0;
+  }
+  .actor-type-select {
+    width: 100%;
+    input {
+      border-color: transparent;
+      color: #606266;
+    }
+    .el-input__inner:hover {
+      border-color: transparent;
+      background-color: #ecf5ff;
+    }
+    .el-input.is-focus {
+      border-color: transparent;
+    }
+    .el-input__suffix-inner {
+      pointer-events: none;
+    }
+    .el-select .el-input__inner:focus {
+      border-color: transparent;
+    }
+    .el-input.is-active .el-input__inner, .el-input__inner:focus {
+      border-color: transparent;
+    }
+  }
+  .actor-type-select .el-input.is-focus .el-input__inner {
+      border-color: transparent;
+    }
   .moderate-user-button {
     text-align: left;
     width: 350px;

From a55b76208e3dd12ffee1d601ad2e0896287a6b15 Mon Sep 17 00:00:00 2001
From: Angelina Filippova <linakirsanova@gmail.com>
Date: Sat, 22 Aug 2020 01:33:40 +0300
Subject: [PATCH 2/6] Add ability to change actor type from moderation dropdown

---
 src/store/modules/users.js                    | 12 ++++++++++-
 .../users/components/ModerationDropdown.vue   | 20 ++++++++++---------
 2 files changed, 22 insertions(+), 10 deletions(-)

diff --git a/src/store/modules/users.js b/src/store/modules/users.js
index 760a4afa..e90c770d 100644
--- a/src/store/modules/users.js
+++ b/src/store/modules/users.js
@@ -16,7 +16,8 @@ import {
   forcePasswordReset,
   approveUserAccount,
   confirmUserEmail,
-  resendConfirmationEmail
+  resendConfirmationEmail,
+  updateUserCredentials
 } from '@/api/users'
 
 const users = {
@@ -277,6 +278,15 @@ const users = {
       const currentFilters = { ...defaultFilters, ...filters }
       commit('SET_USERS_FILTERS', currentFilters)
       dispatch('SearchUsers', { query: state.searchQuery, page: 1 })
+    },
+    async UpdateActorType({ dispatch, getters }, { user, type, _userId, _statusId }) {
+      const updatedUsers = [{ ...user, actor_type: type }]
+      const nickname = [user.nickname]
+      const credentials = { actor_type: type }
+
+      const callApiFn = async() => await updateUserCredentials(nickname, credentials, getters.authHost, getters.token)
+
+      dispatch('ApplyChanges', { updatedUsers, callApiFn, userId: _userId, statusId: _statusId })
     }
   }
 }
diff --git a/src/views/users/components/ModerationDropdown.vue b/src/views/users/components/ModerationDropdown.vue
index 6aa939d0..a0e8c993 100644
--- a/src/views/users/components/ModerationDropdown.vue
+++ b/src/views/users/components/ModerationDropdown.vue
@@ -17,11 +17,10 @@
     </div>
     <el-dropdown-menu slot="dropdown">
       <el-dropdown-item
-        class="actor-type-dropdown"
-        @click.native="getPasswordResetToken(user.nickname)">
+        class="actor-type-dropdown">
         <el-select v-model="actorType" placeholder="Actor Type" class="actor-type-select">
-          <el-option label="Bot" value="bot"/>
-          <el-option label="Person" value="person"/>
+          <el-option value="Service"/>
+          <el-option value="Person"/>
         </el-select>
       </el-dropdown-item>
       <el-dropdown-item
@@ -146,12 +145,15 @@ export default {
       default: ''
     }
   },
-  data() {
-    return {
-      actorType: ''
-    }
-  },
   computed: {
+    actorType: {
+      get() {
+        return this.user.actor_type
+      },
+      set(type) {
+        this.$store.dispatch('UpdateActorType', { user: this.user, type })
+      }
+    },
     isDesktop() {
       return this.$store.state.app.device === 'desktop'
     }

From d8b3e9e9df90adcf2a38b4e1ccaa82758b9118b5 Mon Sep 17 00:00:00 2001
From: Angelina Filippova <linakirsanova@gmail.com>
Date: Sat, 22 Aug 2020 03:04:16 +0300
Subject: [PATCH 3/6] Show actor type on user show page and manage it from that
 page

---
 src/lang/en.js                                    |  7 +++++--
 src/store/modules/users.js                        |  3 +--
 src/views/users/components/ModerationDropdown.vue | 13 +++++++++----
 src/views/users/show.vue                          | 11 ++++++++++-
 4 files changed, 25 insertions(+), 9 deletions(-)

diff --git a/src/lang/en.js b/src/lang/en.js
index 8e9629b2..399f7ec2 100644
--- a/src/lang/en.js
+++ b/src/lang/en.js
@@ -271,7 +271,9 @@ export default {
     invalidNickname: 'invalid nickname',
     passwordResetTokenGenerated: 'Password reset token was generated:',
     linkToResetPassword: 'You can also use this link to reset password:',
-    registrationReason: 'Registration Reason'
+    registrationReason: 'Registration Reason',
+    service: 'Service',
+    person: 'Person'
   },
   statuses: {
     statuses: 'Statuses',
@@ -292,7 +294,8 @@ export default {
     admin: 'Admin',
     local: 'Local',
     external: 'External',
-    accountType: 'Account type',
+    accountType: 'Account Type',
+    actorType: 'Actor Type',
     nickname: 'Nickname',
     recentStatuses: 'Recent Statuses',
     roles: 'Roles',
diff --git a/src/store/modules/users.js b/src/store/modules/users.js
index e90c770d..631bff8e 100644
--- a/src/store/modules/users.js
+++ b/src/store/modules/users.js
@@ -281,10 +281,9 @@ const users = {
     },
     async UpdateActorType({ dispatch, getters }, { user, type, _userId, _statusId }) {
       const updatedUsers = [{ ...user, actor_type: type }]
-      const nickname = [user.nickname]
       const credentials = { actor_type: type }
 
-      const callApiFn = async() => await updateUserCredentials(nickname, credentials, getters.authHost, getters.token)
+      const callApiFn = async() => await updateUserCredentials(user.nickname, credentials, getters.authHost, getters.token)
 
       dispatch('ApplyChanges', { updatedUsers, callApiFn, userId: _userId, statusId: _statusId })
     }
diff --git a/src/views/users/components/ModerationDropdown.vue b/src/views/users/components/ModerationDropdown.vue
index a0e8c993..fd6ba5a3 100644
--- a/src/views/users/components/ModerationDropdown.vue
+++ b/src/views/users/components/ModerationDropdown.vue
@@ -18,9 +18,9 @@
     <el-dropdown-menu slot="dropdown">
       <el-dropdown-item
         class="actor-type-dropdown">
-        <el-select v-model="actorType" placeholder="Actor Type" class="actor-type-select">
-          <el-option value="Service"/>
-          <el-option value="Person"/>
+        <el-select v-model="actorType" :placeholder="$t('userProfile.actorType')" class="actor-type-select">
+          <el-option :label="$t('users.service')" value="Service"/>
+          <el-option :label="$t('users.person')" value="Person"/>
         </el-select>
       </el-dropdown-item>
       <el-dropdown-item
@@ -151,7 +151,12 @@ export default {
         return this.user.actor_type
       },
       set(type) {
-        this.$store.dispatch('UpdateActorType', { user: this.user, type })
+        this.$store.dispatch('UpdateActorType', {
+          user: this.user,
+          type,
+          _userId: this.user.id,
+          _statusId: this.statusId
+        })
       }
     },
     isDesktop() {
diff --git a/src/views/users/show.vue b/src/views/users/show.vue
index 88e18d3a..5aa21038 100644
--- a/src/views/users/show.vue
+++ b/src/views/users/show.vue
@@ -46,10 +46,19 @@
             <tbody>
               <tr class="el-table__row">
                 <td class="name-col">ID</td>
-                <td class="value-col">
+                <td>
                   {{ user.id }}
                 </td>
               </tr>
+              <tr class="el-table__row">
+                <td>{{ $t('userProfile.actorType') }}</td>
+                <td>
+                  <el-tag
+                    :type="userCredentials.actor_type === 'Person' ? 'success' : 'warning'">
+                    {{ userCredentials.actor_type }}
+                  </el-tag>
+                </td>
+              </tr>
               <tr class="el-table__row">
                 <td>{{ $t('userProfile.tags') }}</td>
                 <td>

From 2a42c5ed6bd67f18de0f4c8f0b4a859d812d3046 Mon Sep 17 00:00:00 2001
From: Angelina Filippova <linakirsanova@gmail.com>
Date: Sun, 23 Aug 2020 19:37:40 +0300
Subject: [PATCH 4/6] Fix tests

---
 test/views/users/index.test.js | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/test/views/users/index.test.js b/test/views/users/index.test.js
index 78ce7f06..8849340f 100644
--- a/test/views/users/index.test.js
+++ b/test/views/users/index.test.js
@@ -70,7 +70,7 @@ describe('Search and filter users', () => {
 describe('Users actions', () => {
   let store
   const htmlElement = (trChild, liChild) =>
-    `.el-table__fixed-body-wrapper table tr:nth-child(${trChild}) ul.el-dropdown-menu li:nth-child(${liChild})`
+    `.el-table__fixed-body-wrapper table tr:nth-child(${trChild}) ul.el-dropdown-menu > li:nth-child(${liChild})`
 
   beforeEach(() => {
     store = new Vuex.Store(cloneDeep(storeConfig))
@@ -88,7 +88,7 @@ describe('Users actions', () => {
     const user = store.state.users.fetchedUsers[2]
     expect(user.roles.admin).toBe(false)
     expect(user.roles.moderator).toBe(false)
-    wrapper.find(htmlElement(3, 1)).trigger('click')
+    wrapper.find(htmlElement(3, 2)).trigger('click')
 
     const updatedUser = store.state.users.fetchedUsers[2]
     expect(updatedUser.roles.admin).toBe(true)
@@ -107,7 +107,7 @@ describe('Users actions', () => {
     const user = store.state.users.fetchedUsers[2]
     expect(user.roles.admin).toBe(false)
     expect(user.roles.moderator).toBe(false)
-    wrapper.find(htmlElement(3, 2)).trigger('click')
+    wrapper.find(htmlElement(3, 3)).trigger('click')
 
     const updatedUser = store.state.users.fetchedUsers[2]
     expect(updatedUser.roles.moderator).toBe(true)
@@ -124,9 +124,9 @@ describe('Users actions', () => {
     await flushPromises()
 
     const dropdownMenuItems = wrapper.findAll(
-      `.el-table__fixed-body-wrapper table tr:nth-child(2) ul.el-dropdown-menu li`
+      `.el-table__fixed-body-wrapper table tr:nth-child(2) ul.el-dropdown-menu > li`
     )
-    expect(dropdownMenuItems.length).toBe(6)
+    expect(dropdownMenuItems.length).toBe(7)
     done()
   })
 
@@ -141,7 +141,7 @@ describe('Users actions', () => {
 
     const user = store.state.users.fetchedUsers[1]
     expect(user.deactivated).toBe(false)
-    wrapper.find(htmlElement(2, 1)).trigger('click')
+    wrapper.find(htmlElement(2, 2)).trigger('click')
 
     const updatedUser = store.state.users.fetchedUsers[1]
     expect(updatedUser.deactivated).toBe(true)
@@ -158,7 +158,7 @@ describe('Users actions', () => {
     await flushPromises()
     expect(store.state.users.fetchedUsers[1].deactivated).toBe(false)
 
-    wrapper.find(htmlElement(2, 2)).trigger('click')
+    wrapper.find(htmlElement(2, 3)).trigger('click')
     store.dispatch('DeleteUsers', { users: [{ active: true, deactivated: false, id: '10', nickname: 'bob', local: false, external: true, roles: { admin: false, moderator: false }, tags: ['mrf_tag:sandbox'] }] })
 
     await flushPromises()
@@ -180,8 +180,8 @@ describe('Users actions', () => {
     expect(user1.tags.length).toBe(0)
     expect(user2.tags.length).toBe(1)
 
-    wrapper.find(htmlElement(1, 5)).trigger('click')
-    wrapper.find(htmlElement(2, 5)).trigger('click')
+    wrapper.find(htmlElement(1, 6)).trigger('click')
+    wrapper.find(htmlElement(2, 6)).trigger('click')
 
     const updatedUser1 = store.state.users.fetchedUsers[0]
     const updatedUser2 = store.state.users.fetchedUsers[1]
@@ -201,7 +201,7 @@ describe('Users actions', () => {
 
     const user = store.state.users.fetchedUsers[1]
     expect(user.tags.length).toBe(1)
-    wrapper.find(htmlElement(2, 6)).trigger('click')
+    wrapper.find(htmlElement(2, 7)).trigger('click')
 
     const updatedUser = store.state.users.fetchedUsers[1]
     expect(updatedUser.tags.length).toBe(0)
@@ -247,7 +247,7 @@ describe('Users actions', () => {
     expect(wrapper.vm.resetPasswordDialogOpen).toBe(false)
     expect(store.state.users.passwordResetToken.token).toBe('')
 
-    wrapper.find(htmlElement(1, 11)).trigger('click')
+    wrapper.find(htmlElement(1, 12)).trigger('click')
     await flushPromises()
 
     expect(wrapper.vm.resetPasswordDialogOpen).toBe(true)

From cd92dfa6957977901bf2958af72bad791e08f654 Mon Sep 17 00:00:00 2001
From: Angelina Filippova <linakirsanova@gmail.com>
Date: Mon, 24 Aug 2020 00:31:38 +0300
Subject: [PATCH 5/6] Add test for updating actor_type

---
 src/api/__mocks__/status.js    |  8 ++++----
 src/api/__mocks__/users.js     | 12 ++++++++----
 test/views/users/index.test.js | 19 +++++++++++++++++++
 3 files changed, 31 insertions(+), 8 deletions(-)

diff --git a/src/api/__mocks__/status.js b/src/api/__mocks__/status.js
index b24142d2..e77f0a0b 100644
--- a/src/api/__mocks__/status.js
+++ b/src/api/__mocks__/status.js
@@ -55,11 +55,11 @@ export async function fetchStatusesByInstance({ instance, authHost, token, pageS
           'nickname': 'sky',
           'url': 'http://localhost:4000/users/sky'
         },
-        'content': 'A nice young couple contacted us from Brazil to decorate their newly acquired apartment.',
-        'created_at': '2020-01-31T18:20:01.000Z',
-        'id': '9rZIr0Jzao5Gjgfmro',
+        'content': 'i love parks&rec',
+        'created_at': '2020-04-12T18:20:01.000Z',
+        'id': 'o5Gjgfmro9rZIr0Jza',
         'sensitive': false,
-        'url': 'http://localhost:4000/objects/7af9abbd-fb6c-4318-aeb7-6636c138ac98',
+        'url': 'http://localhost:4000/objects/7af9abbd-aeb7-6636c138ac98-fb6c-4318',
         'visibility': 'unlisted'
       },
       {
diff --git a/src/api/__mocks__/users.js b/src/api/__mocks__/users.js
index b21cac33..f6b9c3fe 100644
--- a/src/api/__mocks__/users.js
+++ b/src/api/__mocks__/users.js
@@ -1,8 +1,8 @@
 export let users = [
-  { active: true, approval_pending: false, deactivated: false, id: '2', nickname: 'allis', local: true, external: false, roles: { admin: true, moderator: false }, tags: [] },
-  { active: true, approval_pending: false, deactivated: false, id: '10', nickname: 'bob', local: false, external: true, roles: { admin: false, moderator: false }, tags: ['mrf_tag:sandbox'] },
-  { active: false, approval_pending: false, deactivated: true, id: 'abc', nickname: 'john', local: true, external: false, roles: { admin: false, moderator: false }, tags: ['mrf_tag:media-strip'] },
-  { active: true, approval_pending: true, deactivated: false, id: '100', nickname: 'sally', local: true, external: false, roles: { admin: false, moderator: false }, tags: [] }
+  { active: true, approval_pending: false, deactivated: false, id: '2', nickname: 'allis', local: true, external: false, roles: { admin: true, moderator: false }, tags: [], actor_type: 'Person' },
+  { active: true, approval_pending: false, deactivated: false, id: '10', nickname: 'bob', local: false, external: true, roles: { admin: false, moderator: false }, tags: ['mrf_tag:sandbox'], actor_type: 'Person' },
+  { active: false, approval_pending: false, deactivated: true, id: 'abc', nickname: 'john', local: true, external: false, roles: { admin: false, moderator: false }, tags: ['mrf_tag:media-strip'], actor_type: 'Person' },
+  { active: true, approval_pending: true, deactivated: false, id: '100', nickname: 'sally', local: true, external: false, roles: { admin: false, moderator: false }, tags: [], actor_type: 'Service' }
 ]
 
 const userProfile = { avatar: 'avatar.jpg', nickname: 'allis', id: '2', tags: [], roles: { admin: true, moderator: false }, local: true, external: false }
@@ -119,3 +119,7 @@ export async function createNewAccount(nickname, email, password, authHost, toke
   users = [...users, newUser]
   return Promise.resolve()
 }
+
+export async function updateUserCredentials(nickname, credentials, authHost, token) {
+  return Promise.resolve()
+}
diff --git a/test/views/users/index.test.js b/test/views/users/index.test.js
index 8849340f..dc9eb370 100644
--- a/test/views/users/index.test.js
+++ b/test/views/users/index.test.js
@@ -353,7 +353,26 @@ describe('Creates new account', () => {
 
     expect(wrapper.vm.validatePassword(validatePasswordRule, '', identity)).toBeInstanceOf(Error)
     expect(wrapper.vm.validatePassword(validatePasswordRule, '1234', identity)).toBeUndefined()
+  })
 
+  it('updates actor type', async (done) => {
+    const wrapper = mount(Users, {
+      store,
+      localVue,
+      sync: false,
+      stubs: ['router-link']
+    })
+    await flushPromises()
 
+    const user = store.state.users.fetchedUsers[0]
+    expect(user.actor_type).toBe('Person')
+
+    const findWrapper = (trChild, liChild1, liChild2) =>
+    `.el-table__fixed-body-wrapper table tr:nth-child(${trChild}) ul.el-dropdown-menu > li:nth-child(${liChild1}) ul li:nth-child(${liChild2})`
+    wrapper.find(findWrapper(1, 1, 1)).trigger('click')
+
+    const updatedUser = store.state.users.fetchedUsers[0]
+    expect(updatedUser.actor_type).toBe('Service')
+    done()
   })
 })

From 50bd778252f777cd91afb343456caf53d01717d0 Mon Sep 17 00:00:00 2001
From: Angelina Filippova <linakirsanova@gmail.com>
Date: Mon, 24 Aug 2020 00:50:56 +0300
Subject: [PATCH 6/6] Update Changelog

---
 CHANGELOG.md | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 45eb0a9b..ecf9c913 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -6,6 +6,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ## Unreleased
 
+### Added
+
+- Allow managing user's actor_type field via Admin API
+
 ### Fixed
 
 - Fix following and unfollowing relays from Admin-FE, update mobile UI