[taler-exchange] branch master updated: remove select-* logics'

[gnunet]

This is an automated email from the git hooks/post-receive script.

grothoff pushed a commit to branch master
in repository exchange.

The following commit(s) were added to refs/heads/master by this push:
     new 4e380d49 remove select-* logics'
4e380d49 is described below

commit 4e380d4937da4d51809d57eb60d4b6f0650483a9
Author: Christian Grothoff <grothoff@gnunet.org>
AuthorDate: Tue Oct 17 10:24:18 2023 +0200

    remove select-* logics'
---
 src/exchangedb/exchange_do_age_withdraw.sql      | 5 +++--
 src/exchangedb/exchange_do_batch_withdraw.sql    | 6 ++++--
 src/exchangedb/exchange_do_recoup_by_reserve.sql | 9 +++++++--
 src/exchangedb/exchange_do_reserve_open.sql      | 6 +++---
 4 files changed, 17 insertions(+), 9 deletions(-)

diff --git a/src/exchangedb/exchange_do_age_withdraw.sql 
b/src/exchangedb/exchange_do_age_withdraw.sql
index c696bd80..89a29144 100644
--- a/src/exchangedb/exchange_do_age_withdraw.sql
+++ b/src/exchangedb/exchange_do_age_withdraw.sql
@@ -49,8 +49,9 @@ BEGIN
 --         reserves_in by reserve_pub (SELECT)
 --         wire_targets by wire_target_h_payto
 
--- FIXME-Oec: never select-*!
-SELECT *
+SELECT current_balance
+      ,birthday
+      ,gc_date
   INTO reserve
   FROM exchange.reserves
  WHERE reserves.reserve_pub=rpub;
diff --git a/src/exchangedb/exchange_do_batch_withdraw.sql 
b/src/exchangedb/exchange_do_batch_withdraw.sql
index a27f2348..a48561a9 100644
--- a/src/exchangedb/exchange_do_batch_withdraw.sql
+++ b/src/exchangedb/exchange_do_batch_withdraw.sql
@@ -41,8 +41,10 @@ BEGIN
 --         reserves_in by reserve_pub (SELECT)
 --         wire_targets by wire_target_h_payto
 
--- FIXME-Oec: do not use select-*!
-SELECT *
+SELECT current_balance
+      ,reserve_uuid
+      ,birthday
+      ,gc_date
   INTO reserve
   FROM exchange.reserves
  WHERE reserves.reserve_pub=rpub;
diff --git a/src/exchangedb/exchange_do_recoup_by_reserve.sql 
b/src/exchangedb/exchange_do_recoup_by_reserve.sql
index a4a191a4..01682661 100644
--- a/src/exchangedb/exchange_do_recoup_by_reserve.sql
+++ b/src/exchangedb/exchange_do_recoup_by_reserve.sql
@@ -62,12 +62,17 @@ BEGIN
              rc.amount,
              rc.recoup_timestamp
       FROM (
-        SELECT * --FIXME: bad code, we never use '*'!
+        SELECT denom_sig
+              ,denominations_serial
         FROM exchange.known_coins
         WHERE known_coins.coin_pub = c_pub
       ) kc
       JOIN (
-        SELECT *  --FIXME: bad code, we never use '*'!
+        SELECT coin_pub
+              ,coin_sig
+              ,coin_blind
+              ,amount
+              ,recoup_timestamp
         FROM exchange.recoup
         WHERE recoup.coin_pub = c_pub
       ) rc USING (coin_pub);
diff --git a/src/exchangedb/exchange_do_reserve_open.sql 
b/src/exchangedb/exchange_do_reserve_open.sql
index 5d36d1af..f37129e5 100644
--- a/src/exchangedb/exchange_do_reserve_open.sql
+++ b/src/exchangedb/exchange_do_reserve_open.sql
@@ -43,9 +43,9 @@ DECLARE
   reserve RECORD;
 BEGIN
 
--- FIXME: do not use SELECT-*
--- FIXME: use SELECT FOR UPDATE?
-SELECT *
+SELECT current_balance
+      ,expiration_time
+      ,purses_allowed
   INTO reserve
   FROM reserves
  WHERE reserve_pub=in_reserve_pub;

-- 
To stop receiving notification emails like this one, please contact
gnunet@gnunet.org.