wallet: properly calculate confirmations in GetTransaction

Fix bug where GetTransaction was setting confirmations to the
block height instead of calculating them correctly.

Author: mohamedawnallah

wallet/wallet.go

@@ -2601,14 +2601,20 @@ func (w *Wallet) GetTransaction(txHash chainhash.Hash) (*GetTransactionResult,
 		res = GetTransactionResult{
 			Summary:       makeTxSummary(dbtx, w, txDetail),
 			Timestamp:     txDetail.Block.Time.Unix(),
-			Confirmations: txDetail.Block.Height,
+			Confirmations: 0,
 		}
 
 		// If it is a confirmed transaction we set the corresponding
 		// block height and hash.
 		if txDetail.Block.Height != -1 {
 			res.Height = txDetail.Block.Height
 			res.BlockHash = &txDetail.Block.Hash
+
+			bestBlock := w.SyncedTo()
+			blockHeight := txDetail.Block.Height
+			res.Confirmations = calculateConfirmations(
+				blockHeight, bestBlock.Height,
+			)
 		}
 
 		return nil