refactor: follow-up dash#6742, implement review suggestions, avoid redundant transaction queries - #6770

Potential conflicts detected!

{ pull_number: 6770, conflictPrs: [ { number: 6761, files: [ 'src/Makefile.am', 'src/bench/rpc_blockchain.cpp', 'src/chainlock/chainlock.cpp', 'src/chainlock/chainlock.h', 'src/chainlock/clsig.cpp', 'src/chainlock/clsig.h', 'src/chainlock/signing.cpp', 'src/chainlock/signing.h', 'src/coinjoin/coinjoin.cpp', 'src/dsnotificationinterface.cpp', 'src/dsnotificationinterface.h', 'src/evo/chainhelper.cpp', 'src/evo/specialtxman.cpp', 'src/instantsend/instantsend.cpp', 'src/instantsend/signing.cpp', 'src/interfaces/chain.h', 'src/llmq/clsig.cpp', 'src/llmq/clsig.h', 'src/llmq/context.cpp', 'src/net_processing.cpp', 'src/node/interfaces.cpp', 'src/node/miner.cpp', 'src/rest.cpp', 'src/rpc/blockchain.cpp', 'src/rpc/mining.cpp', 'src/rpc/quorums.cpp', 'src/rpc/rawtransaction.cpp', 'src/test/fuzz/process_message.cpp', 'src/test/miner_tests.cpp', 'src/validation.cpp', 'src/validationinterface.cpp', 'src/validationinterface.h', 'src/wallet/rpc/wallet.cpp', 'src/wallet/wallet.cpp', 'src/wallet/wallet.h', 'src/zmq/zmqabstractnotifier.cpp', 'src/zmq/zmqabstractnotifier.h', 'src/zmq/zmqnotificationinterface.cpp', 'src/zmq/zmqnotificationinterface.h', 'src/zmq/zmqpublishnotifier.cpp', 'src/zmq/zmqpublishnotifier.h', 'test/lint/lint-circular-dependencies.py', 'test/util/data/non-backported.txt' ], conflicts: [ 'src/dsnotificationinterface.cpp', 'src/instantsend/instantsend.cpp', 'src/instantsend/signing.cpp', 'test/lint/lint-circular-dependencies.py' ] } ] }