From 4ea0bce19d27eb41eb27aa5dc0f6960e0c25328d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tim=20Gr=C3=B6ger?= <tim@groeger-clan.de>
Date: Thu, 12 Nov 2020 22:48:19 +0100
Subject: [PATCH] =?UTF-8?q?[Admin]=20neuer=20User=20kann=20hinzugef=C3=BCg?=
 =?UTF-8?q?t=20werden?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/plugins/user/components/NewUser.vue       | 42 ++++++++
 src/plugins/user/components/UpdateUser.vue    | 50 ++++++++++
 .../components/settings/MainUserSettings.vue  | 41 ++++++--
 src/plugins/user/pages/AdminSettings.vue      | 99 ++++++++++---------
 src/plugins/user/pages/Settings.vue           | 10 +-
 src/plugins/user/store/user.ts                | 17 ++++
 6 files changed, 199 insertions(+), 60 deletions(-)
 create mode 100644 src/plugins/user/components/NewUser.vue
 create mode 100644 src/plugins/user/components/UpdateUser.vue

diff --git a/src/plugins/user/components/NewUser.vue b/src/plugins/user/components/NewUser.vue
new file mode 100644
index 0000000..cd59e2a
--- /dev/null
+++ b/src/plugins/user/components/NewUser.vue
@@ -0,0 +1,42 @@
+<template>
+  <q-card class="12">
+    <q-card-section class="fit row justify-start content-center items-center">
+      <div class="col-xs-12 col-sm-6 text-center text-h6">
+        Neues Mitglied
+      </div>
+    </q-card-section>
+    <q-card-section>
+      <MainUserSettings :user="user" @update:user="setUser" :new-user="true" />
+    </q-card-section>
+  </q-card>
+</template>
+
+<script lang="ts">
+import { defineComponent, ref } from '@vue/composition-api';
+import MainUserSettings from 'src/plugins/user/components/settings/MainUserSettings.vue';
+import { Store } from 'vuex';
+import { StateInterface } from 'src/store';
+export default defineComponent({
+  name: 'NewUser',
+  components: { MainUserSettings },
+  setup(_, { root }) {
+    const store = <Store<StateInterface>>root.$store;
+    const user = ref<FG.User>({
+      userid: '',
+      display_name: '',
+      firstname: '',
+      lastname: '',
+      mail: '',
+      roles: []
+    });
+    function setUser(value: FG.User) {
+      store.dispatch('user/setUser', value).catch(error => {
+        console.warn(error);
+      });
+    }
+    return { user, setUser };
+  }
+});
+</script>
+
+<style scoped></style>
diff --git a/src/plugins/user/components/UpdateUser.vue b/src/plugins/user/components/UpdateUser.vue
new file mode 100644
index 0000000..634bc56
--- /dev/null
+++ b/src/plugins/user/components/UpdateUser.vue
@@ -0,0 +1,50 @@
+<template>
+  <q-card class="col-12">
+    <q-card-section class="fit row justify-start content-center items-center">
+      <div class="col-xs-12 col-sm-6 text-center text-h6">
+        Benutzereinstellungen
+      </div>
+      <div class="col-xs-12 col-sm-6 q-pa-sm">
+        <UserSelector :user="user" @update:user="userUpdated" />
+      </div>
+    </q-card-section>
+    <MainUserSettings :user="user" @update:user="updateUser" />
+  </q-card>
+</template>
+
+<script lang="ts">
+import { defineComponent, ref } from '@vue/composition-api';
+import { Store } from 'vuex';
+import { StateInterface } from '../../../store';
+import UserSelector from '../components/UserSelector.vue';
+import MainUserSettings from '../components/settings/MainUserSettings.vue';
+
+export default defineComponent({
+  name: 'UpdateUser',
+  components: { UserSelector, MainUserSettings },
+  setup(_, { root }) {
+    const store = <Store<StateInterface>>root.$store;
+    const user = ref(<FG.User>store.state.user.currentUser);
+
+    // can be dropped with VUE3
+    const userUpdated = (value: FG.User) => {
+      user.value = value;
+      console.log(value);
+    };
+
+    function updateUser(value: FG.User) {
+      store.dispatch('user/updateUser', value).catch(error => {
+        console.warn(error);
+      });
+    }
+
+    return {
+      user,
+      userUpdated,
+      updateUser
+    };
+  }
+});
+</script>
+
+<style scoped></style>
diff --git a/src/plugins/user/components/settings/MainUserSettings.vue b/src/plugins/user/components/settings/MainUserSettings.vue
index e429a3e..c2f3376 100644
--- a/src/plugins/user/components/settings/MainUserSettings.vue
+++ b/src/plugins/user/components/settings/MainUserSettings.vue
@@ -19,8 +19,9 @@
       <q-input
         class="col-xs-12 col-sm-6 q-pa-sm"
         label="Benutzername"
-        readonly
-        :value="props.user.userid"
+        :readonly="!props.newUser"
+        v-model="props.user.userid"
+        :rules="[isUseridUsed, notEmpty]"
         filled
       />
       <q-input
