From a3526bea74f6076887f30ea978c5f9e6b2d57260 Mon Sep 17 00:00:00 2001
From: Sergey Svistunov <zenden2k@gmail.com>
Date: Mon, 27 Jan 2025 16:21:49 +0300
Subject: [PATCH] Fixed filters in Network Debugger

---
 Source/Gui/Controls/NetworkDebugListView.cpp | 10 ++++-
 Source/Gui/Models/NetworkDebugModel.cpp      | 43 +++++++++++++-------
 2 files changed, 38 insertions(+), 15 deletions(-)

diff --git a/Source/Gui/Controls/NetworkDebugListView.cpp b/Source/Gui/Controls/NetworkDebugListView.cpp
index e062c32a..73d8d4b6 100644
--- a/Source/Gui/Controls/NetworkDebugListView.cpp
+++ b/Source/Gui/Controls/NetworkDebugListView.cpp
@@ -23,7 +23,7 @@ void CNetworkDebugListView::Init() {
         { NetworkDebugModelNS::COLUMN_N, TR("N"), HDFT_ISSTRING, 25 },
         { NetworkDebugModelNS::COLUMN_THREAD_ID, TR("Thread"), HDFT_ISSTRING  /* HDFT_ISNUMBER*/, 55 },
         { NetworkDebugModelNS::COLUMN_TIME, TR("Time"), HDFT_ISSTRING, 160 },
-        { NetworkDebugModelNS::COLUMN_TYPE, TR("Type"), HDFT_ISSTRING, 130},
+        { NetworkDebugModelNS::COLUMN_TYPE, TR("Type"), HDFT_ISSTRING, 130 },
         { NetworkDebugModelNS::COLUMN_TEXT, TR("Text"), HDFT_ISSTRING, 320 }
     };
 
@@ -116,6 +116,8 @@ void CNetworkDebugListView::onItemCountChanged(size_t index) {
 void CNetworkDebugListView::applyFilters() {
     std::vector<std::string> filters(NetworkDebugModelNS::COLUMN_COUNT);
 
+    bool allEmpty = true;
+
     for (const auto& column: columns_) {
         HDITEM item {};
         item.mask = HDI_FILTER;
@@ -139,6 +141,12 @@ void CNetworkDebugListView::applyFilters() {
                 filters[column.index] = std::to_string(val);
             }
         }
+        if (!filters[column.index].empty()) {
+            allEmpty = false;
+        }
+    }
+    if (allEmpty) {
+        filters.clear();
     }
     model_->applyFilter(filters);
 }
diff --git a/Source/Gui/Models/NetworkDebugModel.cpp b/Source/Gui/Models/NetworkDebugModel.cpp
index 958244c4..7080610e 100644
--- a/Source/Gui/Models/NetworkDebugModel.cpp
+++ b/Source/Gui/Models/NetworkDebugModel.cpp
@@ -16,8 +16,7 @@
 
 namespace {
 
-std::string CurlTypeToString(curl_infotype type)
-{
+std::string CurlTypeToString(curl_infotype type) {
     switch (type) {
     case CURLINFO_TEXT:
         return "== Info: ";
@@ -53,26 +52,42 @@ size_t StringSearch(const std::string& str1, const std::string& str2) {
     if (it != str1.end()) {
         return it - str1.begin();
     } else {
-        return -1; // not found
+        return std::string::npos; // not found
     }
 }
 
 }
 
 bool NetworkDebugModelData::acceptFilter(const std::vector<std::string>& fields) const {
-    bool result = true;
-
     if (fields.size() < NetworkDebugModelNS::COLUMN_COUNT) {
         return true;
     }
-    if (!fields[NetworkDebugModelNS::COLUMN_THREAD_ID].empty()) {
-        result = result && (threadId_ == fields[NetworkDebugModelNS::COLUMN_THREAD_ID]);
+    if (!fields[NetworkDebugModelNS::COLUMN_THREAD_ID].empty() && threadId_ != fields[NetworkDebugModelNS::COLUMN_THREAD_ID]) {
+        return false;
     }
     if (!fields[NetworkDebugModelNS::COLUMN_TYPE].empty()) {
         std::string val = CurlTypeToString(type);
-        result = result && (StringSearch(val, fields[NetworkDebugModelNS::COLUMN_TYPE]) != std::string::npos);
+        if (StringSearch(val, fields[NetworkDebugModelNS::COLUMN_TYPE]) == std::string::npos) {
+            return false;
+        }
+    }
+    if (!fields[NetworkDebugModelNS::COLUMN_TIME].empty()) {
+        std::stringstream ss;
+        ss << std::put_time(std::localtime(&time_), "%F %T");
+        if (StringSearch(ss.str(), fields[NetworkDebugModelNS::COLUMN_TIME]) == std::string::npos) {
+            return false;
+        }
+    }
+    if (!fields[NetworkDebugModelNS::COLUMN_TEXT].empty()) {
+        if (type == CURLINFO_HEADER_IN || type == CURLINFO_HEADER_OUT || type == CURLINFO_TEXT) {
+            if (StringSearch(data.substr(0, data.find_first_of("\r\n")), fields[NetworkDebugModelNS::COLUMN_TEXT]) == std::string::npos) {
+                return false;
+            }
+        } else {
+            return false;
+        }
     }
-    return result;
+    return true;
 }
 
 std::string NetworkDebugModelData::getDecoded() {
@@ -258,17 +273,17 @@ void NetworkDebugModel::saveToFile(const std::string& fileName) {
 }
 
 std::string NetworkDebugModel::getCell(const NetworkDebugModelData& modelData, int row, int column) const {
-    if (column == 0) {
+    if (column == NetworkDebugModelNS::COLUMN_N) {
         // return std::to_string(row + 1);
-    } else if (column == 1) {
+    } else if (column == NetworkDebugModelNS::COLUMN_THREAD_ID) {
         return modelData.threadId_;
-    } else if (column == 2) {
+    } else if (column == NetworkDebugModelNS::COLUMN_TIME) {
         std::stringstream ss;
         ss << std::put_time(std::localtime(&modelData.time_), "%F %T");
         return ss.str();
-    } else if (column == 3) {
+    } else if (column == NetworkDebugModelNS::COLUMN_TYPE) {
         return CurlTypeToString(modelData.type);
-    } else if (column == 4) {
+    } else if (column == NetworkDebugModelNS::COLUMN_TEXT) {
         if (modelData.type == CURLINFO_HEADER_IN || modelData.type == CURLINFO_HEADER_OUT || modelData.type == CURLINFO_TEXT) {
             return modelData.data.substr(0, modelData.data.find_first_of("\r\n"));
         }