diff options
author | Runxi Yu <me@runxiyu.org> | 2025-03-05 09:32:40 +0800 |
---|---|---|
committer | Runxi Yu <me@runxiyu.org> | 2025-03-05 09:47:01 +0800 |
commit | 134b84f2672a9fe3e2e8a92b712261b47c4bd022 (patch) | |
tree | 5d666a509384f210a78b0e691d8f531b71b04875 /http_handle_repo_contrib_index.go | |
parent | *: Replace some := with var (diff) | |
download | forge-134b84f2672a9fe3e2e8a92b712261b47c4bd022.tar.gz forge-134b84f2672a9fe3e2e8a92b712261b47c4bd022.tar.zst forge-134b84f2672a9fe3e2e8a92b712261b47c4bd022.zip |
repo/*: Use var instead of :=
Diffstat (limited to 'http_handle_repo_contrib_index.go')
-rw-r--r-- | http_handle_repo_contrib_index.go | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/http_handle_repo_contrib_index.go b/http_handle_repo_contrib_index.go index 2e0dc1d..f352a3f 100644 --- a/http_handle_repo_contrib_index.go +++ b/http_handle_repo_contrib_index.go @@ -5,6 +5,8 @@ package main import ( "net/http" + + "github.com/jackc/pgx/v5" ) type id_title_status_t struct { @@ -14,24 +16,29 @@ type id_title_status_t struct { } func handle_repo_contrib_index(w http.ResponseWriter, r *http.Request, params map[string]any) { - rows, err := database.Query(r.Context(), "SELECT id, COALESCE(title, 'Untitled'), status FROM merge_requests WHERE repo_id = $1", params["repo_id"]) - if err != nil { + var rows pgx.Rows + var result []id_title_status_t + var err error + + if rows, err = database.Query(r.Context(), + "SELECT id, COALESCE(title, 'Untitled'), status FROM merge_requests WHERE repo_id = $1", + params["repo_id"], + ); err != nil { http.Error(w, "Error querying merge requests: "+err.Error(), http.StatusInternalServerError) return } defer rows.Close() - result := []id_title_status_t{} for rows.Next() { var id int var title, status string - if err := rows.Scan(&id, &title, &status); err != nil { + if err = rows.Scan(&id, &title, &status); err != nil { http.Error(w, "Error scanning merge request: "+err.Error(), http.StatusInternalServerError) return } result = append(result, id_title_status_t{id, title, status}) } - if err := rows.Err(); err != nil { + if err = rows.Err(); err != nil { http.Error(w, "Error ranging over merge requests: "+err.Error(), http.StatusInternalServerError) return } |