refactor(FE-63): simplify createMovementHandler by removing unnecessary payload checks

This commit is contained in:
rstubryan
2025-10-17 10:13:14 +07:00
parent caac9c20e6
commit cfb9b53b54
@@ -17,23 +17,16 @@ export const useMovementFormHandlers = (initialValuesId?: number) => {
const createMovementHandler = useCallback( const createMovementHandler = useCallback(
async (payload: CreateMovementPayload, documents: File[] = []) => { async (payload: CreateMovementPayload, documents: File[] = []) => {
let finalPayload: CreateMovementPayload | FormData;
if (documents.length > 0) {
const formData = new FormData(); const formData = new FormData();
formData.append('data', JSON.stringify(payload)); formData.append('data', JSON.stringify(payload));
documents.forEach((file, index) => { documents.forEach((file, index) => {
formData.append(`documents[${index}]`, file); formData.append(`documents[${index}]`, file);
}); });
finalPayload = formData as unknown as CreateMovementPayload; const res = await MovementApi.create(
} else { formData as unknown as CreateMovementPayload
finalPayload = payload; );
}
const res = await MovementApi.create(finalPayload);
if (isResponseError(res)) { if (isResponseError(res)) {
console.error('API Error:', res);
setMovementFormErrorMessage(res.message); setMovementFormErrorMessage(res.message);
return; return;
} }