Bookhoard/bookhoard
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 12 Wiki Activity

fix(handlers): Correct database call parameters in processing issues handler

Browse Source 
Fix ResolveProcessingIssue and DeleteProcessingIssue methods to use proper
parameter structs instead of individual arguments.

Changes:
- ResolveProcessingIssue: Use database.ResolveProcessingIssueParams struct
  with ID and MediaItemID fields instead of separate arguments
- DeleteProcessingIssue: Wrap issueID in pgtype.UUID struct
- Use map[string]any instead of map[string]interface{} for JSON responses

These changes align with the sqlc-generated database interface and ensure
type-safe parameter passing to the database layer.
This commit is contained in:
john-okeefe
2026-04-13 09:24:01 -04:00
parent 12b07058bc
commit 67b3282831
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/handlers/processing_issues.go
+6 -4
View File
@@ -107,8 +107,10 @@ func (h *ProcessingIssuesHandler) ResolveProcessingIssue(c *echo.Context) error
_, err = h.db.ResolveProcessingIssue( _, err = h.db.ResolveProcessingIssue(
c.Request().Context(), c.Request().Context(),
issueID, database.ResolveProcessingIssueParams{
pgtype.UUID{Bytes: mediaItemID, Valid: true}, ID: pgtype.UUID{Bytes: issueID, Valid: true},
MediaItemID: pgtype.UUID{Bytes: mediaItemID, Valid: true},
},
) )
if err != nil { if err != nil {
return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Failed to resolve issue"}) return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Failed to resolve issue"})
@@ -127,12 +129,12 @@ func (h *ProcessingIssuesHandler) DeleteProcessingIssue(c *echo.Context) error {
return c.JSON(http.StatusBadRequest, map[string]string{"error": "Invalid issue ID"}) return c.JSON(http.StatusBadRequest, map[string]string{"error": "Invalid issue ID"})
} }
_, err = h.db.DeleteProcessingIssue(c.Request().Context(), issueID) _, err = h.db.DeleteProcessingIssue(c.Request().Context(), pgtype.UUID{Bytes: issueID, Valid: true})
if err != nil { if err != nil {
return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Failed to delete issue"}) return c.JSON(http.StatusInternalServerError, map[string]string{"error": "Failed to delete issue"})
} }
return c.JSON(http.StatusOK, map[string]interface{}{ return c.JSON(http.StatusOK, map[string]any{
"success": true, "success": true,
"message": "Issue deleted", "message": "Issue deleted",
}) })