From cab1dc5c290d657c99dad2e094834c8440083d28 Mon Sep 17 00:00:00 2001
From: SomberNight <somber.night@protonmail.com>
Date: Fri, 14 Mar 2025 17:03:49 +0000
Subject: [PATCH] wallet: towards killing create_transaction: pass through
 "locktime", "version"

---
 electrum/commands.py |  6 ++++--
 electrum/wallet.py   | 16 ++++++++++------
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/electrum/commands.py b/electrum/commands.py
index dab371749..50c57b8dc 100644
--- a/electrum/commands.py
+++ b/electrum/commands.py
@@ -793,7 +793,8 @@ class Commands(Logger):
             domain_addr=domain_addr,
             domain_coins=domain_coins,
             rbf=rbf,
-            locktime=locktime)
+            locktime=locktime,
+        )
         if not unsigned:
             wallet.sign_transaction(tx, password)
         result = tx.serialize()
@@ -823,7 +824,8 @@ class Commands(Logger):
             domain_addr=domain_addr,
             domain_coins=domain_coins,
             rbf=rbf,
-            locktime=locktime)
+            locktime=locktime,
+        )
         if not unsigned:
             wallet.sign_transaction(tx, password)
         result = tx.serialize()
diff --git a/electrum/wallet.py b/electrum/wallet.py
index 5f8f4a0c1..d88dd3b6b 100644
--- a/electrum/wallet.py
+++ b/electrum/wallet.py
@@ -1841,6 +1841,8 @@ class Abstract_Wallet(ABC, Logger, EventListener):
             base_tx: Optional[Transaction] = None,
             send_change_to_lightning: bool = False,
             merge_duplicate_outputs: bool = False,
+            locktime: Optional[int] = None,
+            tx_version: Optional[int] = None,
     ) -> PartialTransaction:
         """Can raise NotEnoughFunds or NoDynamicFeeEstimates."""
 
@@ -1953,8 +1955,12 @@ class Abstract_Wallet(ABC, Logger, EventListener):
             outputs[i].value += (amount - distr_amount)
             tx = PartialTransaction.from_io(list(coins), list(outputs))
 
-        # Timelock tx to current height.
-        tx.locktime = get_locktime_for_new_transaction(self.network)
+        if locktime is None:
+            # Timelock tx to current height.
+            locktime = get_locktime_for_new_transaction(self.network)
+        tx.locktime = locktime
+        if tx_version is not None:
+            tx.version = tx_version
         tx.rbf_merge_txid = rbf_merge_txid
         tx.add_info_from_wallet(self)
         run_hook('make_unsigned_transaction', self, tx)
@@ -3132,11 +3138,9 @@ class Abstract_Wallet(ABC, Logger, EventListener):
             merge_duplicate_outputs=merge_duplicate_outputs,
             rbf=rbf,
             BIP69_sort=BIP69_sort,
+            locktime=locktime,
+            tx_version=tx_version,
         )
-        if locktime is not None:
-            tx.locktime = locktime
-        if tx_version is not None:
-            tx.version = tx_version
         return tx
 
     def _check_risk_of_burning_coins_as_fees(self, tx: 'PartialTransaction') -> TxSighashDanger: