469a86ffdec95416953d7617b85557743b2ae729

Author
Ayman Bagabas <ayman.bagabas@gmail.com>
Committer
Ayman Bagabas <ayman.bagabas@gmail.com>
Date

Message

fix(backend): delete non-existing repo

Fixes: 6eba2455d03c ("fix(backend): delete repo when importing existing one")

Diff

 1diff --git a/server/backend/sqlite/sqlite.go b/server/backend/sqlite/sqlite.go
 2index d45a9f4052394438b56ac5719afb3e15c25d0eb8..51090683ad17b54ee9a031965bb30cbf9a9c4bd5 100644
 3--- a/server/backend/sqlite/sqlite.go
 4+++ b/server/backend/sqlite/sqlite.go
 5@@ -223,18 +223,14 @@ func (d *SqliteBackend) DeleteRepository(name string) error {
 6 	name = utils.SanitizeRepo(name)
 7 	repo := name + ".git"
 8 	rp := filepath.Join(d.reposPath(), repo)
 9-	if _, err := os.Stat(rp); err != nil {
10-		return os.ErrNotExist
11-	}
12 
13-	if err := wrapTx(d.db, d.ctx, func(tx *sqlx.Tx) error {
14+	return wrapTx(d.db, d.ctx, func(tx *sqlx.Tx) error {
15+		if err := os.RemoveAll(rp); err != nil {
16+			return err
17+		}
18 		_, err := tx.Exec("DELETE FROM repo WHERE name = ?;", name)
19 		return err
20-	}); err != nil {
21-		return wrapDbErr(err)
22-	}
23-
24-	return os.RemoveAll(rp)
25+	})
26 }
27 
28 // RenameRepository renames a repository.