diff --git a/backend/internal/web/handlers_tasks.go b/backend/internal/web/handlers_tasks.go index 78977ff..8786bba 100644 --- a/backend/internal/web/handlers_tasks.go +++ b/backend/internal/web/handlers_tasks.go @@ -445,13 +445,18 @@ func TaskReorderHandler(deps TasksDeps) http.HandlerFunc { if _, scanErr := fmt.Sscanf(positionVal, "%d", &newPos); scanErr != nil || newPos <= 0 { newPos = 100 } - _, _ = deps.Queries.UpdateTask(ctx, sqlc.UpdateTaskParams{ + if _, err := deps.Queries.UpdateTask(ctx, sqlc.UpdateTaskParams{ ID: existing.ID, Title: existing.Title, Description: existing.Description, Status: newStatus, Position: newPos, - }) + }); err != nil { + slog.Default().Error("tasks reorder: UpdateTask failed (single)", + "task_id", existing.ID, "err", err) + http.Error(w, "internal server error", http.StatusInternalServerError) + return + } } } tasks, _ := deps.Queries.ListTasksByTablo(ctx, tablo.ID) @@ -486,13 +491,18 @@ func TaskReorderHandler(deps TasksDeps) http.HandlerFunc { } // Preserve title+description; only update status+position (T-04-08). - _, _ = deps.Queries.UpdateTask(ctx, sqlc.UpdateTaskParams{ + if _, err := deps.Queries.UpdateTask(ctx, sqlc.UpdateTaskParams{ ID: existing.ID, Title: existing.Title, Description: existing.Description, Status: newStatus, Position: newPos, - }) + }); err != nil { + slog.Default().Error("tasks reorder: UpdateTask failed", + "task_id", existing.ID, "err", err) + http.Error(w, "internal server error", http.StatusInternalServerError) + return + } } tasks, err := deps.Queries.ListTasksByTablo(ctx, tablo.ID)