thepaperpilot/forgejo
1
0
Fork 0
Code Issues Pull requests Releases Activity Actions

Move check IncludeAllRepositories to removeRepository.

Browse source
This commit is contained in:
David Svantesson 2019-11-02 08:52:47 +01:00
parent ee52d2dd01
commit 753b7d205b
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
models/org_team.go
View file

@ -265,6 +265,10 @@ func (t *Team) AddRepository(repo *Repository) (err error) {
} }
func (t *Team) removeRepository(e Engine, repo *Repository, recalculate bool) (err error) { func (t *Team) removeRepository(e Engine, repo *Repository, recalculate bool) (err error) {
if t.IncludesAllRepositories {
return nil
}
if err = removeTeamRepo(e, t.ID, repo.ID); err != nil { if err = removeTeamRepo(e, t.ID, repo.ID); err != nil {
return err return err
} }
@ -312,10 +316,6 @@ func (t *Team) RemoveRepository(repoID int64) error {
return nil return nil
} }
if t.IncludesAllRepositories {
return nil
}
repo, err := GetRepositoryByID(repoID) repo, err := GetRepositoryByID(repoID)
if err != nil { if err != nil {
return err return err