-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Modernize all the syntax!! #60
Modernize all the syntax!! #60
Conversation
instead of defaulting to SQLite
in unit tests
…rmatting # Conflicts: # Paging Data Source Example/Core Data/Data Sources/VOKPlayerDataSource.h # Paging Data Source Example/VOKPagingViewController.h # Pod/Classes/VOKCoreDataManager.m # SampleProject/DataSource/VOKPersonDataSource.h # SampleProject/Models/VOKMappablePerson.h # SampleProject/VOKCoreDataManagerTests/CoreDataManagerDeleteTests.m # SampleProject/VOKCoreDataManagerTests/ManagedObjectImportExportTests.m # SampleProject/VOKViewController.h
@@ -223,24 +223,24 @@ - (void)tableView:(UITableView *)tableView didDeselectRowAtIndexPath:(NSIndexPat | |||
|
|||
- (NSInteger)numberOfSectionsInTableView:(UITableView *)tableView | |||
{ | |||
NSInteger sectionCount = [[_fetchedResultsController sections] count]; | |||
NSInteger sectionCount = _fetchedResultsController.sections.count; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this still need to access the _fetchedResultsController
or can it grab self.fetchedResultsController
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can probably make it work as a property
@@ -188,7 +188,7 @@ - (void)initPersistentStoreCoordinator | |||
self.migrationFailureOptions == VOKMigrationFailureOptionWipeRecovery) { | |||
VOK_CDLog(@"Full database delete and rebuild"); | |||
[[NSFileManager defaultManager] removeItemAtPath:storeURL.path error:nil]; | |||
if (![_persistentStoreCoordinator addPersistentStoreWithType:NSSQLiteStoreType | |||
if (![_persistentStoreCoordinator addPersistentStoreWithType:storeType |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
whoooops
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
although, how often to we have store types than than SQLite?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just about never, which is why nobody noticed this
- (NSFetchedResultsController *)fetchedResultsController | ||
{ | ||
if (!_fetchedResultsController) { | ||
[self initFetchedResultsController]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Without this lazy-loader, is initFetchedResultsController
being called somewhere else?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, the init method
Cool, LGTM |
@designatednerd, what say you? |
Modernize all the syntax!!
Does what it says on the tin.
dispatch_async
@vokal/ios-developers, want to look at some dot syntax?