From ba2e5eca45c95f64dbab7e3970011492d597efae Mon Sep 17 00:00:00 2001
From: Carlos Felix <63804190+carlosfelix2@users.noreply.github.com>
Date: Tue, 10 Dec 2024 12:09:39 -0500
Subject: [PATCH] Search for the same tract ID as already selected does not
 unselect it

---
 client/src/components/J40Map.tsx              |  3 ++-
 client/src/components/MapSearch/MapSearch.tsx | 18 ++++++++++++------
 2 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/client/src/components/J40Map.tsx b/client/src/components/J40Map.tsx
index 66107590..7d8919a5 100644
--- a/client/src/components/J40Map.tsx
+++ b/client/src/components/J40Map.tsx
@@ -393,7 +393,8 @@ const J40Map = ({location}: IJ40Interface) => {
 
           {/* This is the first overlayed row on the map: Search and Geolocation */}
           <div className={styles.mapHeaderRow}>
-            <MapSearch goToPlace={goToPlace} mapRef={mapRef} selectFeatureOnMap={selectFeatureOnMap}/>
+            <MapSearch goToPlace={goToPlace} mapRef={mapRef} selectFeatureOnMap={selectFeatureOnMap}
+              selectedFeatureId={selectedFeatureId}/>
 
             {/* Geolocate Icon */}
             <div className={styles.geolocateBox}>
diff --git a/client/src/components/MapSearch/MapSearch.tsx b/client/src/components/MapSearch/MapSearch.tsx
index 4966b9c0..13e600f7 100644
--- a/client/src/components/MapSearch/MapSearch.tsx
+++ b/client/src/components/MapSearch/MapSearch.tsx
@@ -18,6 +18,7 @@ interface IMapSearch {
   goToPlace(bounds: LngLatBoundsLike):void;
   mapRef:RefObject<MapRef>;
   selectFeatureOnMap: (feature: any) => void;
+  selectedFeatureId: string;
 }
 
 interface ISearchTractRecord {
@@ -26,7 +27,7 @@ interface ISearchTractRecord {
   INTPTLON10: string;
 }
 
-const MapSearch = ({goToPlace, mapRef, selectFeatureOnMap}:IMapSearch) => {
+const MapSearch = ({goToPlace, mapRef, selectFeatureOnMap, selectedFeatureId}:IMapSearch) => {
   // State to hold if the search results are empty or not:
   const [isSearchResultsNull, setIsSearchResultsNull] = useState(false);
   const intl = useIntl();
@@ -82,6 +83,10 @@ const MapSearch = ({goToPlace, mapRef, selectFeatureOnMap}:IMapSearch) => {
    * @param {string} tract the 11 digit tract ID as a string
    */
   const searchForTract = async (tract: string) => {
+    // We create a bounding box just to get the tract in the view box.
+    // The size is not important.
+    const BOUNDING_BOX_SIZE_DD = 0.1;
+
     /**
      * Wait for the map to be done loading and moving.
      * @param {function()} callback the callback to run after the map is ready
@@ -97,14 +102,15 @@ const MapSearch = ({goToPlace, mapRef, selectFeatureOnMap}:IMapSearch) => {
       }
     };
 
+    // Convert 10 digit tracts to 11.
+    const searchTerm = tract.length == 10 ? '0' + tract : tract;
+
+    // If the search is for the same tract then do nothing.
+    if (selectedFeatureId == searchTerm) return;
+
     setIsSearchResultsNull(true);
 
-    // We create a bounding box just to get the tract in the view box.
-    // The size is not important.
-    const BOUNDING_BOX_SIZE_DD = 0.1;
     if (tractSearch) {
-      // Convert 10 digit tracts to 11.
-      const searchTerm = tract.length == 10 ? '0' + tract : tract;
       const result = tractSearch.search(searchTerm);
       if (result.length > 0) {
         const searchTractRecord = result[0] as ISearchTractRecord;