From 2c2bc4f297f44dcd9edb82381d084c3d32350181 Mon Sep 17 00:00:00 2001 From: brookjeynes Date: Wed, 15 Oct 2025 12:02:39 +1000 Subject: [PATCH] feat(handlers/stats): use slogger Change-Id: tyrzuxkutmttkpqpqlxzztownwzuprlx Signed-off-by: brookjeynes --- internal/server/handlers/stats.go | 25 ++++++++++++++----------- 1 file changed, 14 insertions(+), 11 deletions(-) diff --git a/internal/server/handlers/stats.go b/internal/server/handlers/stats.go index cc61b21..b4b7b36 100644 --- a/internal/server/handlers/stats.go +++ b/internal/server/handlers/stats.go @@ -1,7 +1,6 @@ package handlers import ( - "log" "net/http" "yoten.app/internal/clients/bsky" @@ -12,9 +11,11 @@ import ( ) func (h *Handler) HandleTimePerGraphs(w http.ResponseWriter, r *http.Request) { + l := h.Logger.With("handler", "HandleTimePerGraphs") + user, err := bsky.GetUserWithBskyProfile(h.Oauth, r) if err != nil { - log.Println("failed to get logged-in user:", err) + l.Error("failed to get logged-in user", "err", err) htmx.HxRedirect(w, "/login") return } @@ -24,7 +25,7 @@ func (h *Handler) HandleTimePerGraphs(w http.ResponseWriter, r *http.Request) { chartData, err := db.GetTimePerData(h.Db, user.Did, period) if err != nil { - log.Println("failed to get time per chart data:", err) + l.Error("failed to get time per chart data", "err", err) chartData = db.ChartsData{ ActivityData: []db.ChartData{}, CategoryData: []db.ChartData{}, @@ -38,46 +39,48 @@ func (h *Handler) HandleTimePerGraphs(w http.ResponseWriter, r *http.Request) { } func (h *Handler) HandleStatsPage(w http.ResponseWriter, r *http.Request) { + l := h.Logger.With("handler", "HandleStatsPage") + user, err := bsky.GetUserWithBskyProfile(h.Oauth, r) if err != nil { - log.Println("failed to get logged-in user:", err) + l.Error("failed to get logged-in user", "err", err) htmx.HxRedirect(w, "/login") return } totalStudyTime, err := db.GetTotalStudyTime(h.Db, user.Did) if err != nil { - log.Println("failed to get total study time:", err) + l.Error("failed to get total study time", "err", err) } totalStudySessions, err := db.GetTotalStudySessions(h.Db, user.Did) if err != nil { - log.Println("failed to get total study study sessions:", err) + l.Error("failed to get total study study sessions", "err", err) } totalActiveDays, err := db.GetTotalActiveDays(h.Db, user.Did) if err != nil { - log.Println("failed to get total active days:", err) + l.Error("failed to get total active days", "err", err) } streak, err := db.GetCurrentStreak(h.Db, user.Did) if err != nil { - log.Println("failed to get streak:", err) + l.Error("failed to get streak", "err", err) } heatmap, err := db.GetHeatmapData(h.Db, user.Did) if err != nil { - log.Println("failed to get heatmap data:", err) + l.Error("failed to get heatmap data", "err", err) } inputOutputPercentage, err := db.GetInputOutputPercentage(h.Db, user.Did) if err != nil { - log.Println("failed to get input vs output data:", err) + l.Error("failed to get input vs output data", "err", err) } languageSummary, err := db.GetLanguageSummary(h.Db, user.Did) if err != nil { - log.Println("failed to get language time summary:", err) + l.Error("failed to get language time summary", "err", err) } languageChartSegments := db.ConvertToDonutChartSegments(languageSummary) -- 2.43.0