From 0c8133c802f5fd85d4244331924ee81a5ddd4988 Mon Sep 17 00:00:00 2001
From: ericprd <ericprd_@outlook.com>
Date: Thu, 20 Mar 2025 11:19:41 +0800
Subject: [PATCH] fix: don't record log if staff

---
 internal/api/http/news/get_by_slug.go | 56 ++++++++++++++-------------
 1 file changed, 29 insertions(+), 27 deletions(-)

diff --git a/internal/api/http/news/get_by_slug.go b/internal/api/http/news/get_by_slug.go
index e81adab..e0dd2bc 100644
--- a/internal/api/http/news/get_by_slug.go
+++ b/internal/api/http/news/get_by_slug.go
@@ -34,40 +34,42 @@ func GetBySlug(
 			return
 		}
 
-		var specReq logsdomain.LogsSpec
-
-		specReq.ContentID = news.ID
-
 		userDetail, _ := utils.GetTokenDetail(r)
 
-		if userDetail.ID != "" {
-			specReq.UserID = &userDetail.ID
-		}
+		if userDetail.Role != "staff" {
+			var specReq logsdomain.LogsSpec
 
-		ip := r.RemoteAddr
+			specReq.ContentID = news.ID
 
-		if forwarded := r.Header.Get("X-Forwarded-For"); forwarded != "" {
-			ip = strings.Split(forwarded, ",")[0]
-		}
+			if userDetail.ID != "" {
+				specReq.UserID = &userDetail.ID
+			}
 
-		if ip != "" {
-			specReq.IP = &ip
-		}
+			ip := r.RemoteAddr
 
-		if userAgent := r.UserAgent(); userAgent != "" {
-			specReq.UserAgent = &userAgent
-		}
+			if forwarded := r.Header.Get("X-Forwarded-For"); forwarded != "" {
+				ip = strings.Split(forwarded, ",")[0]
+			}
 
-		if err := logSvc.CreateLogNews(specReq); err != nil {
-			response.RespondJsonErrorWithCode(
-				ctx,
-				w,
-				err,
-				response.ErrBadRequest.Code,
-				response.ErrBadRequest.HttpCode,
-				err.Error(),
-			)
-			return
+			if ip != "" {
+				specReq.IP = &ip
+			}
+
+			if userAgent := r.UserAgent(); userAgent != "" {
+				specReq.UserAgent = &userAgent
+			}
+
+			if err := logSvc.CreateLogNews(specReq); err != nil {
+				response.RespondJsonErrorWithCode(
+					ctx,
+					w,
+					err,
+					response.ErrBadRequest.Code,
+					response.ErrBadRequest.HttpCode,
+					err.Error(),
+				)
+				return
+			}
 		}
 
 		response.RespondJsonSuccess(ctx, w, news)