Remove instance variables that have a FIXME on them

Reference the repository and windowController proeprties instead

Author: ksuther

Classes/Controllers/PBGitCommitController.m

@@ -98,7 +98,7 @@ - (void)awakeFromNib
 																					   ascending:true]]];
 
 	// listen for updates
-	[[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(_repositoryUpdatedNotification:) name:PBGitRepositoryEventNotification object:repository];
+	[[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(_repositoryUpdatedNotification:) name:PBGitRepositoryEventNotification object:self.repository];
 
 	[stagedFilesController setAutomaticallyRearrangesObjects:NO];
 	[unstagedFilesController setAutomaticallyRearrangesObjects:NO];
@@ -145,12 +145,12 @@ - (NSResponder *)firstResponder;
 
 - (PBGitIndex *)index
 {
-	return repository.index;
+	return self.repository.index;
 }
 
 - (void)commitWithVerification:(BOOL)doVerify
 {
-	if ([[NSFileManager defaultManager] fileExistsAtPath:[repository.gitURL.path stringByAppendingPathComponent:@"MERGE_HEAD"]]) {
+	if ([[NSFileManager defaultManager] fileExistsAtPath:[self.repository.gitURL.path stringByAppendingPathComponent:@"MERGE_HEAD"]]) {
 		NSString *message = NSLocalizedString(@"Cannot commit merges",
 											  @"Title for sheet that GitX cannot create merge commits");
 		NSString *info = NSLocalizedString(@"GitX cannot commit merges yet. Please commit your changes from the command line.",
@@ -188,7 +188,7 @@ - (void)commitWithVerification:(BOOL)doVerify
 	self.isBusy = YES;
 	commitMessageView.editable = NO;
 
-	[repository.index commitWithMessage:commitMessage andVerify:doVerify];
+	[self.repository.index commitWithMessage:commitMessage andVerify:doVerify];
 }
 
 - (void)discardChangesForFiles:(NSArray *)files force:(BOOL)force
@@ -217,7 +217,7 @@ - (void)discardChangesForFiles:(NSArray *)files force:(BOOL)force
 - (IBAction)signOff:(id)sender
 {
 	NSError *error = nil;
-	GTConfiguration *config = [repository.gtRepo configurationWithError:&error];
+	GTConfiguration *config = [self.repository.gtRepo configurationWithError:&error];
 	NSString *userName = [config stringForKey:@"user.name"];
 	NSString *userEmail = [config stringForKey:@"user.email"];
 	if (!(userName && userEmail)) {
@@ -243,10 +243,10 @@ - (IBAction)refresh:(id)sender
 {
 	self.isBusy = YES;
 	self.status = NSLocalizedString(@"Refreshing index…", @"Message in status bar while the index is refreshing");
-	[repository.index refresh];
+	[self.repository.index refresh];
 
 	// Reload refs (in case HEAD changed)
-	[repository reloadRefs];
+	[self.repository reloadRefs];
 }
 
 - (IBAction)commit:(id)sender

Classes/Controllers/PBGitHistoryController.m

@@ -88,6 +88,8 @@ - (void)loadView
 
 	self.selectedCommitDetailsIndex = [[NSUserDefaults standardUserDefaults] integerForKey:kHistorySelectedDetailIndexKey];
 
+	PBGitRepository *repository = self.repository;
+
 	[commitController addObserver:self
 						  keyPath:@"selection"
 						  options:0
@@ -265,6 +267,8 @@ + (NSSet *)keyPathsForValuesAffectingSingleNonHeadCommitSelected
 
 - (void)updateBranchFilterMatrix
 {
+	PBGitRepository *repository = self.repository;
+
 	if ([repository.currentBranch isSimpleRef]) {
 		[allBranchesFilterItem setEnabled:YES];
 		[localRemoteBranchesFilterItem setEnabled:YES];
@@ -321,7 +325,7 @@ - (NSInteger)selectedCommitDetailsIndex
 
 - (void)updateStatus
 {
-	self.isBusy = repository.revisionList.isUpdating;
+	self.isBusy = self.repository.revisionList.isUpdating;
 	self.status = [NSString stringWithFormat:@"%lu commits loaded", [[commitController arrangedObjects] count]];
 }
 
@@ -417,6 +421,8 @@ - (IBAction)setTreeView:(id)sender
 
 - (IBAction)setBranchFilter:(id)sender
 {
+	PBGitRepository *repository = self.repository;
+
 	repository.currentBranchFilter = [(NSView *)sender tag];
 	[PBGitDefaults setBranchFilter:repository.currentBranchFilter];
 	[self updateBranchFilterMatrix];
@@ -426,7 +432,7 @@ - (IBAction)setBranchFilter:(id)sender
 - (void)keyDown:(NSEvent *)event
 {
 	if ([[event charactersIgnoringModifiers] isEqualToString:@"f"] && [event modifierFlags] & NSAlternateKeyMask && [event modifierFlags] & NSCommandKeyMask)
-		[superController.window makeFirstResponder:searchField];
+		[self.windowController.window makeFirstResponder:searchField];
 	else
 		[super keyDown:event];
 }
@@ -518,7 +524,7 @@ - (void)updateQuicklookForce:(BOOL)force
 
 - (IBAction)refresh:(id)sender
 {
-	[repository forceUpdateRevisions];
+	[self.repository forceUpdateRevisions];
 }
 
 - (void)updateView
@@ -628,7 +634,7 @@ - (void)checkoutFiles:(id)sender
 		[files addObject:[filePath stringByTrimmingCharactersInSet:[NSCharacterSet whitespaceCharacterSet]]];
 
 	NSError *error = nil;
-	BOOL success = [repository checkoutFiles:files fromRefish:self.selectedCommits.firstObject error:&error];
+	BOOL success = [self.repository checkoutFiles:files fromRefish:self.selectedCommits.firstObject error:&error];
 	if (!success) {
 		[self.windowController showErrorSheet:error];
 	}
@@ -664,7 +670,7 @@ - (BOOL)tableView:(NSTableView *)tv writeRowsWithIndexes:(NSIndexSet *)rowIndexe
 		if ([ref isTag] || [ref isRemoteBranch])
 			return NO;
 
-		if ([[[repository headRef] ref] isEqualToRef:ref])
+		if ([[[self.repository headRef] ref] isEqualToRef:ref])
 			return NO;
 
 		NSData *data = [NSKeyedArchiver archivedDataWithRootObject:[NSArray arrayWithObjects:[NSNumber numberWithInteger:row], [NSNumber numberWithInt:index], NULL]];
@@ -779,6 +785,7 @@ - (NSArray *)menuItemsForPaths:(NSArray *)paths
 														 action:@selector(showCommitsFromTree:)
 												  keyEquivalent:@""];
 
+	PBGitRepository *repository = self.repository;
 	PBGitRef *headRef = [[repository headRef] ref];
 	NSString *headRefName = [headRef shortName];
 	NSString *diffTitleFormat = multiple ? NSLocalizedString(@"Diff files with %@", @"Diff with ref menu item for multiple files") : NSLocalizedString(@"Diff file with %@", @"Diff with ref menu item for single file");

Classes/Controllers/PBGitSidebarController.m

@@ -67,6 +67,8 @@ - (void)awakeFromNib
 	window.contentView = self.view;
 	[self populateList];
 
+	PBGitRepository *repository = self.repository;
+
 	[repository addObserver:self
 					keyPath:@"currentBranch"
 					options:0
@@ -147,6 +149,7 @@ - (void)selectStage
 
 - (void)selectCurrentBranch
 {
+	PBGitRepository *repository = self.repository;
 	PBGitRevSpecifier *rev = repository.currentBranch;
 	if (!rev) {
 		[repository reloadRefs];
@@ -234,18 +237,19 @@ - (void)outlineViewSelectionDidChange:(NSNotification *)notification
 {
 	NSInteger index = [sourceView selectedRow];
 	PBSourceViewItem *item = [sourceView itemAtRow:index];
+	PBGitWindowController *windowController = self.windowController;
 
 	if ([item revSpecifier]) {
-		if (![repository.currentBranch isEqual:[item revSpecifier]]) {
-			repository.currentBranch = [item revSpecifier];
+		if (![self.repository.currentBranch isEqual:[item revSpecifier]]) {
+			self.repository.currentBranch = [item revSpecifier];
 		}
 
-		[superController changeContentController:superController.historyViewController];
+		[windowController changeContentController:windowController.historyViewController];
 		[PBGitDefaults setShowStageView:NO];
 	}
 
 	if (item == stage) {
-		[superController changeContentController:superController.commitViewController];
+		[windowController changeContentController:windowController.commitViewController];
 		[PBGitDefaults setShowStageView:YES];
 	}
 
@@ -266,7 +270,7 @@ - (void)doubleClicked:(id)object
 		PBSourceViewGitBranchItem *branch = item;
 
 		NSError *error = nil;
-		BOOL success = [repository checkoutRefish:[branch ref] error:&error];
+		BOOL success = [self.repository checkoutRefish:[branch ref] error:&error];
 		if (!success) {
 			[self.windowController showErrorSheet:error];
 		}
@@ -292,7 +296,7 @@ - (NSView *)outlineView:(NSOutlineView *)outlineView viewForTableColumn:(NSTable
 
 	cell.textField.stringValue = [[item title] copy];
 	cell.imageView.image = item.icon;
-	cell.isCheckedOut = [item.revSpecifier isEqual:[repository headRef]];
+	cell.isCheckedOut = [item.revSpecifier isEqual:[self.repository headRef]];
 
 	return cell;
 }
@@ -324,6 +328,7 @@ - (BOOL)outlineView:(NSOutlineView *)outlineView shouldShowOutlineCellForItem:(i
 
 - (void)populateList
 {
+	PBGitRepository *repository = self.repository;
 	PBSourceViewItem *project = [PBSourceViewItem groupItemWithTitle:[repository projectName]];
 	project.uncollapsible = YES;
 
@@ -415,7 +420,7 @@ - (void)addMenuItemsForRef:(PBGitRef *)ref toMenu:(NSMenu *)menu
 	if (!ref)
 		return;
 
-	for (NSMenuItem *menuItem in [superController.historyViewController menuItemsForRef:ref])
+	for (NSMenuItem *menuItem in [self.windowController.historyViewController menuItemsForRef:ref])
 		[menu addItem:menuItem];
 }
 
@@ -488,7 +493,7 @@ - (void)updateRemoteControls
 	BOOL hasRemote = NO;
 
 	PBGitRef *ref = [[self selectedItem] ref];
-	if ([ref isRemote] || ([ref isBranch] && [[repository remoteRefForBranch:ref error:NULL] remoteName]))
+	if ([ref isRemote] || ([ref isBranch] && [[self.repository remoteRefForBranch:ref error:NULL] remoteName]))
 		hasRemote = YES;
 
 	[remoteControls setEnabled:hasRemote forSegment:kFetchSegment];
@@ -515,7 +520,7 @@ - (IBAction)fetchPullPushAction:(id)sender
 	if (![ref isRemote] && ![ref isBranch])
 		return;
 
-	PBGitRef *remoteRef = [repository remoteRefForBranch:ref error:NULL];
+	PBGitRef *remoteRef = [self.repository remoteRefForBranch:ref error:NULL];
 	if (!remoteRef)
 		return;
 

Classes/Controllers/PBViewController.h

@@ -11,11 +11,7 @@
 #import "PBGitRepository.h"
 #import "PBGitWindowController.h"
 
-@interface PBViewController : NSViewController {
-	// FIXME: these ivars must go, but most controller out there access it directly, so, not today
-	__weak PBGitRepository *repository;
-	__weak PBGitWindowController *superController;
-}
+@interface PBViewController : NSViewController
 
 @property (nonatomic, weak, readonly) PBGitRepository *repository;
 @property (weak, readonly) PBGitWindowController *windowController;