From e54faab91e7d06e09232e5a560a35a952d90ea5d Mon Sep 17 00:00:00 2001
From: Ryan Wang <i@ryanc.cc>
Date: Mon, 27 Jun 2022 17:03:08 +0800
Subject: [PATCH] feat: support update a user

Signed-off-by: Ryan Wang <i@ryanc.cc>
---
 src/modules/system/users/UserList.vue         | 36 +++++++++++--------
 .../users/components/UserCreationModal.vue    | 18 ++++++++--
 2 files changed, 38 insertions(+), 16 deletions(-)

diff --git a/src/modules/system/users/UserList.vue b/src/modules/system/users/UserList.vue
index bec7cd33..12aad833 100644
--- a/src/modules/system/users/UserList.vue
+++ b/src/modules/system/users/UserList.vue
@@ -13,13 +13,14 @@ import {
   VTag,
 } from "@halo-dev/components";
 import UserCreationModal from "./components/UserCreationModal.vue";
-import { ref } from "vue";
+import { onMounted, ref } from "vue";
 import { axiosInstance } from "@halo-dev/admin-shared";
 import type { User } from "@/types/extension";
 
 const checkAll = ref(false);
 const creationModal = ref<boolean>(false);
 const users = ref<User[]>([]);
+const selectedUser = ref<User | null>(null);
 
 const handleFetchUsers = async () => {
   try {
@@ -30,11 +31,19 @@ const handleFetchUsers = async () => {
   }
 };
 
-handleFetchUsers();
+const handleOpenCreateModal = (user: User) => {
+  selectedUser.value = user;
+  creationModal.value = true;
+};
+
+onMounted(() => {
+  handleFetchUsers();
+});
 </script>
 <template>
   <UserCreationModal
     v-model:visible="creationModal"
+    :user="selectedUser"
     @close="handleFetchUsers"
   />
 
@@ -181,16 +190,7 @@ handleFetchUsers();
         </div>
       </template>
       <ul class="box-border h-full w-full divide-y divide-gray-100" role="list">
-        <li
-          v-for="(user, index) in users"
-          :key="index"
-          @click="
-            $router.push({
-              name: 'UserDetail',
-              params: { name: user.metadata.name },
-            })
-          "
-        >
+        <li v-for="(user, index) in users" :key="index">
           <div
             :class="{
               'bg-gray-100': checkAll,
@@ -220,7 +220,15 @@ handleFetchUsers();
               </div>
               <div class="flex-1">
                 <div class="flex flex-row items-center">
-                  <span class="mr-2 truncate text-sm font-medium text-gray-900">
+                  <span
+                    class="mr-2 truncate text-sm font-medium text-gray-900"
+                    @click="
+                      $router.push({
+                        name: 'UserDetail',
+                        params: { name: user.metadata.name },
+                      })
+                    "
+                  >
                     {{ user.spec.displayName }}
                   </span>
                   <VTag class="sm:hidden">{{ user.metadata.name }}</VTag>
@@ -246,7 +254,7 @@ handleFetchUsers();
                     {{ user.metadata.creationTimestamp }}
                   </time>
                   <span class="cursor-pointer">
-                    <IconSettings />
+                    <IconSettings @click="handleOpenCreateModal(user)" />
                   </span>
                 </div>
               </div>
diff --git a/src/modules/system/users/components/UserCreationModal.vue b/src/modules/system/users/components/UserCreationModal.vue
index 4832c8c8..784dba3f 100644
--- a/src/modules/system/users/components/UserCreationModal.vue
+++ b/src/modules/system/users/components/UserCreationModal.vue
@@ -1,6 +1,6 @@
 <script lang="ts" name="UserCreationModal" setup>
 import type { PropType } from "vue";
-import { computed, ref } from "vue";
+import { computed, ref, watch } from "vue";
 import { axiosInstance } from "@halo-dev/admin-shared";
 import {
   IconSave,
@@ -59,6 +59,12 @@ const creationModalTitle = computed(() => {
   return isUpdateMode.value ? "编辑用户" : "新增用户";
 });
 
+watch(props, (newVal) => {
+  if (newVal.visible && props.user) {
+    creationForm.value.user = props.user;
+  }
+});
+
 const handleVisibleChange = (visible: boolean) => {
   emit("update:visible", visible);
   if (!visible) {
@@ -69,7 +75,15 @@ const handleVisibleChange = (visible: boolean) => {
 const handleCreateUser = async () => {
   try {
     creationForm.value.saving = true;
-    await axiosInstance.post("/api/v1alpha1/users", creationForm.value.user);
+
+    if (isUpdateMode.value) {
+      await axiosInstance.put(
+        `/api/v1alpha1/users/${creationForm.value.user.metadata.name}`,
+        creationForm.value.user
+      );
+    } else {
+      await axiosInstance.post("/api/v1alpha1/users", creationForm.value.user);
+    }
 
     handleVisibleChange(false);
   } catch (e) {