typo and function refactor

Author: lunny

models/action.go

@@ -670,13 +670,13 @@ func CommitRepoAction(opts CommitRepoActionOptions) error {
 
 			for _, issue := range issues {
 				if isForcePush {
-					if err := ClearPullPushComent(issue); err != nil {
+					if err := ClearPullPushComment(issue); err != nil {
 						return fmt.Errorf("ClearPullPushComent: %v", err)
 					}
 				}
 
 				for _, c := range opts.Commits.Commits {
-					if err := CreatePullPushComment(pusher, issue.Repo, issue, c.Message, c.Sha1, issue.Repo.FullName()); err != nil {
+					if err := CreatePullPushComment(pusher, issue.Repo, issue, c); err != nil {
 						return fmt.Errorf("CreatePullPushComment: %v", err)
 					}
 				}

models/issue_comment.go

@@ -600,8 +600,8 @@ func CreateRefComment(doer *User, repo *Repository, issue *Issue, content, commi
 	return err
 }
 
-// ClearPullPushComent clear all the push commit on issue since fore push
-func ClearPullPushComent(issue *Issue) error {
+// ClearPullPushComment clear all the push commit on issue since fore push
+func ClearPullPushComment(issue *Issue) error {
 	_, err := x.Delete(&Comment{
 		Type:    CommentTypePullPushCommit,
 		IssueID: issue.ID,
@@ -610,16 +610,16 @@ func ClearPullPushComent(issue *Issue) error {
 }
 
 // CreatePullPushComment creates a commit when push commit to a pull request.
-func CreatePullPushComment(doer *User, repo *Repository, issue *Issue, content, commitSHA, repoFullName string) error {
-	if len(commitSHA) == 0 {
+func CreatePullPushComment(doer *User, repo *Repository, issue *Issue, commit *PushCommit) error {
+	if len(commit.Sha1) == 0 {
 		return fmt.Errorf("cannot create reference with empty commit SHA")
 	}
 
 	// Check if same reference from same commit has already existed.
 	has, err := x.Get(&Comment{
 		Type:      CommentTypePullPushCommit,
 		IssueID:   issue.ID,
-		CommitSHA: commitSHA,
+		CommitSHA: commit.Sha1,
 	})
 	if err != nil {
 		return fmt.Errorf("check pull push comment: %v", err)
@@ -631,9 +631,9 @@ func CreatePullPushComment(doer *User, repo *Repository, issue *Issue, content,
 			Doer:         doer,
 			Repo:         repo,
 			Issue:        issue,
-			CommitSHA:    commitSHA,
-			Content:      content,
-			RepoFullName: repoFullName,
+			CommitSHA:    commit.Sha1,
+			Content:      commit.Message,
+			RepoFullName: repo.FullName(),
 		})
 	}
 	return err