Lindenii Project Forge
Commit info | |
---|---|
ID | 953d8195ba55832c9d880c08ae6e062a362c14c3 |
Author | Runxi Yu<me@runxiyu.org> |
Author date | Fri, 07 Mar 2025 08:02:44 +0800 |
Committer | Runxi Yu<me@runxiyu.org> |
Committer date | Fri, 07 Mar 2025 08:02:44 +0800 |
Actions | Get patch |
group/index: Fix repo_desc in creation form handler
// SPDX-License-Identifier: AGPL-3.0-only // SPDX-FileContributor: Runxi Yu <https://runxiyu.org> package main import ( "net/http" "path/filepath" "strconv" "github.com/jackc/pgx/v5" "github.com/jackc/pgx/v5/pgtype" ) func handle_group_index(w http.ResponseWriter, r *http.Request, params map[string]any) { var group_path []string var repos []name_desc_t var subgroups []name_desc_t var err error var group_id int var group_description string group_path = params["group_path"].([]string) // The group itself err = database.QueryRow(r.Context(), ` WITH RECURSIVE group_path_cte AS ( SELECT id, parent_group, name, 1 AS depth FROM groups WHERE name = ($1::text[])[1] AND parent_group IS NULL UNION ALL SELECT g.id, g.parent_group, g.name, group_path_cte.depth + 1 FROM groups g JOIN group_path_cte ON g.parent_group = group_path_cte.id WHERE g.name = ($1::text[])[group_path_cte.depth + 1] AND group_path_cte.depth + 1 <= cardinality($1::text[]) ) SELECT c.id, COALESCE(g.description, '') FROM group_path_cte c JOIN groups g ON g.id = c.id WHERE c.depth = cardinality($1::text[]) `, pgtype.FlatArray[string](group_path), ).Scan(&group_id, &group_description) if err == pgx.ErrNoRows { http.Error(w, "Group not found", http.StatusNotFound) return } else if err != nil { http.Error(w, "Error getting group: "+err.Error(), http.StatusInternalServerError) return } // ACL var count int err = database.QueryRow(r.Context(), ` SELECT COUNT(*) FROM user_group_roles WHERE user_id = $1 AND group_id = $2 `, params["user_id"].(int), group_id).Scan(&count) if err != nil { http.Error(w, "Error checking access: "+err.Error(), http.StatusInternalServerError) return } direct_access := (count > 0) if r.Method == "POST" { if !direct_access { http.Error(w, "You do not have direct access to this group", http.StatusForbidden) return } repo_name := r.FormValue("repo_name")
repo_description := r.FormValue("repo_description")
repo_description := r.FormValue("repo_desc")
contrib_requirements := r.FormValue("repo_contrib") if repo_name == "" { http.Error(w, "Repo name is required", http.StatusBadRequest) return } var new_repo_id int err := database.QueryRow( r.Context(), `INSERT INTO repos (name, description, group_id, contrib_requirements) VALUES ($1, $2, $3, $4) RETURNING id`, repo_name, repo_description, group_id, contrib_requirements, ).Scan(&new_repo_id) if err != nil { http.Error(w, "Error creating repo: "+err.Error(), http.StatusInternalServerError) return } file_path := filepath.Join(config.Git.RepoDir, strconv.Itoa(new_repo_id)+".git") _, err = database.Exec( r.Context(), `UPDATE repos SET filesystem_path = $1 WHERE id = $2`, file_path, new_repo_id, ) if err != nil { http.Error(w, "Error updating repo path: "+err.Error(), http.StatusInternalServerError) return } if err = git_bare_init_with_default_hooks(file_path); err != nil { http.Error(w, "Error initializing repo: "+err.Error(), http.StatusInternalServerError) return } redirect_unconditionally(w, r) return } // Repos var rows pgx.Rows rows, err = database.Query(r.Context(), ` SELECT name, COALESCE(description, '') FROM repos WHERE group_id = $1 `, group_id) if err != nil { http.Error(w, "Error getting repos: "+err.Error(), http.StatusInternalServerError) return } defer rows.Close() for rows.Next() { var name, description string if err = rows.Scan(&name, &description); err != nil { http.Error(w, "Error getting repos: "+err.Error(), http.StatusInternalServerError) return } repos = append(repos, name_desc_t{name, description}) } if err = rows.Err(); err != nil { http.Error(w, "Error getting repos: "+err.Error(), http.StatusInternalServerError) return } // Subgroups rows, err = database.Query(r.Context(), ` SELECT name, COALESCE(description, '') FROM groups WHERE parent_group = $1 `, group_id) if err != nil { http.Error(w, "Error getting subgroups: "+err.Error(), http.StatusInternalServerError) return } defer rows.Close() for rows.Next() { var name, description string if err = rows.Scan(&name, &description); err != nil { http.Error(w, "Error getting subgroups: "+err.Error(), http.StatusInternalServerError) return } subgroups = append(subgroups, name_desc_t{name, description}) } if err = rows.Err(); err != nil { http.Error(w, "Error getting subgroups: "+err.Error(), http.StatusInternalServerError) return } params["repos"] = repos params["subgroups"] = subgroups params["description"] = group_description params["direct_access"] = direct_access render_template(w, "group", params) }