First pass at handling remote deletions.

Classes/PTProject.h

@@ -18,7 +18,6 @@
 
 + (NSEntityDescription *)entityFromContext:(NSManagedObjectContext *)context;
 
-+ (NSArray *)findAll:(NSManagedObjectContext *)inContext;
 + (NSArray *)findInContext:(NSManagedObjectContext *)inContext predicate:(NSPredicate *)predicate;
 
 + (id)findAllRemote:(id<PTResultsDelegate>)resultsDelegate;

Classes/PTProject.m

@@ -76,11 +76,6 @@ - (void)syncSelfToManagedObject:(NSManagedObject *)object;
 #pragma mark -
 #pragma mark Queries
 
-+ (NSArray *)findAll:(NSManagedObjectContext *)inContext;
-{
-  return [self findInContext:inContext predicate:nil];
-}
-
 + (NSArray *)findInContext:(NSManagedObjectContext *)inContext predicate:(NSPredicate *)predicate;
 {
   NSFetchRequest *fetchRequest = [[NSFetchRequest alloc] init];

Classes/PTSyncManager.m

@@ -24,6 +24,12 @@ - (id)initWithManagedObjectContext:(NSManagedObjectContext *)context;
   return self;
 }
 
+- (void)dealloc;
+{
+  [managedObjectContext release];
+  [super dealloc];
+}
+
 - (void)synchronizeRemote:(id)remoteModel;
 {
   NSAssert1([remoteModel respondsToSelector:@selector(findAllRemote:)], 
@@ -40,20 +46,32 @@ - (void)remoteModel:(id)modelKlass didFinishLoading:(NSArray *)results;
 {
   NSEntityDescription *entity = [modelKlass performSelector:@selector(entityFromContext:) withObject:managedObjectContext];
 
-
   // TODO it seems wrong that remoteId is hardcoded here, what if I want to use UUID instead?
   NSPredicate *predicate = [NSPredicate predicateWithFormat:@"remoteId in %@", [results valueForKeyPath:@"remoteId"]];
   NSFetchRequest *fetchRequest = [[NSFetchRequest alloc] init];
   [fetchRequest setEntity:entity];
   [fetchRequest setPredicate:predicate];
 
-  NSArray *managedObjectsForResults = [managedObjectContext executeFetchRequest:fetchRequest error:nil];
+  NSSet *managedObjectsForResultsSet = [NSSet setWithArray:[managedObjectContext executeFetchRequest:fetchRequest error:nil]];
   [fetchRequest release];
 
+  // delete all objects that no longer exist on the server
+  NSFetchRequest *fetchRequestForAll = [[NSFetchRequest alloc] init];
+  [fetchRequestForAll setEntity:entity];  
+  NSArray *fetchResults = [managedObjectContext executeFetchRequest:fetchRequestForAll error:nil];
+  [fetchRequestForAll release];
+
+  NSMutableSet *allObjectSet = [[NSMutableSet alloc] initWithArray:fetchResults];
+  [allObjectSet minusSet:managedObjectsForResultsSet];
+  for (NSManagedObject *object in allObjectSet) {
+    [managedObjectContext deleteObject:object];
+  }
+  [allObjectSet release];
+
   // the reason I'm munging this into dictionary keyed by remote ID is to make it easier
   // to look up an existing NSManagedObject for a given record, perhaps there is a better way?
   NSMutableDictionary *managedObjectsByRemoteId = [NSMutableDictionary dictionary];
-  for (NSManagedObject *object in managedObjectsForResults) {
+  for (NSManagedObject *object in managedObjectsForResultsSet) {
     [managedObjectsByRemoteId setObject:object forKey:[object valueForKey:@"remoteId"]];
   }
 

Classes/PTTrackerRemoteModel.h

@@ -21,4 +21,6 @@
 - (void)setManagedObject:(NSManagedObject *)object isMaster:(BOOL)isMaster;
 - (void)syncManagedObjectToSelf:(NSManagedObject *)_managedObject;
 - (void)syncSelfToManagedObject:(NSManagedObject *)_managedObject;
+
++ (NSArray *)findAll:(NSManagedObjectContext *)inContext;
 @end

Classes/PTTrackerRemoteModel.m

@@ -63,6 +63,14 @@ - (void)syncSelfToManagedObject:(NSManagedObject *)object;
   self.remoteId = [object valueForKey:@"remoteId"];
 }
 
+#pragma mark -
+#pragma mark Queries
+
++ (NSArray *)findAll:(NSManagedObjectContext *)inContext;
+{
+  return [self findInContext:inContext predicate:nil];
+}
+
 #pragma mark -
 #pragma mark Default HRRestConnection delegate methods