Merge lp:~stolowski/unity-lens-files/hidden-files-5.0 into lp:unity-lens-files/5.0

[Paweł Stołowski]

Check paths for "/." substring to exclude hidden files. Added manual test.
Cherry-picked trunk rev 240..241.

Check paths for "/." substring to exclude hidden files. Added manual test.
Cherry-picked trunk rev 240..241.

[Michal Hruby (mhr3)]

review: Approve

=== modified file 'src/daemon.vala'
--- src/daemon.vala	2012-04-05 16:11:45 +0000
+++ src/daemon.vala	2012-11-08 14:18:26 +0000
@@ -24,9 +24,9 @@
 using Gee;
 
 namespace Unity.FilesLens {
-  
+
   const string ICON_PATH = Config.DATADIR + "/icons/unity-icon-theme/places/svg/";
-  
+
   public class Daemon : GLib.Object
   {
     private Zeitgeist.Log log;
@@ -72,11 +72,11 @@
       populate_categories ();
       populate_filters ();
       lens.add_local_scope (scope);
-      
+
       /* Bring up Zeitgeist interfaces */
       log = new Zeitgeist.Log ();
       index = new Zeitgeist.Index ();
-      
+
       /* Listen for all file:// related events from Zeitgeist */
       var templates = new PtrArray ();
       var event = new Zeitgeist.Event ();
@@ -194,7 +194,7 @@
       var cat = new Unity.Category (_("Recent"),
                                     new FileIcon (icon_dir.get_child ("group-recent.svg")));
       categories.append (cat);
-      
+
       cat = new Unity.Category (_("Recent Files"),
                                 new FileIcon (icon_dir.get_child ("group-recent.svg")));
       categories.append (cat);
@@ -224,7 +224,7 @@
                              new Subject.full (uri_prefix,
                                                "", "", "", "", "", ""));
       templates["all"] = event;
-      
+
       /* Section.DOCUMENTS */
       event = new Event.full("", manifestation, "",
                              new Subject.full (uri_prefix,
@@ -247,7 +247,7 @@
                              new Subject.full (uri_prefix,
                                                NFO_IMAGE, "", "", "", "", ""));
       templates["images"] = event;
-      
+
       /* Section.AUDIO */
       event = new Event.full("", manifestation, "",
                              new Subject.full (uri_prefix,
@@ -269,7 +269,7 @@
                                                NFO_PRESENTATION, "", "", "", "", ""));
       templates["presentations"] = event;
 
-      /* Section.OTHER 
+      /* Section.OTHER
        * Note that subject templates are joined with logical AND */
       event = new Event.full("", manifestation, "");
       event.add_subject (new Subject.full (uri_prefix,
@@ -367,7 +367,7 @@
     private bool is_search_empty (LensSearch search)
     {
       if (search.search_string == null) return true;
-      
+
       return search.search_string.strip () == "";
     }
 
@@ -390,7 +390,7 @@
 
       return s;
     }
-    
+
     private async void update_global_search_async (LensSearch search,
                                                    Cancellable cancellable)
     {
@@ -413,7 +413,7 @@
 
         results_model.clear ();
 
-        /* check if the thing typed isn't a url (like facebook.com) 
+        /* check if the thing typed isn't a url (like facebook.com)
          * or isn't a remote mountable url (like ftp://ftp.ubuntu.com)*/
         var url_type = UrlType.UNKNOWN;
         var checked_url = urls.check_url (search.search_string, out url_type);
@@ -483,7 +483,7 @@
                  search.search_string, e.message);
       }
     }
-    
+
     private async void update_search_async (LensSearch search,
                                             Cancellable cancellable)
     {
@@ -514,13 +514,13 @@
 
         txn.clear ();
 
-        /* check if the thing typed isn't a url (like facebook.com) 
+        /* check if the thing typed isn't a url (like facebook.com)
          * or isn't a remote mountable url (like ftp://ftp.ubuntu.com)*/
-        var url_type = UrlType.UNKNOWN; 
+        var url_type = UrlType.UNKNOWN;
         var checked_url = urls.check_url (search.search_string, out url_type);
         if (checked_url != null)
         {
-          txn.append (checked_url, urls.get_icon_for_type(url_type), 
+          txn.append (checked_url, urls.get_icon_for_type(url_type),
                 Categories.RECENT,
                 "text/html", search.search_string,
                 checked_url, checked_url);
@@ -677,7 +677,7 @@
         return;
       }
 
-      int64[] sizes = 
+      int64[] sizes =
       {
         0,
         1024,
@@ -863,9 +863,9 @@
     {
       debug (@"Activating: $uri");
       try {
-        if (bookmarks.launch_if_bookmark (uri)) 
+        if (bookmarks.launch_if_bookmark (uri))
           return new Unity.ActivationResponse(Unity.HandledType.HIDE_DASH);
-          
+
         /* this code ensures that a file manager will be used
          * * if uri it's a remote location that should be mounted */
         var url_type = UrlType.UNKNOWN;
@@ -940,6 +940,10 @@
           if (excluded_uris != null && origin in excluded_uris) continue;
           if (!(origin in origins) && f.is_native () && f.query_exists ())
           {
+            string? path = f.get_path ();
+            if (path != null && path.contains ("/."))
+              continue;
+
             var display_name = Path.get_basename (f.get_parse_name ());
             if (display_name.casefold ().has_prefix (folded_search))
             {
@@ -1012,7 +1016,8 @@
           {
             FileInfo info = file.query_info (check_size ?
               ATTRS_TYPE_SIZE_HIDDEN : ATTRS_TYPE_HIDDEN, 0, null);
-            if (info.get_is_hidden())
+            string? path = file.get_path ();
+            if (path != null && path.contains ("/."))
               continue;
             if (check_size &&
               (info.get_size () < min_size || info.get_size () > max_size))
@@ -1062,4 +1067,3 @@
     }
   }
 } /* namespace */
-

=== added file 'tests/manual/misc.txt'
--- tests/manual/misc.txt	1970-01-01 00:00:00 +0000
+++ tests/manual/misc.txt	2012-11-08 14:18:26 +0000
@@ -0,0 +1,14 @@
+Hidden files / dirs are not displayed
+-------------------------------------
+Test that hidden files, hidden folders, or normal files in hidden folders are not displayed in search results.
+
+Setup:
+Create a few combinations of hidden files/folders using nautilus (CTRL+H in nautilus to view hidden files), e.g. ~/.Foo/bar.txt, ~/.foo.txt, ~/.Foo/bar/baz.txt.
+Update locatedb (e.g. sudo /etc/cron.daily/mlocate).
+
+Actions:
+Type search queries such as "Foo", "bar", "baz" in the dash search query.
+
+Expected Result:
+Verify none of the newly created hidden files are displayed in the Dash home view with empty search.
+Verify none of the hidden files are displayed after entering a search query.