UpdateRepoStats must be run from InsertIssues not insertIssue

Loïc Dachary · realaravinth · commit 099c7cf3f2e6 · 2022-01-17T14:45:05.000+05:30
diff --git a/models/migrate.go b/models/migrate.go
@@ -52,6 +52,10 @@ func InsertIssues(issues ...*Issue) error {
 			return err
 		}
 	}
+	err = UpdateRepoStats(ctx, issues[0].RepoID)
+	if err != nil {
+		return err
+	}
 	return committer.Commit()
 }
 
@@ -83,7 +87,7 @@ func insertIssue(ctx context.Context, issue *Issue) error {
 		}
 	}
 
-	return UpdateRepoStats(ctx, issue.RepoID)
+	return nil
 }
 
 // InsertIssueComments inserts many comments of issues.

Original file line number	Diff line number	Diff line change
`@@ -52,6 +52,10 @@ func InsertIssues(issues ...*Issue) error {`
`52`	`52`	`return err`
`53`	`53`	`}`
`54`	`54`	`}`
	`55`	`+ err = UpdateRepoStats(ctx, issues[0].RepoID)`
	`56`	`+ if err != nil {`
	`57`	`+ return err`
	`58`	`+ }`
`55`	`59`	`return committer.Commit()`
`56`	`60`	`}`
`57`	`61`
`@@ -83,7 +87,7 @@ func insertIssue(ctx context.Context, issue *Issue) error {`
`83`	`87`	`}`
`84`	`88`	`}`
`85`	`89`
`86`		`- return UpdateRepoStats(ctx, issue.RepoID)`
	`90`	`+ return nil`
`87`	`91`	`}`
`88`	`92`
`89`	`93`	`// InsertIssueComments inserts many comments of issues.`