[tracker/wip/carlosg/libtracker-miner-cleanups: 16/36] libtracker-miner: Fix content filter with default policy=deny

[Carlos Garnacho <carlosg src gnome org>]

commit 8121b941256ade91ae653a169a6780497b62afa5
Author: Carlos Garnacho <carlosg gnome org>
Date:   Wed Oct 18 22:45:10 2017 +0200

    libtracker-miner: Fix content filter with default policy=deny
    
    We gave the directory as bad whenever any file didn't match the
    filter, but files that don't match the filter should not trigger
    it in one way or another.

 src/libtracker-miner/tracker-indexing-tree.c |   24 ++++++++++++++++--------
 1 files changed, 16 insertions(+), 8 deletions(-)
---
diff --git a/src/libtracker-miner/tracker-indexing-tree.c b/src/libtracker-miner/tracker-indexing-tree.c
index 42f6955..59f4399 100644
--- a/src/libtracker-miner/tracker-indexing-tree.c
+++ b/src/libtracker-miner/tracker-indexing-tree.c
@@ -917,23 +917,31 @@ tracker_indexing_tree_parent_is_indexable (TrackerIndexingTree *tree,
                                            GFile               *parent,
                                            GList               *children)
 {
+       TrackerIndexingTreePrivate *priv;
+       gboolean has_match = FALSE;
+
+       g_return_val_if_fail (TRACKER_IS_INDEXING_TREE (tree), FALSE);
+       g_return_val_if_fail (G_IS_FILE (parent), FALSE);
+
+       priv = tree->priv;
+
        if (!tracker_indexing_tree_file_is_indexable (tree,
                                                      parent,
                                                      G_FILE_TYPE_DIRECTORY)) {
                return FALSE;
        }
 
-       while (children) {
-               if (indexing_tree_file_is_filtered (tree,
-                                                   TRACKER_FILTER_PARENT_DIRECTORY,
-                                                   children->data)) {
-                       return FALSE;
-               }
-
+       while (children && !has_match) {
+               has_match = tracker_indexing_tree_file_matches_filter (tree,
+                                                                      TRACKER_FILTER_PARENT_DIRECTORY,
+                                                                      children->data);
                children = children->next;
        }
 
-       return TRUE;
+       if (priv->policies[TRACKER_FILTER_PARENT_DIRECTORY] == TRACKER_FILTER_POLICY_ACCEPT)
+               return !has_match;
+       else
+               return has_match;
 }
 
 /**