From 21f87047ac013c22bd405de584d655ad9cdb76f8 Mon Sep 17 00:00:00 2001
From: jp1ac4 <121959000+jp1ac4@users.noreply.github.com>
Date: Tue, 6 Feb 2024 09:32:26 +0000
Subject: [PATCH] gui(transactions): filter coins by tx outpoints

---
 gui/src/app/state/transactions.rs | 23 ++++++++++++-----------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/gui/src/app/state/transactions.rs b/gui/src/app/state/transactions.rs
index 6737471c..3f2f84ca 100644
--- a/gui/src/app/state/transactions.rs
+++ b/gui/src/app/state/transactions.rs
@@ -7,6 +7,7 @@ use std::{
 
 use iced::Command;
 use liana::{
+    commands::CoinStatus,
     miniscript::bitcoin::{OutPoint, Txid},
     spend::{SpendCreationError, MAX_FEERATE},
 };
@@ -146,23 +147,23 @@ impl State for TransactionsPanel {
                 if let Some(tx) = &self.selected_tx {
                     if tx.fee_amount.is_some() {
                         let tx = tx.clone();
-                        let txid = tx.tx.txid();
+                        let outpoints: Vec<_> = (0..tx.tx.output.len())
+                            .map(|vout| {
+                                OutPoint::new(
+                                    tx.tx.txid(),
+                                    vout.try_into()
+                                        .expect("number of transaction outputs must fit in u32"),
+                                )
+                            })
+                            .collect();
                         return Command::perform(
                             async move {
                                 daemon
-                                    // TODO: filter for spending coins when this is possible:
-                                    // https://github.com/wizardsardine/liana/issues/677
-                                    .list_coins(&[], &[])
+                                    .list_coins(&[CoinStatus::Spending], &outpoints)
                                     .map(|res| {
                                         res.coins
                                             .iter()
-                                            .filter_map(|c| {
-                                                if c.outpoint.txid == txid {
-                                                    c.spend_info.map(|info| info.txid)
-                                                } else {
-                                                    None
-                                                }
-                                            })
+                                            .filter_map(|c| c.spend_info.map(|info| info.txid))
                                             .collect()
                                     })
                                     .map_err(|e| e.into())