Skip to content

Commit

Permalink
on startObservering we listen to changes for any moc for a given pers…
Browse files Browse the repository at this point in the history
…istent store
  • Loading branch information
vijayj committed Jul 24, 2014
1 parent 154a9cd commit a72feeb
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions ParcelKit/PKSyncManager.m
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,7 @@ - (void)startObserving
});
}];

[[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(managedObjectContextWillSave:) name:NSManagedObjectContextWillSaveNotification object:self.managedObjectContext];
[[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(managedObjectContextWillSave:) name:NSManagedObjectContextWillSaveNotification object:nil];
}

- (void)stopObserving
Expand All @@ -175,7 +175,7 @@ - (void)stopObserving
self.persistentStoreCoordinator = nil;

[self.datastore removeObserver:self];
[[NSNotificationCenter defaultCenter] removeObserver:self name:NSManagedObjectContextWillSaveNotification object:self.managedObjectContext];
[[NSNotificationCenter defaultCenter] removeObserver:self name:NSManagedObjectContextWillSaveNotification object:nil];
}

#pragma mark - Updating Core Data
Expand Down Expand Up @@ -266,7 +266,7 @@ - (void)managedObjectContextWillSave:(NSNotification *)notification
if (![self isObserving]) return;

NSManagedObjectContext *managedObjectContext = notification.object;
if (self.managedObjectContext != managedObjectContext) return;
if (self.managedObjectContext.persistentStoreCoordinator != managedObjectContext.persistentStoreCoordinator) return;

NSSet *deletedObjects = [managedObjectContext deletedObjects];
for (NSManagedObject *managedObject in [self syncableManagedObjectsFromManagedObjects:deletedObjects]) {
Expand Down
4 changes: 2 additions & 2 deletions ParcelKitTests/PKSyncManagerTests.m
Original file line number Diff line number Diff line change
Expand Up @@ -220,10 +220,10 @@ - (void)testStartObservingShouldObserveDatastore
[self.datastore verify];
}

- (void)testStartObservingShouldObserveManagedObjectContext
- (void)testStartObservingShouldObserveAllChangesToMoc
{
id mockNotificationCenter = [OCMockObject niceMockForClass:[NSNotificationCenter class]];
[[mockNotificationCenter expect] addObserver:self.syncManager selector:[OCMArg anySelector] name:NSManagedObjectContextWillSaveNotification object:self.syncManager.managedObjectContext];
[[mockNotificationCenter expect] addObserver:self.syncManager selector:[OCMArg anySelector] name:NSManagedObjectContextWillSaveNotification object:nil];

id mockNotificationCenterClass = [OCMockObject mockForClass:[NSNotificationCenter class]];
[[[mockNotificationCenterClass expect] andReturn:mockNotificationCenter] defaultCenter];
Expand Down

0 comments on commit a72feeb

Please sign in to comment.