fix: some update sample revisions

Author: HotPotatoC

internal/api/update_sample.go

@@ -45,10 +45,9 @@ func (i UpdateSampleInput) Validate() error {
 }
 
 func (h *handler) UpdateSample(c *gin.Context) {
-
-	sampleID, err := strconv.Atoi(c.Param("id"))
 	var input UpdateSampleInput
 
+	sampleID, err := strconv.Atoi(c.Param("id"))
 	if err != nil {
 		c.JSON(400, M{
 			"message": "invalid sample id",

internal/backend/update_sample.go

@@ -4,10 +4,9 @@ import (
 	"context"
 	"errors"
 	"io"
-	"strconv"
 
-	"github.com/gin-gonic/gin"
 	"github.com/tunema-org/sound-function/internal/jwt"
+	"github.com/tunema-org/sound-function/internal/repository"
 	"github.com/tunema-org/sound-function/model"
 )
 
@@ -23,13 +22,7 @@ type UpdateSampleParams struct {
 	TagIDs        []int
 }
 
-
-func (b *Backend) UpdateSample(ctx context.Context, accessToken string, sampleID params UpdateSampleParams) (int, error) {
-	sampleID, err := strconv.Atoi(c.Param("id"))
-	if err != nil {
-		return 0, err
-	}
-
+func (b *Backend) UpdateSample(ctx context.Context, accessToken string, sampleID int, params UpdateSampleParams) (int, error) {
 	_, claims, err := jwt.Verify(accessToken, b.cfg.JWTSecretKey)
 	if err != nil {
 		return 0, err
@@ -40,15 +33,17 @@ func (b *Backend) UpdateSample(ctx context.Context, accessToken string, sampleID
 		return 0, errors.New("invalid claims")
 	}
 
-	//msih error
-	err := b.repo.UpdateSample(ctx, int(sampleID), model.Sample{
-		UserID:   int(userID),
-		Name:     params.Name,
-		BPM:      params.BPM,
-		Key:      params.Key,
-		KeyScale: params.KeyScale,
-		Time:     params.Time,
-		Price:    params.Price,
+	err = b.repo.UpdateSample(ctx, sampleID, repository.UpdateSampleParams{
+		Sample: model.Sample{
+			UserID:   int(userID),
+			Name:     params.Name,
+			BPM:      params.BPM,
+			Key:      params.Key,
+			KeyScale: params.KeyScale,
+			Time:     params.Time,
+			Price:    params.Price,
+		},
+		TagIDs: params.TagIDs,
 	})
 	if err != nil {
 		return 0, err

internal/repository/update_sample.go

@@ -2,39 +2,36 @@ package repository
 
 import (
 	"context"
-	"fmt"
 
 	"github.com/jackc/pgx/v5"
 	"github.com/tunema-org/sound-function/model"
 )
 
 type UpdateSampleParams struct {
 	Sample model.Sample
+	TagIDs []int
 }
 
-func (r *Repository) UpdateSample(ctx context.Context, params UpdateSampleParams) (int, error) {
+func (r *Repository) UpdateSample(ctx context.Context, sampleID int, params UpdateSampleParams) error {
 	tx, err := r.db.BeginTx(ctx, pgx.TxOptions{})
 	if err != nil {
-		return 0, err
+		return err
 	}
 
 	updateSampleQuery := `UPDATE samples SET name = $1, bpm = $2, key = $4, key_scale = $5, time = $6, price = $7 WHERE id = $8`
 
-	var SampleID int
-	err = tx.QueryRow(ctx, updateSampleQuery,
+	_, err = tx.Exec(ctx, updateSampleQuery,
 		params.Sample.Name,
 		params.Sample.BPM,
 		params.Sample.Key,
 		params.Sample.KeyScale,
 		params.Sample.Time,
 		params.Sample.Price,
-	).Scan(&SampleID)
+	)
 	if err != nil {
 		tx.Rollback(ctx)
-		return 0, err
+		return err
 	}
 
-	fmt.Println("Sample Updated")
-
-	return 0, err
+	return tx.Commit(ctx)
 }