From 03c445bfaf82902e7d5b519021d5970bbecbbdd3 Mon Sep 17 00:00:00 2001
From: Jeremy Lenz <jlenz@redhat.com>
Date: Thu, 29 Jan 2026 08:09:01 -0500
Subject: [PATCH] Fixes #39018 - Fix infinite loop on job invocations page

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
---
 .../components/TargetingHosts/index.js        | 23 +++++++++++++++----
 1 file changed, 19 insertions(+), 4 deletions(-)

diff --git a/webpack/react_app/components/TargetingHosts/index.js b/webpack/react_app/components/TargetingHosts/index.js
index 0c6fd64ea..76f1ec728 100644
--- a/webpack/react_app/components/TargetingHosts/index.js
+++ b/webpack/react_app/components/TargetingHosts/index.js
@@ -56,8 +56,9 @@ const WrappedTargetingHosts = () => {
     }
   }, [dispatch, intervalExists]);
 
-  // Use ref to avoid infinite loop from handleSearch depending on pagination.per_page
+  // Use refs to avoid infinite loop from handleSearch depending on pagination.per_page and searchQuery
   const perPageRef = useRef(pagination.per_page);
+  const searchQueryRef = useRef(searchQuery);
 
   const handleSearch = useCallback(
     (query, status) => {
@@ -71,11 +72,15 @@ const WrappedTargetingHosts = () => {
     [stopApiInterval]
   );
 
-  // Keep ref in sync with pagination
+  // Keep refs in sync with state
   useEffect(() => {
     perPageRef.current = pagination.per_page;
   }, [pagination.per_page]);
 
+  useEffect(() => {
+    searchQueryRef.current = searchQuery;
+  }, [searchQuery]);
+
   const handlePagination = useCallback(
     args => {
       stopApiInterval();
@@ -112,9 +117,19 @@ const WrappedTargetingHosts = () => {
     };
   }, [dispatch, apiUrl, autoRefresh, getData]);
 
+  // Only respond to external statusFilter changes (from chart clicks)
+  // searchQuery changes are handled directly by handleSearch calls from the UI
   useEffect(() => {
-    handleSearch(searchQuery, statusFilter);
-  }, [statusFilter, searchQuery, handleSearch]);
+    dispatch(stopInterval(TARGETING_HOSTS));
+    const defaultPagination = { page: 1, per_page: perPageRef.current };
+    setApiUrl(
+      getApiUrl(
+        buildSearchQuery(searchQueryRef.current, statusFilter),
+        defaultPagination
+      )
+    );
+    setPagination(defaultPagination);
+  }, [statusFilter, dispatch]);
 
   return (
     <TargetingHostsPage