dkforest

commit 49c7831117293bee79baf659620284f3b55d279a
parent b15ee0b723fb9287ecd7ecb2d4bd982b47dbc531
Author: n0tr1v <n0tr1v@protonmail.com>
Date:   Mon, 12 Jun 2023 13:23:26 -0700

cleanup

Diffstat:
M
pkg/web/handlers/handlers.go
 | 
4
++--
M
pkg/web/handlers/interceptors/chess.go
 | 
4
++++

2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/pkg/web/handlers/handlers.go b/pkg/web/handlers/handlers.go
@@ -5007,10 +5007,10 @@ func ChessGameHandler(c echo.Context) error {
 	isFlipped := authUser.ID == g.Player2.ID
 
 	if c.Request().Method == http.MethodPost {
-		msg := c.Request().PostFormValue("message")
-		if authUser.ID != g.Player1.ID && authUser.ID != g.Player2.ID {
+		if !g.IsPlayer(authUser.ID) {
 			return c.Redirect(http.StatusFound, c.Request().Referer())
 		}
+		msg := c.Request().PostFormValue("message")
 		if msg == "resign" {
 			resignColor := utils.Ternary(isFlipped, chess.Black, chess.White)
 			g.Game.Resign(resignColor)
diff --git a/pkg/web/handlers/interceptors/chess.go b/pkg/web/handlers/interceptors/chess.go
@@ -573,6 +573,10 @@ func (b *Chess) SendMove(gameKey string, userID database.UserID, g *ChessGame, c
 	return nil
 }
 
+func (g *ChessGame) IsPlayer(userID database.UserID) bool {
+	return g.Player1.ID == userID || g.Player2.ID == userID
+}
+
 func (g *ChessGame) MoveStr(m string) {
 	validMoves := g.Game.Position().ValidMoves()
 	var mov chess.Move