Merge pull request #170 from landryb/fix/168

only call getBatiments if there are buildings on the plot (#168)
georchestra · Aug 22, 2022 · f45ae66 · f45ae66
2 parents 3cab1ba + 056ea39
commit f45ae66
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/js/extension/components/information/Buildings.jsx b/js/extension/components/information/Buildings.jsx
@@ -31,12 +31,14 @@ export default function Buildings({
     const [loading, setLoading] = useState(false);
 
     useEffect(() => {
-        setLoading(true);
-        getBatiments({ parcelle, dnubat: letter }).then((data) => {
-            setBuildings(data);
-            setSelected([]);
-            setLoading(false);
-        });
+        if (letter) {
+            setLoading(true);
+            getBatiments({ parcelle, dnubat: letter }).then((data) => {
+                setBuildings(data);
+                setSelected([]);
+                setLoading(false);
+            });
+        }
     }, [letter]);
 
     // Description