Move check IncludeAllRepositories to removeRepository.

davidsvantesson · Nov 2, 2019 · 753b7d2 · 753b7d2
1 parent ee52d2d
commit 753b7d2
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/models/org_team.go b/models/org_team.go
@@ -265,6 +265,10 @@ func (t *Team) AddRepository(repo *Repository) (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 {
 		return err
 	}
@@ -312,10 +316,6 @@ func (t *Team) RemoveRepository(repoID int64) error {
 		return nil
 	}
 
-	if t.IncludesAllRepositories {
-		return nil
-	}
-
 	repo, err := GetRepositoryByID(repoID)
 	if err != nil {
 		return err