changed "CountMilestonesByRepo" to "CountMilestonesByRepoIDs" so the naming matches up with the function "GetMilestonesByRepoIDs"

Author: bhalbright

models/issue_milestone.go

@@ -494,8 +494,8 @@ func DeleteMilestoneByRepoID(repoID, id int64) error {
 	return sess.Commit()
 }
 
-// CountMilestonesByRepo map from repoIDs to number of milestones matching the options`
-func CountMilestonesByRepo(repoIDs []int64, isClosed bool) (map[int64]int64, error) {
+// CountMilestonesByRepoIDs map from repoIDs to number of milestones matching the options`
+func CountMilestonesByRepoIDs(repoIDs []int64, isClosed bool) (map[int64]int64, error) {
 	sess := x.Where("is_closed = ?", isClosed)
 	sess.In("repo_id", repoIDs)
 

models/issue_milestone_test.go

@@ -290,7 +290,7 @@ func TestMilestoneList_LoadTotalTrackedTimes(t *testing.T) {
 	assert.Equal(t, miles[0].TotalTrackedTime, int64(3662))
 }
 
-func TestCountMilestonesByRepo(t *testing.T) {
+func TestCountMilestonesByRepoIDs(t *testing.T) {
 	assert.NoError(t, PrepareTestDatabase())
 	milestonesCount := func(repoID int64) (int, int) {
 		repo := AssertExistsAndLoadBean(t, &Repository{ID: repoID}).(*Repository)
@@ -299,12 +299,12 @@ func TestCountMilestonesByRepo(t *testing.T) {
 	repo1OpenCount, repo1ClosedCount := milestonesCount(1)
 	repo2OpenCount, repo2ClosedCount := milestonesCount(2)
 
-	openCounts, err := CountMilestonesByRepo([]int64{1, 2}, false)
+	openCounts, err := CountMilestonesByRepoIDs([]int64{1, 2}, false)
 	assert.NoError(t, err)
 	assert.EqualValues(t, repo1OpenCount, openCounts[1])
 	assert.EqualValues(t, repo2OpenCount, openCounts[2])
 
-	closedCounts, err := CountMilestonesByRepo([]int64{1, 2}, true)
+	closedCounts, err := CountMilestonesByRepoIDs([]int64{1, 2}, true)
 	assert.NoError(t, err)
 	assert.EqualValues(t, repo1ClosedCount, closedCounts[1])
 	assert.EqualValues(t, repo2ClosedCount, closedCounts[2])

routers/user/home.go

@@ -210,9 +210,9 @@ func Milestones(ctx *context.Context) {
 		repoIDs = userRepoIDs
 	}
 
-	counts, err := models.CountMilestonesByRepo(repoIDs, isShowClosed)
+	counts, err := models.CountMilestonesByRepoIDs(repoIDs, isShowClosed)
 	if err != nil {
-		ctx.ServerError("CountMilestonesByRepo", err)
+		ctx.ServerError("CountMilestonesByRepoIDs", err)
 		return
 	}