@@ -50,8 +51,11 @@
         option-value="name"
       />
     </q-card-section>
-    <q-separator />
-    <q-card-section class="fit row justify-start content-center items-center">
+    <q-separator v-if="!props.newUser" />
+    <q-card-section
+      class="fit row justify-start content-center items-center"
+      v-if="!props.newUser"
+    >
       <q-input
         v-if="isCurrentUser"
         class="col-xs-12 col-sm-6 col-md-4 q-pa-sm"
@@ -99,12 +103,20 @@ import { hasPermission } from 'src/components/permission';
 
 interface Props {
   user?: FG.User;
+  newUser?: boolean;
 }
 
 export default defineComponent({
   name: 'MainUserSettings',
-  props: ['user'],
-  setup(props: Props, { root }) {
+  props: {
+    user: {
+      required: true
+    },
+    newUser: {
+      default: false
+    }
+  },
+  setup(props: Props, { root, emit }) {
     const store = <Store<StateInterface>>root.$store;
 
     onBeforeMount(() => {
@@ -144,9 +156,9 @@ export default defineComponent({
           new_password: new_password.value
         });
       }
-      store.dispatch('user/updateUser', changed).catch(error => {
-        console.warn(error);
-      });
+      console.log(changed);
+
+      emit('update:user', changed);
     }
 
     function reset() {
@@ -172,6 +184,16 @@ export default defineComponent({
       );
     }
 
+    function isUseridUsed(val: string) {
+      return (
+        !store.state.user.users.find((user: FG.User) => {
+          return user.userid == val;
+        }) ||
+        !props.newUser ||
+        'Benutzername ist schon vergeben'
+      );
+    }
+
     const loading = computed(() => store.state.user.loading > 0);
 
     return {
@@ -185,6 +207,7 @@ export default defineComponent({
       isCurrentUser,
       isEmail,
       notEmpty,
+      isUseridUsed,
       save,
       reset,
       loading
diff --git a/src/plugins/user/pages/AdminSettings.vue b/src/plugins/user/pages/AdminSettings.vue
index e60494d..8604986 100644
--- a/src/plugins/user/pages/AdminSettings.vue
+++ b/src/plugins/user/pages/AdminSettings.vue
@@ -1,42 +1,52 @@
 <template>
   <div>
     <q-tabs v-model="tab" v-if="$q.screen.gt.sm">
-      <q-tab v-for="(tabindex, index) in tabs" :key="'tab'+index" :name="tabindex.name" :label="tabindex.label"/>
+      <q-tab
+        v-for="(tabindex, index) in tabs"
+        :key="'tab' + index"
+        :name="tabindex.name"
+        :label="tabindex.label"
+      />
     </q-tabs>
     <div class="fit row justify-end" v-else>
-      <q-btn flat round icon="mdi-menu" @click="showDrawer = !showDrawer"/>
+      <q-btn flat round icon="mdi-menu" @click="showDrawer = !showDrawer" />
     </div>
-    <q-drawer side="right" v-model="showDrawer" @click="showDrawer = !showDrawer" behavior="mobile">
+    <q-drawer
+      side="right"
+      v-model="showDrawer"
+      @click="showDrawer = !showDrawer"
+      behavior="mobile"
+    >
       <q-list v-model="tab">
-        <q-item v-for="(tabindex, index) in tabs" :key="'tab'+index" :active="tab == tabindex.name" clickable
-                @click="tab = tabindex.name">
+        <q-item
+          v-for="(tabindex, index) in tabs"
+          :key="'tab' + index"
+          :active="tab == tabindex.name"
+          clickable
+          @click="tab = tabindex.name"
+        >
           <q-item-label>{{ tabindex.label }}</q-item-label>
         </q-item>
       </q-list>
     </q-drawer>
-    <q-page padding
-            class="fit row justify-center content-start items-start q-gutter-sm">
-      <q-tab-panels v-model="tab" style="background-color: transparent;" class="q-ma-none q-pa-none" animated>
+    <q-page
+      padding
+      class="fit row justify-center content-start items-start q-gutter-sm"
+    >
+      <q-tab-panels
+        v-model="tab"
+        style="background-color: transparent;"
+        class="q-ma-none q-pa-none"
+        animated
+      >
         <q-tab-panel name="user">
-          <q-card class="col-12">
-            <q-card-section
-              class="fit row justify-start content-center items-center"
-            >
-              <div class="col-xs-12 col-sm-6 text-center text-h6">
-                Benutzereinstellungen
-              </div>
-              <div class="col-xs-12 col-sm-6 q-pa-sm">
-                <UserSelector :user="user" @update:user="userUpdated"/>
-              </div>
-            </q-card-section>
-            <MainUserSettings :user="user"/>
-          </q-card>
+          <UpdateUser />
         </q-tab-panel>
         <q-tab-panel name="newUser">
-          <h1>hier kommt noch was</h1>
+          <NewUser />
         </q-tab-panel>
         <q-tab-panel name="roles">
-          <RoleSettings v-if="canEditRoles"/>
+          <RoleSettings v-if="canEditRoles" />
         </q-tab-panel>
       </q-tab-panels>
     </q-page>
@@ -50,19 +60,19 @@ import {
   onBeforeMount,
   ref
 } from '@vue/composition-api';
-import UserSelector from '../components/UserSelector.vue';
-import MainUserSettings from '../components/settings/MainUserSettings.vue';
 import RoleSettings from '../components/settings/RoleSettings.vue';
-import {Store} from 'vuex';
-import {StateInterface} from 'src/store';
-import {hasPermission} from 'src/components/permission';
-import {PERMISSIONS} from '../permissions';
-import {Screen} from 'quasar';
+import UpdateUser from '../components/UpdateUser.vue';
+import NewUser from '../components/NewUser.vue';
+import { Store } from 'vuex';
+import { StateInterface } from 'src/store';
+import { hasPermission } from 'src/components/permission';
+import { PERMISSIONS } from '../permissions';
+import { Screen } from 'quasar';
 
 export default defineComponent({
   name: 'AdminSettings',
-  components: {UserSelector, MainUserSettings, RoleSettings},
-  setup(_, {root}) {
+  components: { RoleSettings, UpdateUser, NewUser },
+  setup(_, { root }) {
     const store = <Store<StateInterface>>root.$store;
 
     onBeforeMount(() => {
@@ -72,13 +82,6 @@ export default defineComponent({
     const canEditRoles = computed(() =>
       hasPermission(PERMISSIONS.ROLES_EDIT, store)
     );
-    const user = ref(<FG.User>store.state.user.currentUser);
-
-    // can be dropped with VUE3
-    const userUpdated = (value: FG.User) => {
-      user.value = value;
-      console.log(value);
-    };
 
     interface Tab {
       name: string;
@@ -86,27 +89,25 @@ export default defineComponent({
     }
 
     const tabs: Tab[] = [
-      {name: 'user', label: 'Mitglieder'},
-      {name: 'newUser', label: 'Neues Mitglied'},
-      {name: 'roles', label: 'Rollen'}
-    ]
+      { name: 'user', label: 'Mitglieder' },
+      { name: 'newUser', label: 'Neues Mitglied' },
+      { name: 'roles', label: 'Rollen' }
+    ];
 
-    const drawer = ref<boolean>(false)
+    const drawer = ref<boolean>(false);
 
     const showDrawer = computed({
       get: () => {
-        return !Screen.gt.sm && drawer.value
+        return !Screen.gt.sm && drawer.value;
       },
       set: (val: boolean) => {
-        drawer.value = val
+        drawer.value = val;
       }
-    })
+    });
 
     const tab = ref<string>('user');
 
     return {
-      user,
-      userUpdated,
       canEditRoles,
       showDrawer,
       tab,
diff --git a/src/plugins/user/pages/Settings.vue b/src/plugins/user/pages/Settings.vue
index 68f9ce2..b8c4e08 100644
--- a/src/plugins/user/pages/Settings.vue
+++ b/src/plugins/user/pages/Settings.vue
@@ -11,7 +11,7 @@
         >
           <div class="col-12 text-center text-h6">Benutzereinstellungen</div>
         </q-card-section>
-        <MainUserSettings :user="currentUser" />
+        <MainUserSettings :user="currentUser" @update:user="updateUser" />
       </q-card>
       <div class="col-12 text-left text-h6">Aktive Sessions:</div>
       <sessions
@@ -51,11 +51,17 @@ export default defineComponent({
     const currentUser = ref(<FG.User>store.state.user.currentUser);
     const sessions = computed(() => store.state.session.sessions);
     const sessionsLoading = computed(() => store.state.session.loading);
+    function updateUser(value: FG.User) {
+      store.dispatch('user/updateUser', value).catch(error => {
+        console.warn(error);
+      });
+    }
 
     return {
       currentUser,
       sessionsLoading,
-      sessions
+      sessions,
+      updateUser
     };
   }
 });
diff --git a/src/plugins/user/store/user.ts b/src/plugins/user/store/user.ts
index 8a1062a..c6b9513 100644
--- a/src/plugins/user/store/user.ts
+++ b/src/plugins/user/store/user.ts
@@ -103,6 +103,23 @@ const actions: ActionTree<UserStateInterface, StateInterface> = {
       });
   },
 
+  setUser({ commit, state, dispatch }, data: FG.User) {
+    commit('setLoading');
+    axios
+      .post(`users`, data)
+      .then(() => {
+        if (state.currentUser && state.currentUser.userid === data.userid)
+          void dispatch('getCurrentUser');
+        else void dispatch('getUsers');
+      })
+      .catch(error => {
+        console.warn(error);
+      })
+      .finally(() => {
+        commit('setLoading', false);
+      });
+  },
+
   getRoles({ commit, state }, force = false) {
     if (!force && state.roles.length > 0) return;
     commit('setLoading');