summaryrefslogtreecommitdiffstats
path: root/routers
diff options
context:
space:
mode:
Diffstat (limited to 'routers')
-rw-r--r--routers/org/teams.go21
-rw-r--r--routers/user/home.go6
-rw-r--r--routers/user/home_test.go4
3 files changed, 25 insertions, 6 deletions
diff --git a/routers/org/teams.go b/routers/org/teams.go
index d894c86614..87bfb8596a 100644
--- a/routers/org/teams.go
+++ b/routers/org/teams.go
@@ -182,7 +182,14 @@ func NewTeamPost(ctx *context.Context, form auth.CreateTeamForm) {
Authorize: models.ParseAccessMode(form.Permission),
}
if t.Authorize < models.AccessModeAdmin {
- t.UnitTypes = form.Units
+ var units = make([]*models.TeamUnit, 0, len(form.Units))
+ for _, tp := range form.Units {
+ units = append(units, &models.TeamUnit{
+ OrgID: ctx.Org.Organization.ID,
+ Type: tp,
+ })
+ }
+ t.Units = units
}
ctx.Data["Team"] = t
@@ -264,9 +271,17 @@ func EditTeamPost(ctx *context.Context, form auth.CreateTeamForm) {
}
t.Description = form.Description
if t.Authorize < models.AccessModeAdmin {
- t.UnitTypes = form.Units
+ var units = make([]models.TeamUnit, 0, len(form.Units))
+ for _, tp := range form.Units {
+ units = append(units, models.TeamUnit{
+ OrgID: t.OrgID,
+ TeamID: t.ID,
+ Type: tp,
+ })
+ }
+ models.UpdateTeamUnits(t, units)
} else {
- t.UnitTypes = nil
+ models.UpdateTeamUnits(t, nil)
}
if ctx.HasError() {
diff --git a/routers/user/home.go b/routers/user/home.go
index 2a193bbdef..0c84b2498e 100644
--- a/routers/user/home.go
+++ b/routers/user/home.go
@@ -203,7 +203,11 @@ func Issues(ctx *context.Context) {
return
}
} else {
- userRepoIDs, err = ctxUser.GetAccessRepoIDs()
+ unitType := models.UnitTypeIssues
+ if isPullList {
+ unitType = models.UnitTypePullRequests
+ }
+ userRepoIDs, err = ctxUser.GetAccessRepoIDs(unitType)
if err != nil {
ctx.ServerError("ctxUser.GetAccessRepoIDs", err)
return
diff --git a/routers/user/home_test.go b/routers/user/home_test.go
index a9b146b762..8a3d9b9f51 100644
--- a/routers/user/home_test.go
+++ b/routers/user/home_test.go
@@ -26,8 +26,8 @@ func TestIssues(t *testing.T) {
Issues(ctx)
assert.EqualValues(t, http.StatusOK, ctx.Resp.Status())
- assert.EqualValues(t, map[int64]int64{1: 1, 2: 1}, ctx.Data["Counts"])
+ assert.EqualValues(t, map[int64]int64{1: 1}, ctx.Data["Counts"])
assert.EqualValues(t, true, ctx.Data["IsShowClosed"])
assert.Len(t, ctx.Data["Issues"], 1)
- assert.Len(t, ctx.Data["Repos"], 2)
+ assert.Len(t, ctx.Data["Repos"], 1)
}