chore(db): remove unused field (#7202)

This commit is contained in:
Joe Chen
2022-10-23 00:15:43 +08:00
committed by GitHub
parent 2e020b1cf6
commit 8077360cf6
2 changed files with 17 additions and 20 deletions

View File

@@ -7,7 +7,6 @@ package db
import (
"fmt"
"io"
"io/ioutil"
"mime/multipart"
"os"
"os/exec"
@@ -109,14 +108,13 @@ func (repo *Repository) CheckoutNewBranch(oldBranch, newBranch string) error {
}
type UpdateRepoFileOptions struct {
LastCommitID string
OldBranch string
NewBranch string
OldTreeName string
NewTreeName string
Message string
Content string
IsNewFile bool
OldBranch string
NewBranch string
OldTreeName string
NewTreeName string
Message string
Content string
IsNewFile bool
}
// UpdateRepoFile adds or updates a file in repository.
@@ -179,7 +177,7 @@ func (repo *Repository) UpdateRepoFile(doer *User, opts UpdateRepoFileOptions) (
}
}
if err = ioutil.WriteFile(filePath, []byte(opts.Content), 0666); err != nil {
if err = os.WriteFile(filePath, []byte(opts.Content), 0600); err != nil {
return fmt.Errorf("write file: %v", err)
}
@@ -225,7 +223,7 @@ func (repo *Repository) GetDiffPreview(branch, treePath, content string) (diff *
if err = os.MkdirAll(filepath.Dir(filePath), os.ModePerm); err != nil {
return nil, err
}
if err = ioutil.WriteFile(filePath, []byte(content), 0666); err != nil {
if err = os.WriteFile(filePath, []byte(content), 0600); err != nil {
return nil, fmt.Errorf("write file: %v", err)
}
@@ -365,7 +363,7 @@ func NewUpload(name string, buf []byte, file multipart.File) (_ *Upload, err err
if err != nil {
return nil, fmt.Errorf("create: %v", err)
}
defer fw.Close()
defer func() { _ = fw.Close() }()
if _, err = fw.Write(buf); err != nil {
return nil, fmt.Errorf("write: %v", err)

View File

@@ -262,14 +262,13 @@ func editFilePost(c *context.Context, f form.EditRepoFile, isNewFile bool) {
}
if err := c.Repo.Repository.UpdateRepoFile(c.User, db.UpdateRepoFileOptions{
LastCommitID: lastCommit,
OldBranch: oldBranchName,
NewBranch: branchName,
OldTreeName: oldTreePath,
NewTreeName: f.TreePath,
Message: message,
Content: strings.ReplaceAll(f.Content, "\r", ""),
IsNewFile: isNewFile,
OldBranch: oldBranchName,
NewBranch: branchName,
OldTreeName: oldTreePath,
NewTreeName: f.TreePath,
Message: message,
Content: strings.ReplaceAll(f.Content, "\r", ""),
IsNewFile: isNewFile,
}); err != nil {
log.Error("Failed to update repo file: %v", err)
c.FormErr("TreePath")