tapgarden: remove batch key argument from fundGenesisPsbt

The batch key was only used for logging. This commit moves the log
messages outside fundGenesisPsbt, simplifying the function for
better code health.

Author: ffranr

tapgarden/planter.go

@@ -937,12 +937,10 @@ type WalletFundPsbt = func(ctx context.Context,
 // (all outputs need to hold some BTC to not be dust), and with a dummy script.
 // We need to use a dummy script as we can't know the actual script key since
 // that's dependent on the genesis outpoint.
-func fundGenesisPsbt(ctx context.Context,
-	pendingBatch *MintingBatch, batchKey asset.SerializedKey,
+func fundGenesisPsbt(ctx context.Context, pendingBatch *MintingBatch,
 	walletFundPsbt WalletFundPsbt) (FundedMintAnchorPsbt, error) {
 
 	var zero FundedMintAnchorPsbt
-	log.Infof("Attempting to fund batch: %x", batchKey)
 
 	// If universe commitments are enabled, we formulate a pre-commitment
 	// output. This output is spent by the universe commitment transaction.
@@ -994,7 +992,6 @@ func fundGenesisPsbt(ctx context.Context,
 			"funded anchor transaction")
 	}
 
-	log.Infof("Funded GenesisPacket for batch: %x", batchKey)
 	log.Tracef("GenesisPacket: %v", spew.Sdump(fundedGenesisPkt))
 
 	// Classify anchor transaction output indexes.
@@ -2044,14 +2041,16 @@ func (c *ChainPlanter) fundBatch(ctx context.Context, params FundParams,
 			return *fundedPkt, nil
 		}
 
+		log.Infof("Attempting to fund batch: %x", batchKey)
 		mintAnchorTx, err := fundGenesisPsbt(
-			ctx, c.pendingBatch, batchKey, walletFundPsbt,
+			ctx, c.pendingBatch, walletFundPsbt,
 		)
 		if err != nil {
 			return fmt.Errorf("unable to fund minting PSBT for "+
 				"batch: %x %w", batchKey[:], err)
 		}
 
+		log.Infof("Funded GenesisPacket for batch: %x", batchKey)
 		batch.GenesisPacket = &mintAnchorTx
 
 		return nil