mirror of
				https://github.com/go-gitea/gitea
				synced 2025-11-03 21:08:25 +00:00 
			
		
		
		
	Remove unnecessary duplicate code (#34552)
`GetIssuesLastCommitStatus` will revoke `GetIssuesAllCommitStatus` but it has been invoked. The `CommitStatus` template variable has never been used in notification subscription page so that it could be removed.
This commit is contained in:
		@@ -287,16 +287,8 @@ func NotificationSubscriptions(ctx *context.Context) {
 | 
			
		||||
	ctx.Data["CommitLastStatus"] = lastStatus
 | 
			
		||||
	ctx.Data["CommitStatuses"] = commitStatuses
 | 
			
		||||
	ctx.Data["Issues"] = issues
 | 
			
		||||
 | 
			
		||||
	ctx.Data["IssueRefEndNames"], ctx.Data["IssueRefURLs"] = issue_service.GetRefEndNamesAndURLs(issues, "")
 | 
			
		||||
 | 
			
		||||
	commitStatus, err := pull_service.GetIssuesLastCommitStatus(ctx, issues)
 | 
			
		||||
	if err != nil {
 | 
			
		||||
		ctx.ServerError("GetIssuesLastCommitStatus", err)
 | 
			
		||||
		return
 | 
			
		||||
	}
 | 
			
		||||
	ctx.Data["CommitStatus"] = commitStatus
 | 
			
		||||
 | 
			
		||||
	approvalCounts, err := issues.GetApprovalCounts(ctx)
 | 
			
		||||
	if err != nil {
 | 
			
		||||
		ctx.ServerError("ApprovalCounts", err)
 | 
			
		||||
 
 | 
			
		||||
@@ -945,12 +945,6 @@ func GetSquashMergeCommitMessages(ctx context.Context, pr *issues_model.PullRequ
 | 
			
		||||
	return stringBuilder.String()
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
// GetIssuesLastCommitStatus returns a map of issue ID to the most recent commit's latest status
 | 
			
		||||
func GetIssuesLastCommitStatus(ctx context.Context, issues issues_model.IssueList) (map[int64]*git_model.CommitStatus, error) {
 | 
			
		||||
	_, lastStatus, err := GetIssuesAllCommitStatus(ctx, issues)
 | 
			
		||||
	return lastStatus, err
 | 
			
		||||
}
 | 
			
		||||
 | 
			
		||||
// GetIssuesAllCommitStatus returns a map of issue ID to a list of all statuses for the most recent commit as well as a map of issue ID to only the commit's latest status
 | 
			
		||||
func GetIssuesAllCommitStatus(ctx context.Context, issues issues_model.IssueList) (map[int64][]*git_model.CommitStatus, map[int64]*git_model.CommitStatus, error) {
 | 
			
		||||
	if err := issues.LoadPullRequests(ctx); err != nil {
 | 
			
		||||
 
 | 
			
		||||
		Reference in New Issue
	
	Block a user