From eed1ef37dc4d6007f2441a859ab37fb436d9ea97 Mon Sep 17 00:00:00 2001 From: John O'Keefe Date: Mon, 9 Feb 2026 15:46:57 -0500 Subject: [PATCH] Change UpdateUserMaxDevices to return updated user record - Change query from :exec to :one with RETURNING * - Allows handler to detect when user doesn't exist - Follows pattern established by UpdateMediaItem - Required for 404 response on non-existent user Related: Fix for TestUpdateUserMaxDevicesNonExistentUser --- internal/database/queries/queries.sql | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/internal/database/queries/queries.sql b/internal/database/queries/queries.sql index d36e5f7..a9fb1ef 100644 --- a/internal/database/queries/queries.sql +++ b/internal/database/queries/queries.sql @@ -307,8 +307,9 @@ UPDATE users SET email = $2, updated_at = NOW() WHERE id = $1; -- name: UpdatePassword :exec UPDATE users SET password_hash = $2, updated_at = NOW() WHERE id = $1; --- name: UpdateUserMaxDevices :exec -UPDATE users SET max_devices = $2, updated_at = NOW() WHERE id = $1; +-- name: UpdateUserMaxDevices :one +UPDATE users SET max_devices = $2, updated_at = NOW() WHERE id = $1 +RETURNING *; -- name: CountUserDevices :one SELECT COUNT(*) FROM devices WHERE user_id = $1;