From cfb9b53b5444461a29425869c111d3ce3043ce6f Mon Sep 17 00:00:00 2001 From: rstubryan Date: Fri, 17 Oct 2025 10:13:14 +0700 Subject: [PATCH] refactor(FE-63): simplify createMovementHandler by removing unnecessary payload checks --- .../movement/form/useMovementFormHandlers.ts | 23 +++++++------------ 1 file changed, 8 insertions(+), 15 deletions(-) diff --git a/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts index 5c3d80d1..0ad31e38 100644 --- a/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts +++ b/src/components/pages/inventory/movement/form/useMovementFormHandlers.ts @@ -17,23 +17,16 @@ export const useMovementFormHandlers = (initialValuesId?: number) => { const createMovementHandler = useCallback( async (payload: CreateMovementPayload, documents: File[] = []) => { - let finalPayload: CreateMovementPayload | FormData; + const formData = new FormData(); + formData.append('data', JSON.stringify(payload)); + documents.forEach((file, index) => { + formData.append(`documents[${index}]`, file); + }); - if (documents.length > 0) { - const formData = new FormData(); - formData.append('data', JSON.stringify(payload)); - documents.forEach((file, index) => { - formData.append(`documents[${index}]`, file); - }); - - finalPayload = formData as unknown as CreateMovementPayload; - } else { - finalPayload = payload; - } - - const res = await MovementApi.create(finalPayload); + const res = await MovementApi.create( + formData as unknown as CreateMovementPayload + ); if (isResponseError(res)) { - console.error('API Error:', res); setMovementFormErrorMessage(res.message); return; }