multi: set linkTemplate txntype correctly and refactor

This commit fixes a bug where the int value of treasury tx types was used
to construct page links instead of its string value. As a result, page navigation
on the treasury page did not function correctly because the string value is expected
to return correct table values.

Others:
- Replace duplicated code used in parsing URL params in *explorerUI.TreasuryPage with
  improved parseTreasuryParams helper function.
- Remove unused code from when *explorerUI.AddressPage was used to display treasury data.
- Remove resolved TODO and refactor how treasury mempool data is passed to the HTML template.


Signed-off-by: Philemon Ukane <ukanephilemon@gmail.com>

Author: ukane-philemon

cmd/dcrdata/internal/explorer/explorerroutes.go

@@ -1,4 +1,4 @@
-// Copyright (c) 2018-2024, The Decred developers
+// Copyright (c) 2018-2025, The Decred developers
 // Copyright (c) 2017, The dcrdata developers
 // See LICENSE for details.
 
@@ -1408,62 +1408,40 @@ type TreasuryInfo struct {
 	Path          string
 	Limit, Offset int64  // ?n=Limit&start=Offset
 	TxnType       string // ?txntype=TxnType
-
-	// TODO: tadd and tspend can be unconfirmed. tspend for a very long time.
-	// NumUnconfirmed is the number of unconfirmed txns
-	// NumUnconfirmed  int64
-	// UnconfirmedTxns []*dbtypes.TreasuryTx
-
 	// Transactions on the current page
 	Transactions    []*dbtypes.TreasuryTx
 	NumTransactions int64 // len(Transactions) but int64 for dumb template
 
 	Balance          *dbtypes.TreasuryBalance
 	ConvertedBalance *exchanges.Conversion
 	TypeCount        int64
+
+	// tadd and tspend can be unconfirmed. tspend for a very long time.
+	Mempool *TreasuryMempoolInfo
+}
+
+// TreasuryMempoolInfo holds the treasury-related mempool transactions that are
+// not yet confirmed in a block. It is used to display treasury mempool
+// information on the treasury page.
+type TreasuryMempoolInfo struct {
+	NumTSpends int
+	NumTAdds   int
+	TSpends    []types.MempoolTx
+	TAdds      []types.MempoolTx
 }
 
 // TreasuryPage is the page handler for the "/treasury" path
 func (exp *explorerUI) TreasuryPage(w http.ResponseWriter, r *http.Request) {
-	ctx := context.WithValue(r.Context(), ctxAddress, exp.pageData.HomeInfo.DevAddress)
-	r = r.WithContext(ctx)
-	if queryVals := r.URL.Query(); queryVals.Get("txntype") == "" {
-		queryVals.Set("txntype", "tspend")
-		r.URL.RawQuery = queryVals.Encode()
-	}
-
-	limitN := defaultAddressRows
-	if nParam := r.URL.Query().Get("n"); nParam != "" {
-		val, err := strconv.ParseUint(nParam, 10, 64)
-		if err != nil {
-			exp.StatusPage(w, defaultErrorCode, "invalid n value", "", ExpStatusError)
-			return
-		}
-		if int64(val) > MaxTreasuryRows {
-			log.Warnf("TreasuryPage: requested up to %d address rows, "+
-				"limiting to %d", limitN, MaxTreasuryRows)
-			limitN = MaxTreasuryRows
-		} else {
-			limitN = int64(val)
-		}
-	}
+	ctx := r.Context()
 
-	// Number of txns to skip (OFFSET in database query). For UX reasons, the
-	// "start" URL query parameter is used.
-	var offset int64
-	if startParam := r.URL.Query().Get("start"); startParam != "" {
-		val, err := strconv.ParseUint(startParam, 10, 64)
-		if err != nil {
-			exp.StatusPage(w, defaultErrorCode, "invalid start value", "", ExpStatusError)
-			return
-		}
-		offset = int64(val)
+	// Grab the URL query parameters
+	txType, txTypeStr, limitN, offset, err := parseTreasuryParams(r)
+	if err != nil {
+		log.Errorf("TreasuryPage request error: %v", err)
+		http.Error(w, http.StatusText(http.StatusBadRequest), http.StatusBadRequest)
+		return
 	}
 
-	// Transaction types to show.
-	txTypeStr := r.URL.Query().Get("txntype")
-	txType := parseTreasuryTransactionType(txTypeStr)
-
 	txns, err := exp.dataSource.TreasuryTxns(ctx, limitN, offset, txType)
 	if exp.timeoutErrorPage(w, err, "TreasuryTxns") {
 		return
@@ -1477,6 +1455,7 @@ func (exp *explorerUI) TreasuryPage(w http.ResponseWriter, r *http.Request) {
 	exp.pageData.RUnlock()
 
 	typeCount := treasuryTypeCount(treasuryBalance, txType)
+	inv := exp.MempoolInventory()
 
 	treasuryData := &TreasuryInfo{
 		Net:             exp.ChainParams.Net.String(),
@@ -1489,6 +1468,12 @@ func (exp *explorerUI) TreasuryPage(w http.ResponseWriter, r *http.Request) {
 		Transactions:    txns,
 		Balance:         treasuryBalance,
 		TypeCount:       typeCount,
+		Mempool: &TreasuryMempoolInfo{
+			NumTSpends: inv.NumTSpends,
+			NumTAdds:   inv.NumTAdds,
+			TSpends:    inv.TSpends,
+			TAdds:      inv.TAdds,
+		},
 	}
 
 	xcBot := exp.xcBot
@@ -1497,7 +1482,7 @@ func (exp *explorerUI) TreasuryPage(w http.ResponseWriter, r *http.Request) {
 	}
 
 	// Execute the HTML template.
-	linkTemplate := fmt.Sprintf("/treasury?start=%%d&n=%d&txntype=%v", limitN, txType)
+	linkTemplate := fmt.Sprintf("/treasury?start=%%d&n=%d&txntype=%s", limitN, txTypeStr)
 	pageData := struct {
 		*CommonPageData
 		Data        *TreasuryInfo
@@ -1704,7 +1689,7 @@ func (exp *explorerUI) TreasuryTable(w http.ResponseWriter, r *http.Request) {
 	ctx := r.Context()
 
 	// Grab the URL query parameters
-	txType, limitN, offset, err := parseTreasuryParams(r)
+	txType, txTypeStr, limitN, offset, err := parseTreasuryParams(r)
 	if err != nil {
 		log.Errorf("TreasuryTable request error: %v", err)
 		http.Error(w, http.StatusText(http.StatusBadRequest), http.StatusBadRequest)
@@ -1723,7 +1708,7 @@ func (exp *explorerUI) TreasuryTable(w http.ResponseWriter, r *http.Request) {
 	bal := exp.pageData.HomeInfo.TreasuryBalance
 	exp.pageData.RUnlock()
 
-	linkTemplate := "/treasury" + "?start=%d&n=" + strconv.FormatInt(limitN, 10) + "&txntype=" + fmt.Sprintf("%v", txType)
+	linkTemplate := "/treasury" + "?start=%d&n=" + strconv.FormatInt(limitN, 10) + "&txntype=" + fmt.Sprintf("%s", txTypeStr)
 
 	response := struct {
 		TxnCount int64        `json:"tx_count"`
@@ -1816,9 +1801,9 @@ func parseTreasuryTransactionType(txnTypeStr string) (txType stake.TxType) {
 
 // parseTreasuryParams parses the tx filters for the treasury page. Used by both
 // TreasuryPage and TreasuryTable.
-func parseTreasuryParams(r *http.Request) (txType stake.TxType, limitN, offsetAddrOuts int64, err error) {
-	tType, limitN, offsetAddrOuts, err := parsePaginationParams(r)
-	txType = parseTreasuryTransactionType(tType)
+func parseTreasuryParams(r *http.Request) (txType stake.TxType, txTypeStr string, limitN, offsetAddrOuts int64, err error) {
+	txTypeStr, limitN, offsetAddrOuts, err = parsePaginationParams(r)
+	txType = parseTreasuryTransactionType(txTypeStr)
 	return
 }
 
@@ -1830,7 +1815,6 @@ func parsePaginationParams(r *http.Request) (txnType string, limitN, offset int6
 	limitN = defaultAddressRows
 
 	if nParam := r.URL.Query().Get("n"); nParam != "" {
-
 		var val uint64
 		val, err = strconv.ParseUint(nParam, 10, 64)
 		if err != nil {
@@ -1859,9 +1843,8 @@ func parsePaginationParams(r *http.Request) (txnType string, limitN, offset int6
 	}
 
 	// Transaction types to show.
-	txnType = r.URL.Query().Get("txntype")
-	if txnType == "" {
-		txnType = "all"
+	if txnType = r.URL.Query().Get("txntype"); txnType == "" {
+		txnType = "tspend" // Default to tspend
 	}
 
 	return

cmd/dcrdata/views/treasury.tmpl

@@ -3,7 +3,6 @@
 <html lang="en">
 {{template "html-head" headData .CommonPageData "Decred Decentralized Treasury"}}
   {{template "navbar" . }}
-  {{- $mempool := .Mempool -}}
   {{- with .Data}}
   {{- $bal := .Balance -}}
   {{- $TxnCount := $bal.TxCount}}
@@ -158,8 +157,8 @@
               </tr>
             </thead>
             <tbody>
-              {{if gt $mempool.NumTSpends 0 -}}
-              {{- range $mempool.TSpends -}}
+              {{if gt .Mempool.NumTSpends 0 -}}
+              {{- range .Mempool.TSpends -}}
               <tr>
                 <td class="break-word clipboard">
                   <a class="hash lh1rem" href="/tx/{{.Hash}}" title="{{.Hash}}">{{.Hash}}</a>
@@ -180,7 +179,7 @@
           </table>
         </div>
       </div>
-      {{if gt $mempool.NumTAdds 0 -}}{{- /* this will be rare, so only show the section header and table if needed */ -}}
+      {{if gt .Mempool.NumTAdds 0 -}}{{- /* this will be rare, so only show the section header and table if needed */ -}}
       <div class="row">
         <div class="col-sm-24">
           <div class="me-auto h4 col-24">Unconfirmed Treasury Adds</div>
@@ -193,7 +192,7 @@
               </tr>
             </thead>
             <tbody>
-              {{range $mempool.TAdds -}}
+              {{range .Mempool.TAdds -}}
               <tr>
                 <td class="break-word clipboard">
                   <a class="hash lh1rem" href="/tx/{{.Hash}}">{{.Hash}}</a>