Changeset 12897

Timestamp:

Sep 19, 2011, 12:48:30 AM (12 years ago)

Author:

livings124

Message:

#4493 improve search filter behavior

Location:

trunk/macosx

Files:

• Controller.m (modified) (3 diffs)
• FileOutlineController.m (modified) (3 diffs)
• FilterBarController.h (modified) (1 diff)
• FilterBarController.m (modified) (2 diffs)
• NSStringAdditions.h (modified) (1 diff)
• NSStringAdditions.m (modified) (1 diff)

trunk/macosx/Controller.m

@@ -2001 +2001 @@
     const BOOL filterGroup = groupFilterValue != GROUP_FILTER_ALL_TAG;
     
-    NSString * searchString = [fFilterBar searchString];
-    if (searchString && [searchString isEqualToString: @""])
-        searchString = nil;
-    const BOOL filterTracker = searchString && [[fDefaults stringForKey: @"FilterSearchType"] isEqualToString: FILTER_TYPE_TRACKER];
+    NSArray * searchStrings = [fFilterBar searchStrings];
+    if (searchStrings && [searchStrings count] == 0)
+        searchStrings = nil;
+    const BOOL filterTracker = searchStrings && [[fDefaults stringForKey: @"FilterSearchType"] isEqualToString: FILTER_TYPE_TRACKER];
     
     NSMutableArray * allTorrents = [NSMutableArray arrayWithCapacity: [fTorrents count]];
@@ -2044 +2044 @@
         
         //check text field
-        if (searchString)
-        {
+        if (searchStrings)
+        {
+            BOOL removeTextField = NO;
             if (filterTracker)
             {
-                BOOL removeTextField = YES;
-                for (NSString * tracker in [torrent allTrackersFlat])
+                NSArray * trackers = [torrent allTrackersFlat];
+                
+                //to count, we need each string in atleast 1 tracker
+                for (NSString * searchString in searchStrings)
                 {
-                    if ([tracker rangeOfString: searchString options:
-                            (NSCaseInsensitiveSearch | NSDiacriticInsensitiveSearch)].location != NSNotFound)
+                    BOOL found = NO;
+                    for (NSString * tracker in trackers)
                     {
-                        removeTextField = NO;
+                        if ([tracker rangeOfString: searchString options:
+                             (NSCaseInsensitiveSearch | NSDiacriticInsensitiveSearch)].location != NSNotFound)
+                        {
+                            found = YES;
+                            break;
+                        }
+                    }
+                    if (!found)
+                    {
+                        removeTextField = YES;
                         break;
                     }
                 }
-                
-                if (removeTextField)
-                    continue;
             }
             else
             {
-                if ([[torrent name] rangeOfString: searchString options:
-                        (NSCaseInsensitiveSearch | NSDiacriticInsensitiveSearch)].location == NSNotFound)
-                    continue;
+                for (NSString * searchString in searchStrings)
+                    if ([[torrent name] rangeOfString: searchString options: (NSCaseInsensitiveSearch | NSDiacriticInsensitiveSearch)].location == NSNotFound)
+                    {
+                        removeTextField = YES;
+                        break;
+                    }
             }
+            
+            if (removeTextField)
+                continue;
         }
         
@@ -2153 +2168 @@
     [self resetInfo]; //if group is already selected, but the torrents in it change
     
-    [self setBottomCountText: groupRows || filterStatus || filterGroup || searchString];
+    [self setBottomCountText: groupRows || filterStatus || filterGroup || searchStrings];
     
     [self setWindowSizeToFit];

trunk/macosx/FileOutlineController.m

@@ -30 +30 @@
 #import "NSApplicationAdditions.h"
 #import "NSMutableArrayAdditions.h"
+#import "NSStringAdditions.h"
 #import <Quartz/Quartz.h>
 
@@ -113 +114 @@
 - (void) setFilterText: (NSString *) text
 {
-    if ([text isEqualToString: @""])
+    NSArray * components = [text betterComponentsSeparatedByCharactersInSet: [NSCharacterSet whitespaceAndNewlineCharacterSet]];
+    if (!components || [components count] == 0)
+    {
         text = nil;
+        components = nil;
+    }
     
     if ((!text && !fFilterText) || (text && fFilterText && [text isEqualToString: fFilterText]))
-    {
         return;
-    }
     
     const BOOL onLion = [NSApp isOnLionOrBetter];
@@ -143 +146 @@
     for (FileListNode * item in tempList)
     {
-        if (!text || [[item name] rangeOfString: text options: (NSCaseInsensitiveSearch | NSDiacriticInsensitiveSearch)].location != NSNotFound)
+        BOOL filter = NO;
+        if (components)
+        {
+            for (NSString * sub in components)
+                if ([[item name] rangeOfString: sub options: (NSCaseInsensitiveSearch | NSDiacriticInsensitiveSearch)].location == NSNotFound)
+                {
+                    filter = YES;
+                    break;
+                }
+        }
+        
+        if (!filter)
         {
             FileListNode * parent = nil;

trunk/macosx/FilterBarController.h

@@ -56 +56 @@
 - (void) setGroupFilter: (id) sender;
 
-- (NSString *) searchString;
+- (NSArray *) searchStrings;
 - (void) focusSearchField;
 

trunk/macosx/FilterBarController.m

@@ -26 +26 @@
 #import "FilterButton.h"
 #import "GroupsController.h"
+#import "NSStringAdditions.h"
 
 #define FILTER_TYPE_TAG_NAME    401
@@ -238 +239 @@
 }
 
-- (NSString *) searchString
-{
-    return [fSearchField stringValue];
+- (NSArray *) searchStrings
+{
+    return [[fSearchField stringValue] betterComponentsSeparatedByCharactersInSet: [NSCharacterSet whitespaceAndNewlineCharacterSet]];
 }
 

trunk/macosx/NSStringAdditions.h

@@ -47 +47 @@
 - (NSComparisonResult) compareNumeric: (NSString *) string; //simple compare method for strings with numbers (works for IP addresses)
 
+- (NSArray *) betterComponentsSeparatedByCharactersInSet: (NSCharacterSet *) separator; //like betterComponentsSeparatedByCharactersInSet:, but excludes blank values
+
 @end

trunk/macosx/NSStringAdditions.m

@@ -183 +183 @@
 }
 
+- (NSArray *) betterComponentsSeparatedByCharactersInSet: (NSCharacterSet *) separator
+{
+    NSMutableArray * components = [NSMutableArray arrayWithArray: [self componentsSeparatedByCharactersInSet: separator]];
+    [components removeObject: @""];
+    return components;
+}
+
 @end