From 002b597c1f87cd5c69d32053a62f57c08d48d3ee Mon Sep 17 00:00:00 2001 From: Richard Mahn Date: Sat, 29 Jun 2019 11:19:24 -0400 Subject: Fixes #7152 - Allow create/update/delete message to be empty, use default message (#7324) * Fixes #7152 - Allow create/update/delete message to be empty, use default message * Linting fix * Fix to delete integration tests --- routers/api/v1/repo/file.go | 13 +++++++++++++ 1 file changed, 13 insertions(+) (limited to 'routers/api') diff --git a/routers/api/v1/repo/file.go b/routers/api/v1/repo/file.go index 20f80f37f4..d510756283 100644 --- a/routers/api/v1/repo/file.go +++ b/routers/api/v1/repo/file.go @@ -204,6 +204,11 @@ func CreateFile(ctx *context.APIContext, apiOpts api.CreateFileOptions) { Email: apiOpts.Author.Email, }, } + + if opts.Message == "" { + opts.Message = ctx.Tr("repo.editor.add", opts.TreePath) + } + if fileResponse, err := createOrUpdateFile(ctx, opts); err != nil { ctx.Error(http.StatusInternalServerError, "CreateFile", err) } else { @@ -264,6 +269,10 @@ func UpdateFile(ctx *context.APIContext, apiOpts api.UpdateFileOptions) { }, } + if opts.Message == "" { + opts.Message = ctx.Tr("repo.editor.update", opts.TreePath) + } + if fileResponse, err := createOrUpdateFile(ctx, opts); err != nil { ctx.Error(http.StatusInternalServerError, "UpdateFile", err) } else { @@ -346,6 +355,10 @@ func DeleteFile(ctx *context.APIContext, apiOpts api.DeleteFileOptions) { }, } + if opts.Message == "" { + opts.Message = ctx.Tr("repo.editor.delete", opts.TreePath) + } + if fileResponse, err := repofiles.DeleteRepoFile(ctx.Repo.Repository, ctx.User, opts); err != nil { ctx.Error(http.StatusInternalServerError, "DeleteFile", err) } else { -- cgit v1.2.3