From e6059a49823c8243f00ea6186418cdcc1bd55be0 Mon Sep 17 00:00:00 2001
From: Nicolas Mowen <nickmowen213@gmail.com>
Date: Thu, 19 Sep 2024 09:36:54 -0600
Subject: [PATCH] Use query arg for search and remove unused recording opening

---
 web/src/components/input/InputWithTags.tsx |  36 ++---
 web/src/pages/Explore.tsx                  | 154 +++++----------------
 web/src/types/search.ts                    |   1 +
 web/src/views/search/SearchView.tsx        |   1 -
 4 files changed, 52 insertions(+), 140 deletions(-)

diff --git a/web/src/components/input/InputWithTags.tsx b/web/src/components/input/InputWithTags.tsx
index e4b02fa73..7bf72bfee 100644
--- a/web/src/components/input/InputWithTags.tsx
+++ b/web/src/components/input/InputWithTags.tsx
@@ -584,24 +584,26 @@ export default function InputWithTags({
                 )}
                 {Object.entries(filters).map(([filterType, filterValues]) =>
                   Array.isArray(filterValues) ? (
-                    filterValues.map((value, index) => (
-                      <span
-                        key={`${filterType}-${index}`}
-                        className="inline-flex items-center whitespace-nowrap rounded-full bg-green-100 px-2 py-0.5 text-sm capitalize text-green-800"
-                      >
-                        {filterType.replaceAll("_", " ")}:{" "}
-                        {value.replaceAll("_", " ")}
-                        <button
-                          onClick={() =>
-                            removeFilter(filterType as FilterType, value)
-                          }
-                          className="ml-1 focus:outline-none"
-                          aria-label={`Remove ${filterType}:${value.replaceAll("_", " ")} filter`}
+                    filterValues
+                      .filter(() => filterType !== "query")
+                      .map((value, index) => (
+                        <span
+                          key={`${filterType}-${index}`}
+                          className="inline-flex items-center whitespace-nowrap rounded-full bg-green-100 px-2 py-0.5 text-sm capitalize text-green-800"
                         >
-                          <LuX className="h-3 w-3" />
-                        </button>
-                      </span>
-                    ))
+                          {filterType.replaceAll("_", " ")}:{" "}
+                          {value.replaceAll("_", " ")}
+                          <button
+                            onClick={() =>
+                              removeFilter(filterType as FilterType, value)
+                            }
+                            className="ml-1 focus:outline-none"
+                            aria-label={`Remove ${filterType}:${value.replaceAll("_", " ")} filter`}
+                          >
+                            <LuX className="h-3 w-3" />
+                          </button>
+                        </span>
+                      ))
                   ) : (
                     <span
                       key={filterType}
diff --git a/web/src/pages/Explore.tsx b/web/src/pages/Explore.tsx
index 32c2e1579..1bf6b407d 100644
--- a/web/src/pages/Explore.tsx
+++ b/web/src/pages/Explore.tsx
@@ -1,30 +1,24 @@
 import { useApiFilterArgs } from "@/hooks/use-api-filter";
-import { useCameraPreviews } from "@/hooks/use-camera-previews";
-import { useOverlayState, useSearchEffect } from "@/hooks/use-overlay-state";
-import { FrigateConfig } from "@/types/frigateConfig";
-import { RecordingStartingPoint } from "@/types/record";
+import { useSearchEffect } from "@/hooks/use-overlay-state";
 import { SearchFilter, SearchQuery, SearchResult } from "@/types/search";
-import { TimeRange } from "@/types/timeline";
-import { RecordingView } from "@/views/recording/RecordingView";
 import SearchView from "@/views/search/SearchView";
 import { useCallback, useEffect, useMemo, useState } from "react";
-import useSWR from "swr";
 import useSWRInfinite from "swr/infinite";
 
 const API_LIMIT = 25;
 
 export default function Explore() {
-  const { data: config } = useSWR<FrigateConfig>("config", {
-    revalidateOnFocus: false,
-  });
-
   // search field handler
 
   const [search, setSearch] = useState("");
-  const [searchTerm, setSearchTerm] = useState("");
 
-  const [recording, setRecording] =
-    useOverlayState<RecordingStartingPoint>("recording");
+  const [searchFilter, setSearchFilter, searchSearchParams] =
+    useApiFilterArgs<SearchFilter>();
+
+  const searchTerm = useMemo(
+    () => searchSearchParams?.["query"] || "",
+    [searchSearchParams],
+  );
 
   // search filter
 
@@ -36,9 +30,6 @@ export default function Explore() {
     return searchTerm.split(":")[1];
   }, [searchTerm]);
 
-  const [searchFilter, setSearchFilter, searchSearchParams] =
-    useApiFilterArgs<SearchFilter>();
-
   // search api
 
   useSearchEffect("similarity_search_id", (similarityId) => {
@@ -49,7 +40,16 @@ export default function Explore() {
   });
 
   useEffect(() => {
-    setSearchTerm(search);
+    if (!searchTerm && !search) {
+      return;
+    }
+
+    setSearchFilter({
+      ...searchFilter,
+      query: search.length > 0 ? search : undefined,
+    });
+    // only update when search is updated
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [search]);
 
   const searchQuery: SearchQuery = useMemo(() => {
@@ -168,109 +168,19 @@ export default function Explore() {
     }
   }, [isReachingEnd, isLoadingMore, setSize, size, searchResults, searchQuery]);
 
-  // previews
-
-  const previewTimeRange = useMemo<TimeRange>(() => {
-    if (!searchResults) {
-      return { after: 0, before: 0 };
-    }
-
-    return {
-      after: Math.min(...searchResults.map((res) => res.start_time)),
-      before: Math.max(
-        ...searchResults.map((res) => res.end_time ?? Date.now() / 1000),
-      ),
-    };
-  }, [searchResults]);
-
-  const allPreviews = useCameraPreviews(previewTimeRange, {
-    autoRefresh: false,
-    fetchPreviews: searchResults != undefined,
-  });
-
-  // selection
-
-  const onOpenSearch = useCallback(
-    (item: SearchResult) => {
-      setRecording({
-        camera: item.camera,
-        startTime: item.start_time,
-        severity: "alert",
-      });
-    },
-    [setRecording],
+  return (
+    <SearchView
+      search={search}
+      searchTerm={searchTerm}
+      searchFilter={searchFilter}
+      searchResults={searchResults}
+      isLoading={(isLoadingInitialData || isLoadingMore) ?? true}
+      setSearch={setSearch}
+      setSimilaritySearch={(search) => setSearch(`similarity:${search.id}`)}
+      setSearchFilter={setSearchFilter}
+      onUpdateFilter={setSearchFilter}
+      loadMore={loadMore}
+      hasMore={!isReachingEnd}
+    />
   );
-
-  const selectedReviewData = useMemo(() => {
-    if (!recording) {
-      return undefined;
-    }
-
-    if (!config) {
-      return undefined;
-    }
-
-    if (!searchResults) {
-      return undefined;
-    }
-
-    const allCameras = searchFilter?.cameras ?? Object.keys(config.cameras);
-
-    return {
-      camera: recording.camera,
-      start_time: recording.startTime,
-      allCameras: allCameras,
-    };
-
-    // previews will not update after item is selected
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [recording, searchResults]);
-
-  const selectedTimeRange = useMemo(() => {
-    if (!recording) {
-      return undefined;
-    }
-
-    const time = new Date(recording.startTime * 1000);
-    time.setUTCMinutes(0, 0, 0);
-    const start = time.getTime() / 1000;
-    time.setHours(time.getHours() + 2);
-    const end = time.getTime() / 1000;
-    return {
-      after: start,
-      before: end,
-    };
-  }, [recording]);
-
-  if (recording) {
-    if (selectedReviewData && selectedTimeRange) {
-      return (
-        <RecordingView
-          startCamera={selectedReviewData.camera}
-          startTime={selectedReviewData.start_time}
-          allCameras={selectedReviewData.allCameras}
-          allPreviews={allPreviews}
-          timeRange={selectedTimeRange}
-          updateFilter={setSearchFilter}
-        />
-      );
-    }
-  } else {
-    return (
-      <SearchView
-        search={search}
-        searchTerm={searchTerm}
-        searchFilter={searchFilter}
-        searchResults={searchResults}
-        isLoading={(isLoadingInitialData || isLoadingMore) ?? true}
-        setSearch={setSearch}
-        setSimilaritySearch={(search) => setSearch(`similarity:${search.id}`)}
-        setSearchFilter={setSearchFilter}
-        onUpdateFilter={setSearchFilter}
-        onOpenSearch={onOpenSearch}
-        loadMore={loadMore}
-        hasMore={!isReachingEnd}
-      />
-    );
-  }
 }
diff --git a/web/src/types/search.ts b/web/src/types/search.ts
index 141d3a72e..54cd7c948 100644
--- a/web/src/types/search.ts
+++ b/web/src/types/search.ts
@@ -29,6 +29,7 @@ export type SearchResult = {
 };
 
 export type SearchFilter = {
+  query?: string;
   cameras?: string[];
   labels?: string[];
   subLabels?: string[];
diff --git a/web/src/views/search/SearchView.tsx b/web/src/views/search/SearchView.tsx
index b0ce92bec..bdde82115 100644
--- a/web/src/views/search/SearchView.tsx
+++ b/web/src/views/search/SearchView.tsx
@@ -33,7 +33,6 @@ type SearchViewProps = {
   setSimilaritySearch: (search: SearchResult) => void;
   setSearchFilter: (filter: SearchFilter) => void;
   onUpdateFilter: (filter: SearchFilter) => void;
-  onOpenSearch: (item: SearchResult) => void;
   loadMore: () => void;
   hasMore: boolean;
 };