only re-calc divergence if AddTestPullRequestTask() is exec

Author: 6543

services/pull/check.go

@@ -32,13 +32,7 @@ func AddToTaskQueue(pr *models.PullRequest) {
 	go func() {
 		err := prQueue.PushFunc(strconv.FormatInt(pr.ID, 10), func() error {
 			pr.Status = models.PullRequestStatusChecking
-			divergence, err := GetDiverging(pr)
-			if err != nil {
-				log.Error("AddToTaskQueue.GetDiverging[%d]: %v", pr.ID, err)
-			}
-			pr.CommitsAhead = divergence.Ahead
-			pr.CommitsBehind = divergence.Behind
-			err = pr.UpdateColsIfNotMerged("status", "commits_ahead", "commits_behind")
+			err := pr.UpdateColsIfNotMerged("status")
 			if err != nil {
 				log.Error("AddToTaskQueue.UpdateCols[%d].(add to queue): %v", pr.ID, err)
 			} else {

services/pull/pull.go

@@ -244,6 +244,15 @@ func AddTestPullRequestTask(doer *models.User, repoID int64, branch string, isSy
 			return
 		}
 		for _, pr := range prs {
+			divergence, err := GetDiverging(pr)
+			if err != nil {
+				log.Error("GetDiverging: %v", err)
+			} else {
+				err = pr.UpdateCommitDivergence(divergence)
+				if err != nil {
+					log.Error("UpdateCommitDivergence: %v", err)
+				}
+			}
 			AddToTaskQueue(pr)
 		}
 	})