admin/Sportstime
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: rename fetch methods to filter, add allGames

Browse Source 
- fetchGames → filterGames (clarifies local SwiftData query)
- fetchRichGames → filterRichGames
- Add allGames(for:) method for unfiltered game access

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Trey t
2026-01-12 10:51:26 -06:00
parent b89c0d58e2
commit b3ad386d2b
1 changed files with 28 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
SportsTime/Core/Services/DataProvider.swift
View File

@@ -118,8 +118,8 @@ final class AppDataProvider: ObservableObject {
// MARK: - Game Fetching // MARK: - Game Fetching
/// Fetch games from SwiftData within date range /// Filter games from SwiftData within date range
func fetchGames(sports: Set<Sport>, startDate: Date, endDate: Date) async throws -> [Game] { func filterGames(sports: Set<Sport>, startDate: Date, endDate: Date) async throws -> [Game] {
guard let context = modelContext else { guard let context = modelContext else {
throw DataProviderError.contextNotConfigured throw DataProviderError.contextNotConfigured
} }
@@ -143,6 +143,29 @@ final class AppDataProvider: ObservableObject {
} }
} }
/// Get all games for specified sports (no date filtering)
func allGames(for sports: Set<Sport>) async throws -> [Game] {
guard let context = modelContext else {
throw DataProviderError.contextNotConfigured
}
let sportStrings = sports.map { $0.rawValue }
let descriptor = FetchDescriptor<CanonicalGame>(
predicate: #Predicate<CanonicalGame> { game in
game.deprecatedAt == nil
},
sortBy: [SortDescriptor(\.dateTime)]
)
let canonicalGames = try context.fetch(descriptor)
return canonicalGames.compactMap { canonical -> Game? in
guard sportStrings.contains(canonical.sport) else { return nil }
return canonical.toDomain()
}
}
/// Fetch a single game by canonical ID /// Fetch a single game by canonical ID
func fetchGame(by id: String) async throws -> Game? { func fetchGame(by id: String) async throws -> Game? {
guard let context = modelContext else { guard let context = modelContext else {
@@ -160,9 +183,9 @@ final class AppDataProvider: ObservableObject {
return canonical.toDomain() return canonical.toDomain()
} }
/// Fetch games with full team and stadium data /// Filter games with full team and stadium data within date range
func fetchRichGames(sports: Set<Sport>, startDate: Date, endDate: Date) async throws -> [RichGame] { func filterRichGames(sports: Set<Sport>, startDate: Date, endDate: Date) async throws -> [RichGame] {
let games = try await fetchGames(sports: sports, startDate: startDate, endDate: endDate) let games = try await filterGames(sports: sports, startDate: startDate, endDate: endDate)
return games.compactMap { game in return games.compactMap { game in
guard let homeTeam = teamsById[game.homeTeamId], guard let homeTeam = teamsById[game.homeTeamId],