From 753b7d205be260b8be465b5291a02975a81f3093 Mon Sep 17 00:00:00 2001
From: David Svantesson <davidsvantesson@gmail.com>
Date: Sat, 2 Nov 2019 08:52:47 +0100
Subject: [PATCH] Move check IncludeAllRepositories to removeRepository.

---
 models/org_team.go | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/models/org_team.go b/models/org_team.go
index 8ba5d2742d..c33857a411 100644
--- 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