Site admin could create repos even MAX_CREATION_LIMIT=0 (#4645)

lunny · web-flow · commit 578cf52ce5ba · 2018-08-09T09:04:16.000+08:00
* site admin could create repos even MAX_CREATION_LIMIT=0

* Optimize if structure
diff --git a/models/repo.go b/models/repo.go
@@ -1411,7 +1411,7 @@ func createRepository(e *xorm.Session, doer, u *User, repo *Repository) (err err
 
 // CreateRepository creates a repository for the user/organization u.
 func CreateRepository(doer, u *User, opts CreateRepoOptions) (_ *Repository, err error) {
-	if !u.CanCreateRepo() {
+	if !doer.IsAdmin && !u.CanCreateRepo() {
 		return nil, ErrReachLimitOfRepo{u.MaxRepoCreation}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -1411,7 +1411,7 @@ func createRepository(e xorm.Session, doer, u User, repo *Repository) (err err`
`1411`	`1411`
`1412`	`1412`	`// CreateRepository creates a repository for the user/organization u.`
`1413`	`1413`	`func CreateRepository(doer, u User, opts CreateRepoOptions) (_ Repository, err error) {`
`1414`		`- if !u.CanCreateRepo() {`
	`1414`	`+ if !doer.IsAdmin && !u.CanCreateRepo() {`
`1415`	`1415`	`return nil, ErrReachLimitOfRepo{u.MaxRepoCreation}`
`1416`	`1416`	`}`
`1417`	`1417`