From 782769861a75340b71e7763cce9a15b77853a48f Mon Sep 17 00:00:00 2001
From: Gregory Eremin <g@erem.in>
Date: Thu, 19 Mar 2015 19:55:01 +0700
Subject: [PATCH] Fix fetching multiple pages in sync repo task

---
 task/sync.go | 20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

diff --git a/task/sync.go b/task/sync.go
index 363aa6f..a62d4c8 100644
--- a/task/sync.go
+++ b/task/sync.go
@@ -10,9 +10,7 @@ import (
 func SyncRepos(token, owner string) {
 	defer report("SyncRepos", time.Now())
 	client := newGithubClient(token)
-	opt := &github.RepositoryListByOrgOptions{
-		ListOptions: github.ListOptions{},
-	}
+	opt := &github.RepositoryListByOrgOptions{ListOptions: github.ListOptions{PerPage: 100}}
 
 	for {
 		opt.Page++
@@ -28,7 +26,7 @@ func SyncRepos(token, owner string) {
 			}
 			r.Save()
 		}
-		if len(repos) < 30 {
+		if opt.Page == resp.LastPage {
 			break
 		}
 	}
@@ -37,6 +35,7 @@ func SyncRepos(token, owner string) {
 func SyncContrib(token, owner, repo string) {
 	defer report("SyncContrib", time.Now())
 	client := newGithubClient(token)
+
 	contribs, resp, err := client.Repositories.ListContributorsStats(owner, repo)
 	saveResponseMeta(token, resp)
 	if err != nil {
@@ -46,22 +45,25 @@ func SyncContrib(token, owner, repo string) {
 		panic(err)
 	}
 
-	for _, c := range contribs {
-		for _, week := range c.Weeks {
+	for _, contrib := range contribs {
+		for _, week := range contrib.Weeks {
 			if *week.Commits == 0 {
 				continue
 			}
 
-			contrib := &db.Contrib{
+			c := &db.Contrib{
 				Week:      uint64(week.Week.Time.Unix()),
-				Author:    *c.Author.Login,
+				Author:    *contrib.Author.Login,
 				Owner:     owner,
 				Repo:      repo,
 				Commits:   uint64(*week.Commits),
 				Additions: uint64(*week.Additions),
 				Deletions: uint64(*week.Deletions),
 			}
-			contrib.Save()
+			c.Save()
+		}
+	}
+}
 
 func SyncUserOrgs(token string) (err error) {
 	defer report("SyncUserOrgs", time.Now())