From ec1e7d7e805c642a3c9e763c3526ca2c1bcb6d67 Mon Sep 17 00:00:00 2001
From: Gusted <williamzijl7@hotmail.com>
Date: Wed, 15 Jun 2022 18:51:09 +0200
Subject: [PATCH] Use correct count for `NumOpenIssues`

- Don't specify the field in `Count` instead use `Cols` for this.
- Call `log.Error` when a error occur.
---
 models/project/issue.go | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/models/project/issue.go b/models/project/issue.go
index 04efc0e749b6..6e6a8c574666 100644
--- a/models/project/issue.go
+++ b/models/project/issue.go
@@ -9,6 +9,7 @@ import (
 	"fmt"
 
 	"code.gitea.io/gitea/models/db"
+	"code.gitea.io/gitea/modules/log"
 )
 
 // ProjectIssue saves relation from issue to a project
@@ -41,6 +42,7 @@ func (p *Project) NumIssues() int {
 		Cols("issue_id").
 		Count()
 	if err != nil {
+		log.Error("NumIssues: %v", err)
 		return 0
 	}
 	return int(c)
@@ -54,6 +56,7 @@ func (p *Project) NumClosedIssues() int {
 		Cols("issue_id").
 		Count()
 	if err != nil {
+		log.Error("NumClosedIssues: %v", err)
 		return 0
 	}
 	return int(c)
@@ -63,8 +66,11 @@ func (p *Project) NumClosedIssues() int {
 func (p *Project) NumOpenIssues() int {
 	c, err := db.GetEngine(db.DefaultContext).Table("project_issue").
 		Join("INNER", "issue", "project_issue.issue_id=issue.id").
-		Where("project_issue.project_id=? AND issue.is_closed=?", p.ID, false).Count("issue.id")
+		Where("project_issue.project_id=? AND issue.is_closed=?", p.ID, false).
+		Cols("issue_id").
+		Count()
 	if err != nil {
+		log.Error("NumOpenIssues: %v", err)
 		return 0
 	}
 	return int(c)