Errata for Core Data (2nd edition)

#53055: Sec 1.4 The interface property for "serves" should be a NSUInteger type instead of an NSString. This would be consistent with the CD model presented in Figure 2 in Sec 1.1.--Bruce Payan

#51137: NSManagedObjectContext ... Therefor, we hand off the reference to it to our AppDelegate so the the rest of the application can access it.

This is great but in the examples you don't do this.

//
// PPRAppDelegate.h
// PPRecipes
//
// Created by Marcus Zarra on 4/3/12.
// Copyright (c) 2012 The Pragmatic Programmer. All rights reserved.
//

@interface PPRAppDelegate : UIResponder <UIApplicationDelegate>

@property (strong, nonatomic) UIWindow *window;

- (void)saveContext;

@end

--- and part of the .m file

#import "PPRAppDelegate.h"

#import "PPRMasterViewController.h"
#import "PPRImportOperation.h"

@interface PPRAppDelegate()

@property (nonatomic, strong) NSManagedObjectContext *managedObjectContext;

@property (nonatomic, strong) NSURL *fileToOpenURL;

- (void)initializeCoreDataStack;
- (void)contextInitialized;

- (void)consumeIncomingFileURL:(NSURL*)url;

@end

@implementation PPRAppDelegate

@synthesize window;
@synthesize managedObjectContext;
@synthesize fileToOpenURL;

I'm use MVCS and the store needs access to the moc I recommend something like this: in the .h

#define ApplicationDelegate ((AppDelegate *)[UIApplication sharedApplication].delegate)

@interface AppDelegate : UIResponder <UIApplicationDelegate>

@property (strong, nonatomic) UIWindow *window;

// Notice how this is called mainManagedObjectContext all the local instances are called
// ManagedObjectContext this is because core data is not thread safe and each access into it needs to be on the thread that calls it.
@property (strong, nonatomic) NSManagedObjectContext *mainManagedObjectContext;

//This is new it is thread safe
- (void)saveContext;

and in .m change to:

@interface AppDelegate()

//Place the private methods in this category
- (void)initializeCoreDataStack;
- (void)contextInitialized;

@end

#pragma mark - Core Data stack

- (void)initializeCoreDataStack
{
NSURL *modelURL = [[NSBundle mainBundle] URLForResource:@"GAIGAgent"
withExtension:@"momd"];
ZAssert(modelURL, @"Failed to find model URL");

NSManagedObjectModel *mom = nil;
mom = [[NSManagedObjectModel alloc] initWithContentsOfURL:modelURL];
ZAssert(mom, @"Failed to initialize model");

NSPersistentStoreCoordinator *psc = nil;
psc = [[NSPersistentStoreCoordinator alloc] initWithManagedObjectModel:mom];
ZAssert(psc, @"Failed to initialize persistent store coordinator");

NSManagedObjectContext *moc = nil;
NSManagedObjectContextConcurrencyType ccType = NSMainQueueConcurrencyType;
moc = [[NSManagedObjectContext alloc] initWithConcurrencyType:ccType];
[moc setPersistentStoreCoordinator:psc];
//[self setManagedObjectContext:moc];
self.mainManagedObjectContext = moc;

...

so basically make the mainManagedObjectContext accessible so the rest of the app can use it.

The way you have it does work when you set the mod up in the didFinishLaunchingWithOptions but this is not usable when using the MVCS pattern were all the data goes thru the store and thus the store needs access to the "main" moc

--Dave Koz

#50503: These two lines are not correct:

NSArray *newArray = [NSArray arrayWithObject:newIndexPath];
NSArray *oldArray = [NSArray arrayWithObject:indexPath];

arrayWithObject raises an error if its argument is nil, which both of these can be depending upon the mutation type.
--Pete Hopkins

#50838: finalPredicate never declared either in the class or within the local method scope. You might want to check the code being posted on media.pragprog.com too.--Anson Chen

#51654: (Or a suggestion)
1) The results mDictionary is local to -contextUpdated and never assigned/passed so how does it "...pass the newly created NSDictionary to our delegate."? (p. 36)
2) Also the book code version has action in the class extension (changed in the one on GitHub) and it's never set in the existing code.

Were these meant to just be starter code that has to be amended to be functional? The book presentation suggests otherwise but certainly could have misinterpreted that.--Randy Scovil

#50502: The creation of the UIAlertView is done within the dispatch_async block on the global queue, which causes an error. It should be done on a block that's run on the main UI queue.--Pete Hopkins

#76286: Since this is all about getting the export off the main thread, adding the export operation to the mainQueue makes no sense. (The import code has the same problem.)--Randy Saldinger

#52057: In the first code example on the page, this line is incorrect:
[dict addEntriesFromDictionary:[self moToDictionary:childMO]];

It should read:
[dict setValue:[self moToDictionary:childMO] forKey:key];

Otherwise, the resulting NSDictionary will be a combination of keys and values from multiple entities, which is incorrect.--Kyle Oba

#51215: Book describes

5.5 Parent-Child NSManagedObjectContext Instances
With the release of iOS 6.0 and Mac OS X 10.8 Mountain Lion, a number of new features have been added to Core Data.

I believe these features were introduced with iOS 5.0 / Mac OS X 10.7. Certainly is seems -performBlock: and -performBlockAndWait: are available in iOS 5.0. I can't provide a link to docs as your form prevents it.--Ian Kershaw

#52058: I'm using Xcode 4.6.2. I'm having problems with your nifty Macros. Note: I have actually manually typed in the Macros (you provide in Appendix 2) into the <ProjectName>-Prefix.pch file after the Cut & Paste method raised some Compile Type Errors. But I'm receiving the same Compile Time Errors regardless of how I try to incorporate them into the .pch file. The errors are associated with (and localized within) the ALog() and ZAssert() macros. If I don't try to compile the manually entered content I don't receive any hints of any problem. Once I BUILD the source the same 5 Errors appear that I had before. Everything "looks" good, but the compiler is telling me otherwise. I'm kind of stumped. I can email the offending 5 lines of source code if that helps. --Mark Mendenhall