From cf787b837c413adc1279cf1506aa95b2d7e27aca Mon Sep 17 00:00:00 2001 From: vecmatex Date: Mon, 19 May 2025 16:40:56 +0800 Subject: [PATCH] fixbug --- routers/web/shared/actions/runners.go | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/routers/web/shared/actions/runners.go b/routers/web/shared/actions/runners.go index bed8a6f2c1..38ffb56507 100644 --- a/routers/web/shared/actions/runners.go +++ b/routers/web/shared/actions/runners.go @@ -156,6 +156,7 @@ func RunnerRegistARunner(ctx *context.Context, ownerID, repoID int64, redirectTo ctx.Redirect(redirectTo) return } + ctx.Flash.Success(ctx.Tr("actions.runners.regist_runner_success")) ctx.Redirect(redirectTo) } @@ -164,6 +165,12 @@ func RunnerRegistARunner(ctx *context.Context, ownerID, repoID int64, redirectTo func RunnerDeletePost(ctx *context.Context, runnerID int64, successRedirectTo, failedRedirectTo string, ) { + + runner, err := actions_model.GetRunnerByID(ctx, runnerID) + if err != nil { + log.Warn("DeleteRunnerPost GetRunnerID failed") + } + if err := actions_model.DeleteRunner(ctx, runnerID); err != nil { log.Warn("DeleteRunnerPost.UpdateRunner failed: %v, url: %s", err, ctx.Req.URL) ctx.Flash.Warning(ctx.Tr("actions.runners.delete_runner_failed")) @@ -171,10 +178,7 @@ func RunnerDeletePost(ctx *context.Context, runnerID int64, ctx.JSONRedirect(failedRedirectTo) return } - runner, err := actions_model.GetRunnerByID(ctx, runnerID) - if err != nil { - log.Warn("DeleteRunnerPost.DeleteRunnerContainer failed: %v, url: %s", err, ctx.Req.URL) - } + runnerName := runner.Name err = runner_services.DeleteRunnerByName(ctx, runnerName) @@ -185,6 +189,6 @@ func RunnerDeletePost(ctx *context.Context, runnerID int64, log.Info("DeleteRunnerPost success: %s", ctx.Req.URL) ctx.Flash.Success(ctx.Tr("actions.runners.delete_runner_success")) - + log.Info("DeleteRunnerPost success: %s", successRedirectTo) ctx.JSONRedirect(successRedirectTo) }