From 0d232492a2a2feb997b0c2ab38dd3957ccb3821c Mon Sep 17 00:00:00 2001
From: Curtis Conard <cconard96@gmail.com>
Date: Thu, 7 Dec 2023 17:48:05 -0500
Subject: [PATCH] handle corrupt saved searches

---
 src/SavedSearch.php                           |  5 ++++
 .../parts/saved_searches_list.html.twig       | 24 ++++++++++++-------
 2 files changed, 21 insertions(+), 8 deletions(-)

diff --git a/src/SavedSearch.php b/src/SavedSearch.php
index 1f947a3fa212..9b5523168d1b 100644
--- a/src/SavedSearch.php
+++ b/src/SavedSearch.php
@@ -868,6 +868,11 @@ public function getMine(string $itemtype = null, bool $inverse = false, bool $en
                 } catch (\RuntimeException $e) {
                     ErrorHandler::getInstance()->handleException($e);
                     $search_data = false;
+                } catch (\Throwable $e) {
+                    ErrorHandler::getInstance()->handleException($e);
+                    $info_message = __s('A fatal error occurred while executing this saved search. It is not able to be used.');
+                    $count = "<span class='ti ti-alert-triangle-filled' title='$info_message'></span>";
+                    $data['_error'] = true;
                 }
                 if (isset($search_data['data']['totalcount'])) {
                     $count = $search_data['data']['totalcount'];
diff --git a/templates/layout/parts/saved_searches_list.html.twig b/templates/layout/parts/saved_searches_list.html.twig
index cc4bd957f9dd..a6f43f61d546 100644
--- a/templates/layout/parts/saved_searches_list.html.twig
+++ b/templates/layout/parts/saved_searches_list.html.twig
@@ -41,15 +41,23 @@
 {% for search in saved_searches %}
    <div class="savedsearches-item grip-savedsearch list-group-item search-line d-flex align-items-center pe-1 {{ active == search['id'] ? 'active' : '' }}"
          data-id="{{ search['id'] }}">
-      <a href="{{ 'SavedSearch'|itemtype_search_path }}?action=load&amp;id={{ search['id'] }}"
-         class="d-block saved-searches-link text-truncate">
-         {{ search['name']|verbatim_value }}
-      </a>
+      {% if search['_error'] is not defined or search['_error'] == false %}
+          <a href="{{ 'SavedSearch'|itemtype_search_path }}?action=load&amp;id={{ search['id'] }}"
+             class="d-block saved-searches-link text-truncate">
+             {{ search['name']|verbatim_value }}
+          </a>
+      {% else %}
+          <span class="d-block text-truncate">
+              {{ search['name']|verbatim_value }}
+          </span>
+      {% endif %}
       <div class="{{ search['is_default'] > 0 ? '' : 'list-group-item-actions' }} ms-auto default-ctrl">
-         <i class="{{ search['is_default'] > 0 ? 'fas' : 'far' }} fa-star fa-xs mark-default me-1"
-            title="{{ search['is_default'] > 0 ? __('Default search') : __('mark as default') }}"
-            data-bs-toggle="tooltip" data-bs-placement="right"
-            role="button"></i>
+          {% if search['_error'] is not defined or search['_error'] == false %}
+             <i class="{{ search['is_default'] > 0 ? 'fas' : 'far' }} fa-star fa-xs mark-default me-1"
+                title="{{ search['is_default'] > 0 ? __('Default search') : __('mark as default') }}"
+                data-bs-toggle="tooltip" data-bs-placement="right"
+                role="button"></i>
+          {% endif %}
       </div>
       <div class="d-flex flex-nowrap align-items-center">
          {% if search['is_private'] == 1 %}