patrick/sojorn
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Fix ToggleReaction error handling - use pgx.ErrNoRows instead of sql.ErrNoRows

Browse source 
pgx returns pgx.ErrNoRows not sql.ErrNoRows for no rows result set, causing toggle reaction to fail
This commit is contained in:
Patrick Britton 2026-02-01 12:54:08 -06:00
parent 8c3ca59872
commit a3c609c0cc
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
go-backend/internal/repository/post_repository.go
View file

@ -2,10 +2,10 @@ package repository
import ( import (
"context" "context"
"database/sql"
"fmt" "fmt"
"time" "time"
"github.com/jackc/pgx/v5"
"github.com/jackc/pgx/v5/pgxpool" "github.com/jackc/pgx/v5/pgxpool"
"github.com/patbritton/sojorn-backend/internal/models" "github.com/patbritton/sojorn-backend/internal/models"
"github.com/rs/zerolog/log" "github.com/rs/zerolog/log"
@ -1094,7 +1094,7 @@ func (r *PostRepository) ToggleReaction(ctx context.Context, postID string, user
// Still need to calculate and return counts // Still need to calculate and return counts
goto calculate_counts goto calculate_counts
} }
} else if err != sql.ErrNoRows { } else if err != pgx.ErrNoRows {
log.Error().Err(err).Str("postID", postID).Str("userID", userID).Msg("DEBUG: Failed to check existing reaction - unexpected error") log.Error().Err(err).Str("postID", postID).Str("userID", userID).Msg("DEBUG: Failed to check existing reaction - unexpected error")
return nil, nil, fmt.Errorf("failed to check existing reaction: %w", err) return nil, nil, fmt.Errorf("failed to check existing reaction: %w", err)
} else { } else {