3-group-ios/GBA002
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixes Game Save errors not being grouped with Game errors

Browse Source
This commit is contained in:
Riley Testut 2019-03-22 13:04:34 -07:00
parent 8bc9d02e4c
commit 9db68aa9e4
1 changed files with 31 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
Delta/Syncing/SyncResultViewController.swift
View File

@ -45,6 +45,20 @@ extension SyncResultViewController
}
}
extension Record
{
var localizedTitle: String {
guard let type = SyncManager.RecordType(rawValue: self.recordID.type) else { return self.localizedName ?? NSLocalizedString("Unknown", comment: "") }
switch type
{
case .game: return NSLocalizedString("Game", comment: "")
case .gameSave: return NSLocalizedString("Game Save", comment: "")
case .saveState, .cheat, .controllerSkin, .gameCollection, .gameControllerInputMapping: return self.localizedName ?? type.localizedName
}
}
}
class SyncResultViewController: UITableViewController
{
var result: Result<[Record<NSManagedObject>: Result<Void, RecordError>], SyncError>!
@ -114,14 +128,7 @@ private extension SyncResultViewController
switch error.value
{
case let error as RecordError:
guard let recordType = SyncManager.RecordType(rawValue: error.record.recordID.type) else { return }
switch recordType
{
case .game: title = NSLocalizedString("Game", comment: "")
case .gameSave: title = NSLocalizedString("Game Save", comment: "")
case .saveState, .cheat, .controllerSkin, .gameCollection, .gameControllerInputMapping: title = error.record.localizedName ?? recordType.localizedName
}
title = error.record.localizedTitle
switch error
{
@ -204,11 +211,16 @@ private extension SyncResultViewController
switch recordType
{
case .game: group = .game(error.record.recordID)
case .gameSave: group = .game(error.record.recordID)
case .gameCollection: group = .gameCollection
case .controllerSkin: group = .controllerSkin
case .gameControllerInputMapping: group = .gameControllerInputMapping
case .gameSave:
guard let gameID = error.record.metadata?[.gameID] else { continue }
let recordID = RecordID(type: SyncManager.RecordType.game.rawValue, identifier: gameID)
group = .game(recordID)
case .saveState:
guard let gameID = error.record.metadata?[.gameID] else { continue }
@ -228,6 +240,16 @@ private extension SyncResultViewController
errorsByGroup[group, default: []].append(error)
}
for (group, errors) in errorsByGroup
{
let sortedErrors = errors.sorted { (a, b) -> Bool in
guard let a = a as? RecordError, let b = b as? RecordError else { return false }
return a.record.localizedTitle < b.record.localizedTitle
}
errorsByGroup[group] = sortedErrors
}
let sortedErrors = errorsByGroup.sorted { (a, b) in
let groupA = a.key
let groupB = b.key