kodev-org
/
vikunja
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

refactor(attachments): return uploaded attachments instead of writing to store 

uploadFiles now returns the array of uploaded IAttachment objects
so callers can handle state updates themselves.
This commit is contained in:
kolaente 2026-03-19 17:31:00 +01:00
parent 5dbc906d47
commit eaec206301
1 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
frontend/src/helpers/attachments.ts
View File

@ -2,9 +2,8 @@ import AttachmentModel from '@/models/attachment'
import type {IAttachment} from '@/modelTypes/IAttachment'
import AttachmentService from '@/services/attachment'
import {useTaskStore} from '@/stores/tasks'
export async function uploadFile(taskId: number, file: File, onSuccess?: (url: string) => void) {
export async function uploadFile(taskId: number, file: File, onSuccess?: (url: string) => void): Promise<IAttachment[]> {
const attachmentService = new AttachmentService()
const files = [file]
@ -16,16 +15,14 @@ export async function uploadFiles(
taskId: number,
files: File[] | FileList,
onSuccess?: (attachmentUrl: string) => void,
) {
): Promise<IAttachment[]> {
const attachmentModel = new AttachmentModel({taskId})
const response = await attachmentService.create(attachmentModel, files)
console.debug(`Uploaded attachments for task ${taskId}, response was`, response)
const uploaded: IAttachment[] = []
response.success?.map((attachment: IAttachment) => {
useTaskStore().addTaskAttachment({
taskId,
attachment,
})
uploaded.push(attachment)
onSuccess?.(generateAttachmentUrl(taskId, attachment.id))
})
@ -33,6 +30,8 @@ export async function uploadFiles(
const messages = response.errors.map((e: {message: string}) => e.message)
throw new Error(messages.join('\n'))
}
return uploaded
}
export function generateAttachmentUrl(taskId: number, attachmentId: number) {