handle error so template JS could check

go-gitea · Nov 28, 2019 · 0e406a7 · 0e406a7
1 parent cad5add
commit 0e406a7
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/routers/repo/issue.go b/routers/repo/issue.go
@@ -1449,7 +1449,8 @@ func ChangeIssueReaction(ctx *context.Context, form auth.ReactionForm) {
 	switch ctx.Params(":action") {
 	case "react":
 		if !util.IsStringInSlice(form.Content, setting.UI.Reactions) {
-			log.Error("ChangeIssueReaction: '%s' is not an allowed reaction", form.Content)
+			err := fmt.Errorf("ChangeIssueReaction: '%s' is not an allowed reaction", form.Content)
+			ctx.ServerError(err.Error(), err)
 			return
 		}
 
@@ -1549,7 +1550,8 @@ func ChangeCommentReaction(ctx *context.Context, form auth.ReactionForm) {
 	switch ctx.Params(":action") {
 	case "react":
 		if !util.IsStringInSlice(form.Content, setting.UI.Reactions) {
-			log.Error("ChangeIssueReaction: '%s' is not an allowed reaction", form.Content)
+			err := fmt.Errorf("ChangeIssueReaction: '%s' is not an allowed reaction", form.Content)
+			ctx.ServerError(err.Error(), err)
 			return
 		}