[Core Data] Make new Core Data migration path. Remove models below 30 #15987
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,19 @@ | ||
| import CoreData | ||
|
|
||
| struct CoreDataMigratorUtils { | ||
| /// Finds the ModelVersion that corresponds to the given NSManagedObjectModel | ||
| static func findSourceVersion(for sourceModel: NSManagedObjectModel, | ||
| in modelsInventory: ManagedObjectModelsInventory) -> ManagedObjectModelsInventory.ModelVersion? { | ||
| do { | ||
| let allModels = try modelsInventory.models(for: modelsInventory.versions) | ||
| for (index, model) in allModels.enumerated() { | ||
| if model.isEqual(sourceModel) { | ||
| return modelsInventory.versions[index] | ||
| } | ||
| } | ||
|
Comment on lines
+8
to
+13
Contributor
There was a problem hiding this comment. ❓ Trying to ensure that it can return a deleted version. When I printed out If it's not possible, I think the code can be simplified by not having to compare with the oldest model version constant, since the available models in
Contributor
Author
There was a problem hiding this comment. That's a very good catch, and excellent suggestion. We can just rely on the existence of the model rather than having to check the version, which simplifies the logic and allows us to remove some. Updated here: 6080964 |
||
| } catch { | ||
| DDLogError("[CoreDataMigratorUtils] Error loading models for version detection: \(error)") | ||
| } | ||
| return nil | ||
| } | ||
| } | ||
Uh oh!
There was an error while loading. Please reload this page.