From 9f7451a276a7376d6d41074a841f8acb2c9ff410 Mon Sep 17 00:00:00 2001
From: axolotle <nchesnais@mailoo.org>
Date: Tue, 1 Aug 2023 15:13:35 +0200
Subject: [PATCH] edit: add error handling and rename keys

---
 composables/api.ts  | 14 ++++++++++---
 layouts/default.vue |  2 +-
 pages/edit.vue      | 48 +++++++++++++++++++++++++--------------------
 pages/index.vue     |  2 +-
 4 files changed, 40 insertions(+), 26 deletions(-)

diff --git a/composables/api.ts b/composables/api.ts
index f7cc281..662e1a9 100644
--- a/composables/api.ts
+++ b/composables/api.ts
@@ -50,8 +50,8 @@ export interface UserData {
   username: string
   fullname: string
   mail: string
-  'mail-aliases': string[]
-  'mail-forward': string[]
+  mailalias: string[]
+  mailforward: string[]
   groups: string[]
   apps: Record<string, { label: string; url: string }>
 }
@@ -65,5 +65,13 @@ export async function useUserInfo() {
     const { data } = await useApi('/me')
     userData.value = data.value as UserData
   }
-  return userData as Ref<UserData>
+
+  const update = (data: Partial<UserData>) => {
+    Object.assign(userData.value as UserData, data)
+  }
+
+  return {
+    userData: userData as Ref<UserData>,
+    update,
+  }
 }
diff --git a/layouts/default.vue b/layouts/default.vue
index 0bf21bf..9378c83 100644
--- a/layouts/default.vue
+++ b/layouts/default.vue
@@ -5,7 +5,7 @@ const head = useLocaleHead({
 })
 const { t } = useI18n()
 const isLoggedIn = useIsLoggedIn()
-const me = await useUserInfo()
+const { userData: me } = await useUserInfo()
 
 const colorMode = useColorMode()
 const themes = [
diff --git a/pages/edit.vue b/pages/edit.vue
index 0c24d8e..cbe5e0b 100644
--- a/pages/edit.vue
+++ b/pages/edit.vue
@@ -2,32 +2,42 @@
 import { useForm } from 'vee-validate'
 import { toTypedSchema } from '@vee-validate/yup'
 import * as yup from 'yup'
+import type { UserData } from '@/composables/api'
 
-const me = await useUserInfo()
+const { userData, update } = await useUserInfo()
 
-const { handleSubmit } = useForm({
+const { handleSubmit, setFieldError } = useForm({
   validationSchema: toTypedSchema(
     yup.object({
-      username: yup.string(),
+      username: yup.string().required(),
       fullname: yup.string().required().min(1),
-      mailAliases: yup.array().of(yup.string().email()),
-      mailForward: yup.array().of(yup.string().email()),
+      mailalias: yup.array().of(yup.string().email().required()),
+      mailforward: yup.array().of(yup.string().email().required()),
     }),
   ),
   initialValues: {
-    username: me.value.username,
-    fullname: me.value.fullname,
-    mailAliases: me.value['mail-aliases'].length
-      ? me.value['mail-aliases']
+    username: userData.value.username,
+    fullname: userData.value.fullname,
+    mailalias: userData.value.mailalias.length
+      ? userData.value.mailalias
       : [''],
-    mailForward: me.value['mail-forward'].length
-      ? me.value['mail-forward']
+    mailforward: userData.value.mailforward.length
+      ? userData.value.mailforward
       : [''],
   },
 })
 
-const onSubmit = handleSubmit((form) => {
-  console.log('SUBMIT user edit', form)
+const onSubmit = handleSubmit(async (form) => {
+  const { data, error } = await useApi<UserData>('/update', {
+    method: 'PUT',
+    body: form,
+  })
+
+  if (error.value) {
+    setFieldError(error.value.data.path, error.value.data.error_key)
+  } else if (data.value) {
+    update(data.value)
+  }
 })
 </script>
 
@@ -58,21 +68,17 @@ const onSubmit = handleSubmit((form) => {
       </div>
 
       <div class="basis-1/2">
-        <FormField
-          name="mailAliases"
-          :label="$t('mail_addresses')"
-          class="mb-10"
-        >
+        <FormField name="mailalias" :label="$t('mail_addresses')" class="mb-10">
           <TextInputList
-            name="mailAliases"
+            name="mailalias"
             type="text"
             :placeholder="$t('new_mail')"
           />
         </FormField>
 
-        <FormField name="mailForward" :label="$t('mail_forward')">
+        <FormField name="mailforward" :label="$t('mail_forward')">
           <TextInputList
-            name="mailForward"
+            name="mailforward"
             type="text"
             :placeholder="$t('new_forward')"
           />
diff --git a/pages/index.vue b/pages/index.vue
index ed9874d..537e301 100644
--- a/pages/index.vue
+++ b/pages/index.vue
@@ -1,6 +1,6 @@
 <script setup lang="ts">
 const { t } = useI18n()
-const userData = await useUserInfo()
+const { userData } = await useUserInfo()
 
 const me = computed(() => {
   const appTileColors = [