Support for Core Data changes on iOS 11

Workaround for binary store changes on iOS 11. A better way to do this should be using NSBinaryStoreSecureDecodingClasses option in the future.
drewmccormack · Sep 10, 2017 · e7db197 · e7db197
1 parent 83cc4ae
commit e7db197
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 5 deletions.
diff --git a/Framework/Source/Events/CDEEventMigrator.m b/Framework/Source/Events/CDEEventMigrator.m
@@ -157,10 +157,17 @@ - (void)migrateEventsInFromFiles:(NSArray *)paths completion:(CDECompletionBlock
  NSDictionary *metadata = [NSPersistentStoreCoordinator metadataForPersistentStoreOfType:nil URL:fileURL error:&error];
  NSString *storeType = metadata[NSStoreTypeKey];
  if (!storeType) @throw [[NSException alloc] initWithName:CDEException reason:@"" userInfo:nil];
-
-#warning Using hard coded string here to get around problems in Xcode 9 Beta. Replace this with constant in final release
- NSDictionary *options = @{NSMigratePersistentStoresAutomaticallyOption: @YES, NSInferMappingModelAutomaticallyOption: @YES, @"_NSBinaryStoreInsecureDecodingCompatibilityOption": @YES};
+
+ NSDictionary *options = nil;
+ if (@available(iOS 11.0, *)) {
+ options = @{NSMigratePersistentStoresAutomaticallyOption: @YES, NSInferMappingModelAutomaticallyOption: @YES, NSBinaryStoreInsecureDecodingCompatibilityOption: @YES};
+ } else {
+ // Fallback on earlier versions
+ options = @{NSMigratePersistentStoresAutomaticallyOption: @YES, NSInferMappingModelAutomaticallyOption: @YES};
+ }
+
  fileStore = [importContext.persistentStoreCoordinator addPersistentStoreWithType:storeType configuration:nil URL:fileURL options:options error:&error];
+
  if (!fileStore) @throw [[NSException alloc] initWithName:CDEException reason:@"" userInfo:nil];
 
  BOOL success = [self migrateObjectsInContext:importContext toContext:self.eventStore.managedObjectContext error:&error];

diff --git a/Framework/Tests/CDEBaseliningSyncTests.m b/Framework/Tests/CDEBaseliningSyncTests.m
@@ -355,7 +355,12 @@ - (NSManagedObjectContext *)eventFileContextForURL:(NSURL *)baselineURL
  NSManagedObjectModel *eventModel = [[NSManagedObjectModel alloc] initWithContentsOfURL:modelURL];
  NSPersistentStoreCoordinator *coordinator = [[NSPersistentStoreCoordinator alloc] initWithManagedObjectModel:eventModel];
  baselineContext.persistentStoreCoordinator = coordinator;
- NSDictionary *options = @{@"_NSBinaryStoreInsecureDecodingCompatibilityOption": @YES};
+ NSDictionary *options = nil;
+
+ if (@available(iOS 11.0, *)) {
+ options = @{NSBinaryStoreInsecureDecodingCompatibilityOption: @YES};
+ }
+
  NSPersistentStore *store = [coordinator addPersistentStoreWithType:NSBinaryStoreType configuration:nil URL:baselineURL options:options error:NULL];
  XCTAssertNotNil(store, @"Store was nil");
  return baselineContext;

diff --git a/Framework/Tests/CDEEventMigratorTests.m b/Framework/Tests/CDEEventMigratorTests.m
@@ -115,7 +115,12 @@ - (NSManagedObjectContext *)makeFileContext
  NSURL *url = [NSURL fileURLWithPath:exportedEventsFile];
  NSManagedObjectModel *model = self.eventStore.managedObjectContext.persistentStoreCoordinator.managedObjectModel;
  NSPersistentStoreCoordinator *psc = [[NSPersistentStoreCoordinator alloc] initWithManagedObjectModel:model];
- NSDictionary *options = @{@"_NSBinaryStoreInsecureDecodingCompatibilityOption" : @YES};
+ NSDictionary *options = nil;
+
+ if (@available(iOS 11.0, *)) {
+ options = @{NSBinaryStoreInsecureDecodingCompatibilityOption: @YES};
+ }
+
  [psc addPersistentStoreWithType:NSBinaryStoreType configuration:nil URL:url options:options error:NULL];
  NSManagedObjectContext *newContext = [[NSManagedObjectContext alloc] initWithConcurrencyType:NSMainQueueConcurrencyType];
  newContext.persistentStoreCoordinator = psc;