#3250: The visibility logic needs to inspect all child nodes too. Thi…

…s is likely to check the same items over and over again, but as long as this is not a performance problem, this can be left as it is.

libs/wxutil/dataview/ResourceTreeView.cpp

@@ -453,13 +453,39 @@ bool ResourceTreeView::IsTreeModelRowVisible(wxutil::TreeModel::Row& row)
         return true; // done here
     }
 
+    return !IsTreeModelRowFilteredRecursively(row);
+}
+
+bool ResourceTreeView::IsTreeModelRowFilteredRecursively(wxutil::TreeModel::Row& row)
+{
     wxDataViewIconText iconAndName = row[_columns.iconAndName];
 
     auto displayString = iconAndName.GetText().ToStdString();
     string::to_lower(displayString);
-    rMessage() << "displayString: " << displayString << ": " << displayString.find(_filterText) << std::endl;
+    //rMessage() << "displayString: " << displayString << ": " << displayString.find(_filterText) << std::endl;
+
+    if (displayString.find(_filterText) != std::string::npos)
+    {
+        return false; // row itself is visible, no need to check child nodes
+    }
+
+    // This node might also be visible if a single child node is visible, dive into it
+    wxDataViewItemArray children;
+    _treeStore->GetChildren(row.getItem(), children);
+
+    for (const wxDataViewItem& child : children)
+    {
+        wxutil::TreeModel::Row childRow(child, *_treeStore);
+
+        if (!IsTreeModelRowFilteredRecursively(childRow))
+        {
+            return false; // unfiltered node, break the loop
+        }
+    }
 
-    return displayString.find(_filterText) != std::string::npos;
+    // Either we don't have any children, or 
+    // all child nodes are filtered, so this node is filtered too
+    return true;
 }
 
 bool ResourceTreeView::IsTreeModelRowVisibleByViewMode(wxutil::TreeModel::Row& row)

libs/wxutil/dataview/ResourceTreeView.h

@@ -149,6 +149,9 @@ class ResourceTreeView :
     // Returns true if the given row is visible according 
     // to the current view mode (show favourites vs. show all)
     bool IsTreeModelRowVisibleByViewMode(TreeModel::Row& row);
+
+    // Returns true if the given row is filtered (i.e. node and all child nodes are invisible)
+    bool IsTreeModelRowFilteredRecursively(wxutil::TreeModel::Row& row);
 
     void _onContextMenu(wxDataViewEvent& ev);
     void _onTreeStorePopulationProgress(TreeModel::PopulationProgressEvent& ev);