From 6cbeb66a4876d685b9a8d444b793ffcbdaa4da5d Mon Sep 17 00:00:00 2001
From: 6543 <6543@obermui.de>
Date: Mon, 30 Dec 2019 23:57:05 +0100
Subject: [PATCH] declare allowedColumnsUpdateIssueByAPI only once

---
 models/issue.go | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/models/issue.go b/models/issue.go
index 11dbcff8321a..81767d512d59 100644
--- a/models/issue.go
+++ b/models/issue.go
@@ -1586,19 +1586,21 @@ func updateIssueByCols(e Engine, issue *Issue, columns ...string) (err error) {
 	return
 }
 
+// allowed fields to update for a issue by API
+var allowedColumnsUpdateIssueByAPI = []string{
+	"name", "is_closed", "content", "milestone_id", "priority",
+	"deadline_unix", "updated_unix", "closed_unix", "is_locked",
+}
+
 // UpdateIssueByAPI updates all allowed fields of given issue.
 func UpdateIssueByAPI(issue *Issue) error {
-	// allowed fields to update
-	columns := []string{"name", "is_closed", "content", "milestone_id", "priority",
-		"deadline_unix", "updated_unix", "closed_unix", "is_locked"}
-
 	sess := x.NewSession()
 	defer sess.Close()
 	if err := sess.Begin(); err != nil {
 		return err
 	}
 
-	if err := updateIssueByCols(sess, issue, columns...); err != nil {
+	if err := updateIssueByCols(sess, issue, allowedColumnsUpdateIssueByAPI...); err != nil {
 		return err
 	}
 	if err := issue.addCrossReferences(sess, issue.Poster, true); err != nil {