From 4def56671909cd431213e0165d252c01802b8e2d Mon Sep 17 00:00:00 2001 From: Andrew Thornton Date: Sat, 30 Oct 2021 21:56:46 +0100 Subject: [PATCH] Add Reindex buttons to repository settings page This PR adds reindexing request buttons to the repository settings page. Fix #3796 Signed-off-by: Andrew Thornton --- options/locale/locale_en-US.ini | 6 ++++ routers/web/repo/setting.go | 48 ++++++++++++++++++++++++++++ services/forms/repo_form.go | 1 + templates/repo/settings/options.tmpl | 41 +++++++++++++++++++++++- 4 files changed, 95 insertions(+), 1 deletion(-) diff --git a/options/locale/locale_en-US.ini b/options/locale/locale_en-US.ini index c1762799ebcf..f01e71625a34 100644 --- a/options/locale/locale_en-US.ini +++ b/options/locale/locale_en-US.ini @@ -1706,6 +1706,12 @@ settings.pulls.default_delete_branch_after_merge = Delete pull request branch af settings.projects_desc = Enable Repository Projects settings.admin_settings = Administrator Settings settings.admin_enable_health_check = Enable Repository Health Checks (git fsck) +settings.admin_code_indexer = Code Indexer +settings.admin_stats_indexer = Code Statistics Indexer +settings.admin_indexer_commit_sha = Last Indexed SHA +settings.admin_indexer_unindexed = Unindexed +settings.reindex_button = Add to Reindex Queue +settings.reindex_requested=Reindex Requested settings.admin_enable_close_issues_via_commit_in_any_branch = Close an issue via a commit made in a non default branch settings.danger_zone = Danger Zone settings.new_owner_has_same_repo = The new owner already has a repository with same name. Please choose another name. diff --git a/routers/web/repo/setting.go b/routers/web/repo/setting.go index e71a5bf482bb..e139d07a1e16 100644 --- a/routers/web/repo/setting.go +++ b/routers/web/repo/setting.go @@ -19,6 +19,8 @@ import ( "code.gitea.io/gitea/modules/base" "code.gitea.io/gitea/modules/context" "code.gitea.io/gitea/modules/git" + "code.gitea.io/gitea/modules/indexer/code" + "code.gitea.io/gitea/modules/indexer/stats" "code.gitea.io/gitea/modules/lfs" "code.gitea.io/gitea/modules/log" "code.gitea.io/gitea/modules/migrations" @@ -61,6 +63,24 @@ func Settings(ctx *context.Context) { signing, _ := models.SigningKey(ctx.Repo.Repository.RepoPath()) ctx.Data["SigningKeyAvailable"] = len(signing) > 0 ctx.Data["SigningSettings"] = setting.Repository.Signing + ctx.Data["CodeIndexerEnabled"] = setting.Indexer.RepoIndexerEnabled + + if ctx.Repo.IsAdmin() { + if setting.Indexer.RepoIndexerEnabled { + status, err := ctx.Repo.Repository.GetIndexerStatus(models.RepoIndexerTypeCode) + if err != nil { + ctx.ServerError("repo.indexer_status", err) + return + } + ctx.Data["CodeIndexerStatus"] = status + } + status, err := ctx.Repo.Repository.GetIndexerStatus(models.RepoIndexerTypeStats) + if err != nil { + ctx.ServerError("repo.indexer_status", err) + return + } + ctx.Data["StatsIndexerStatus"] = status + } ctx.HTML(http.StatusOK, tplSettingsOptions) } @@ -504,6 +524,34 @@ func SettingsPost(ctx *context.Context) { ctx.Flash.Success(ctx.Tr("repo.settings.update_settings_success")) ctx.Redirect(ctx.Repo.RepoLink + "/settings") + case "admin_index": + if !ctx.User.IsAdmin { + ctx.Error(http.StatusForbidden) + return + } + + switch form.Index { + case "stats": + if err := stats.UpdateRepoIndexer(ctx.Repo.Repository); err != nil { + ctx.ServerError("UpdateStatsRepondexer", err) + return + } + case "code": + if !setting.Indexer.RepoIndexerEnabled { + ctx.Error(http.StatusForbidden) + return + } + code.UpdateRepoIndexer(ctx.Repo.Repository) + default: + ctx.NotFound("", nil) + return + } + + log.Trace("Repository reindex for %s requested: %s/%s", form.Index, ctx.Repo.Owner.Name, repo.Name) + + ctx.Flash.Success(ctx.Tr("repo.settings.reindex_requested")) + ctx.Redirect(ctx.Repo.RepoLink + "/settings") + case "convert": if !ctx.Repo.IsOwner() { ctx.Error(http.StatusNotFound) diff --git a/services/forms/repo_form.go b/services/forms/repo_form.go index 7c61be5e2221..2364943493d2 100644 --- a/services/forms/repo_form.go +++ b/services/forms/repo_form.go @@ -162,6 +162,7 @@ type RepoSettingForm struct { // Admin settings EnableHealthCheck bool + Index string } // Validate validates the fields diff --git a/templates/repo/settings/options.tmpl b/templates/repo/settings/options.tmpl index 211b7da8e7cf..3aca780f0923 100644 --- a/templates/repo/settings/options.tmpl +++ b/templates/repo/settings/options.tmpl @@ -558,11 +558,50 @@ -
+ +
+
+ {{.CsrfTokenHtml}} + + {{if .CodeIndexerEnabled}} +

{{.i18n.Tr "repo.settings.admin_code_indexer"}}

+
+ + + {{if .CodeIndexerStatus}} + + {{ShortSha .CodeIndexerStatus.CommitSha}} + + {{else}} + {{.i18n.Tr "repo.settings.admin_indexer_unindexed"}} + {{end}} + +
+ +
+
+ {{end}} +

{{.i18n.Tr "repo.settings.admin_stats_indexer"}}

+
+ + + {{if .StatsIndexerStatus}} + + {{ShortSha .StatsIndexerStatus.CommitSha}} + + {{else}} + {{.i18n.Tr "repo.settings.admin_indexer_unindexed"}} + {{end}} + +
+ +
+
+
{{end}}