kit: advanced commit view controller search field bugs have been fixed.

Author: lolgear

GitUpKit/Views/GIAdvancedCommitViewController.m

@@ -53,7 +53,7 @@ @implementation GIAdvancedCommitViewController {
 
 #pragma mark - Search
 - (void)setupSearch {
-  [NSNotificationCenter.defaultCenter addObserver:self selector:@selector(textDidChange:) name:NSTextDidChangeNotification object:nil];
+  self.searchTextField.delegate = self;
 }
 
 - (void)resetSearch {
@@ -63,17 +63,16 @@ - (void)resetSearch {
   self.searchTextField.nextResponder = self.messageTextView;
 }
 
-- (void)unsetSearch {
-  [NSNotificationCenter.defaultCenter removeObserver:self];
-}
-
-- (void)textDidChange:(NSNotification *)notification {
-  NSString *text = self.searchTextField.stringValue;
-  if ([@"" isEqualToString:text]) {
-    [self.repository updateFilePattern:nil];
-  }
-  else {
-    [self.repository updateFilePattern:text];
+#pragma mark - NSControlTextEditingDelegate
+- (void)controlTextDidChange:(NSNotification *)obj {
+  if (obj.object == self.searchTextField) {
+    NSString *text = self.searchTextField.stringValue;
+    if ([@"" isEqualToString:text]) {
+      [self.repository updateFilePattern:nil];
+    }
+    else {
+      [self.repository updateFilePattern:text];
+    }
   }
 }
 
@@ -138,10 +137,6 @@ - (void)viewWillDisappear {
   [self resetSearch];
 }
 
-- (void)dealloc {
-  [self unsetSearch];
-}
-
 #pragma mark - Repository Handling
 - (void)repositoryStatusDidUpdate {
   [super repositoryStatusDidUpdate];