Remove Repository.LocalCopyPath()

Author: zeripath

models/repo.go

@@ -730,11 +730,6 @@ func (repo *Repository) DescriptionHTML() template.HTML {
 	return template.HTML(markup.Sanitize(string(desc)))
 }
 
-// LocalCopyPath returns the local repository copy path for the given repo.
-func (repo *Repository) LocalCopyPath() string {
-	return path.Join(LocalCopyPath(), com.ToStr(repo.ID))
-}
-
 // PatchPath returns corresponding patch file path of repository by given issue ID.
 func (repo *Repository) PatchPath(index int64) (string, error) {
 	return repo.patchPath(x, index)
@@ -1516,7 +1511,6 @@ func TransferOwnership(doer *User, newOwnerName string, repo *Repository) error
 	if err = os.Rename(RepoPath(owner.Name, repo.Name), RepoPath(newOwner.Name, repo.Name)); err != nil {
 		return fmt.Errorf("rename repository directory: %v", err)
 	}
-	removeAllWithNotice(sess, "Delete repository local copy", repo.LocalCopyPath())
 
 	// Rename remote wiki repository to new path and delete local copy.
 	wikiPath := UncycloPath(owner.Name, repo.Name)
@@ -1566,14 +1560,6 @@ func ChangeRepositoryName(u *User, oldRepoName, newRepoName string) (err error)
 		return fmt.Errorf("rename repository directory: %v", err)
 	}
 
-	localPath := repo.LocalCopyPath()
-	if com.IsExist(localPath) {
-		_, err := git.NewCommand("remote", "set-url", "origin", newRepoPath).RunInDir(localPath)
-		if err != nil {
-			return fmt.Errorf("git remote set-url origin %s: %v", newRepoPath, err)
-		}
-	}
-
 	wikiPath := repo.UncycloPath()
 	if com.IsExist(wikiPath) {
 		if err = os.Rename(wikiPath, UncycloPath(u.Name, newRepoName)); err != nil {

models/repo_test.go

@@ -5,11 +5,9 @@
 package models
 
 import (
-	"path"
 	"testing"
 
 	"code.gitea.io/gitea/modules/markup"
-	"code.gitea.io/gitea/modules/setting"
 
 	"github.com/Unknwon/com"
 	"github.com/stretchr/testify/assert"
@@ -138,25 +136,6 @@ func TestRepoAPIURL(t *testing.T) {
 	assert.Equal(t, "https://try.gitea.io/api/v1/repos/user12/repo10", repo.APIURL())
 }
 
-func TestRepoLocalCopyPath(t *testing.T) {
-	assert.NoError(t, PrepareTestDatabase())
-
-	repo, err := GetRepositoryByID(10)
-	assert.NoError(t, err)
-	assert.NotNil(t, repo)
-
-	// test default
-	repoID := com.ToStr(repo.ID)
-	expected := path.Join(setting.AppDataPath, setting.Repository.Local.LocalCopyPath, repoID)
-	assert.Equal(t, expected, repo.LocalCopyPath())
-
-	// test absolute setting
-	tempPath := "/tmp/gitea/local-copy-path"
-	expected = path.Join(tempPath, repoID)
-	setting.Repository.Local.LocalCopyPath = tempPath
-	assert.Equal(t, expected, repo.LocalCopyPath())
-}
-
 func TestTransferOwnership(t *testing.T) {
 	assert.NoError(t, PrepareTestDatabase())