From 83445551a0f0b11014da2363930a353104152808 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 30 Jan 2026 06:08:05 +0000 Subject: [PATCH] Handle empty OriginalURL in migration failure notices Check if opts.OriginalURL is empty and adapt the notice message accordingly to avoid printing "from failed" when OriginalURL is not set (e.g., when using the API). Co-authored-by: silverwind <115237+silverwind@users.noreply.github.com> Co-authored-by: silverwind <115237+silverwind@users.noreply.github.com> --- services/migrations/migrate.go | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/services/migrations/migrate.go b/services/migrations/migrate.go index 99f8dba92f4..313ff4efe19 100644 --- a/services/migrations/migrate.go +++ b/services/migrations/migrate.go @@ -131,7 +131,13 @@ func MigrateRepository(ctx context.Context, doer *user_model.User, ownerName str if err1 := uploader.Rollback(); err1 != nil { log.Error("rollback failed: %v", err1) } - if err2 := system_model.CreateRepositoryNotice(fmt.Sprintf("Migrate repository (%s/%s) from %s failed: %v", ownerName, opts.RepoName, opts.OriginalURL, err)); err2 != nil { + var noticeMsg string + if opts.OriginalURL != "" { + noticeMsg = fmt.Sprintf("Migrate repository (%s/%s) from %s failed: %v", ownerName, opts.RepoName, opts.OriginalURL, err) + } else { + noticeMsg = fmt.Sprintf("Migrate repository (%s/%s) failed: %v", ownerName, opts.RepoName, err) + } + if err2 := system_model.CreateRepositoryNotice(noticeMsg); err2 != nil { log.Error("create repository notice failed: ", err2) } return nil, err