Fixed endpoints for getting player 170 statistics

Author: thordy

CHANGELOG.md

@@ -6,6 +6,9 @@
 - Endpoint to get leaderboard data for each match type
 - New `Still Regining` badge when surviving a kill slayer attempt
 
+#### Fixed
+- Endpoints for getting player 170 statistics
+
 ## [2.9.0] - 2025-04-06
 #### Feature
 - Added method for updating venue of a match

controllers/player_controller.go

@@ -352,6 +352,16 @@ func GetPlayerMatchTypeStatistics(w http.ResponseWriter, r *http.Request) {
 		json.NewEncoder(w).Encode(stats)
 		return
 
+	case models.ONESEVENTY:
+		stats, err := data.Get170StatisticsForPlayer(id)
+		if err != nil {
+			log.Println("Unable to get 170 Statistics for player", err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		json.NewEncoder(w).Encode(stats)
+		return
+
 	default:
 		log.Println("Unknown match type parameter")
 		http.Error(w, err.Error(), http.StatusBadRequest)
@@ -542,6 +552,16 @@ func GetPlayerMatchTypeHistory(w http.ResponseWriter, r *http.Request) {
 		json.NewEncoder(w).Encode(legs)
 		return
 
+	case models.ONESEVENTY:
+		legs, err := data.Get170HistoryForPlayer(id, 0, limit)
+		if err != nil {
+			log.Println("Unable to get 170 history for player", err)
+			http.Error(w, err.Error(), http.StatusInternalServerError)
+			return
+		}
+		json.NewEncoder(w).Encode(legs)
+		return
+
 	default:
 		log.Println("Unknown match type parameter")
 		http.Error(w, err.Error(), http.StatusBadRequest)

data/statistics_170.go

@@ -195,6 +195,10 @@ func Get170StatisticsForPlayer(id int) (*models.Statistics170, error) {
 	err := models.DB.QueryRow(`
 			SELECT
 				p.id,
+				COUNT(DISTINCT m.id) AS 'matches_played',
+				COUNT(DISTINCT m2.id) AS 'matches_won',
+				COUNT(DISTINCT l.id) AS 'legs_played',
+				COUNT(DISTINCT l2.id) AS 'legs_won',
 				SUM(s.points),
 				SUM(s.ppd_score) / SUM(s.darts_thrown),
 				SUM(s.rounds),
@@ -218,8 +222,9 @@ func Get170StatisticsForPlayer(id int) (*models.Statistics170, error) {
 			WHERE s.player_id = ?
 				AND l.is_finished = 1 AND m.is_abandoned = 0 AND m.is_walkover = 0
 				AND m.match_type_id = 17
-			GROUP BY p.id`, id).Scan(&s.LegID, &s.PlayerID, &s.Points, &s.PPD, &s.Rounds, &s.CheckoutPercentage, &s.CheckoutAttempts,
-		&s.HighestCheckout, &s.DartsThrown, &darts9, &darts8, &darts7, &darts6, &darts5, &darts4, &darts3)
+			GROUP BY p.id`, id).Scan(&s.PlayerID, &s.MatchesPlayed, &s.MatchesWon, &s.LegsPlayed, &s.LegsWon,
+		&s.Points, &s.PPD, &s.Rounds, &s.CheckoutPercentage, &s.CheckoutAttempts, &s.HighestCheckout,
+		&s.DartsThrown, &darts9, &darts8, &darts7, &darts6, &darts5, &darts4, &darts3)
 	if err != nil {
 		if err == sql.ErrNoRows {
 			return new(models.Statistics170), nil
@@ -285,13 +290,24 @@ func Get170HistoryForPlayer(id int, start int, limit int) ([]*models.Leg, error)
 	legs = make([]*models.Leg, 0)
 	for rows.Next() {
 		s := new(models.Statistics170)
-		s.CheckoutDarts = make(map[int]int)
+		var darts9, darts8, darts7, darts6, darts5, darts4, darts3 int
+
 		err := rows.Scan(&s.LegID, &s.PlayerID, &s.Points, &s.PPD, &s.PPDScore, &s.Rounds, &s.CheckoutPercentage,
-			&s.CheckoutAttempts, &s.CheckoutCompleted, &s.HighestCheckout, s.CheckoutDarts[9], s.CheckoutDarts[8],
-			s.CheckoutDarts[7], s.CheckoutDarts[6], s.CheckoutDarts[5], s.CheckoutDarts[4], s.CheckoutDarts[3])
+			&s.CheckoutAttempts, &s.CheckoutCompleted, &s.HighestCheckout, &darts9, &darts8, &darts7, &darts6, &darts5,
+			&darts4, &darts3)
 		if err != nil {
 			return nil, err
 		}
+		checkoutDarts := make(map[int]int, 0)
+		checkoutDarts[9] = darts9
+		checkoutDarts[8] = darts8
+		checkoutDarts[7] = darts7
+		checkoutDarts[6] = darts6
+		checkoutDarts[5] = darts5
+		checkoutDarts[4] = darts4
+		checkoutDarts[3] = darts3
+		s.CheckoutDarts = checkoutDarts
+
 		leg := m[s.LegID]
 		leg.Statistics = s
 		legs = append(legs, leg)

models/statistics_170.go

@@ -9,6 +9,10 @@ type Statistics170 struct {
 	ID                 int         `json:"id"`
 	LegID              int         `json:"leg_id"`
 	PlayerID           int         `json:"player_id"`
+	MatchesPlayed      int         `json:"matches_played"`
+	MatchesWon         int         `json:"matches_won"`
+	LegsPlayed         int         `json:"legs_played"`
+	LegsWon            int         `json:"legs_won"`
 	Points             int         `json:"points"`
 	PPD                float32     `json:"ppd"`
 	PPDScore           int         `json:"-"`