From 3e512c44592af0931984354e7b06b0c0ecc236d9 Mon Sep 17 00:00:00 2001
From: Vincent Fretin <vincent.fretin@gmail.com>
Date: Fri, 30 Aug 2024 09:46:22 +0200
Subject: [PATCH] simplify condition

---
 .../AddLayerPanel/createLayerFunctions.js     | 33 ++++++-------------
 .../modals/GeoModal/GeoModal.component.jsx    | 31 +++++++----------
 2 files changed, 22 insertions(+), 42 deletions(-)

diff --git a/src/editor/components/components/AddLayerPanel/createLayerFunctions.js b/src/editor/components/components/AddLayerPanel/createLayerFunctions.js
index 770085805..2c04f3ee3 100644
--- a/src/editor/components/components/AddLayerPanel/createLayerFunctions.js
+++ b/src/editor/components/components/AddLayerPanel/createLayerFunctions.js
@@ -140,29 +140,16 @@ export function create3DTiles() {
       ellipsoidalHeight = parseFloat(streetGeo['ellipsoidalHeight']) || 0;
     }
 
-    if (streetGeo) {
-      AFRAME.INSPECTOR.execute('entityupdate', {
-        entity: geoLayer,
-        component: 'street-geo',
-        value: {
-          latitude: latitude,
-          longitude: longitude,
-          ellipsoidalHeight: ellipsoidalHeight,
-          maps: 'google3d'
-        }
-      });
-    } else {
-      AFRAME.INSPECTOR.execute('componentadd', {
-        entity: geoLayer,
-        component: 'street-geo',
-        value: {
-          latitude: latitude,
-          longitude: longitude,
-          ellipsoidalHeight: ellipsoidalHeight,
-          maps: 'google3d'
-        }
-      });
-    }
+    AFRAME.INSPECTOR.execute(streetGeo ? 'entityupdate' : 'componentadd', {
+      entity: geoLayer,
+      component: 'street-geo',
+      value: {
+        latitude: latitude,
+        longitude: longitude,
+        ellipsoidalHeight: ellipsoidalHeight,
+        maps: 'google3d'
+      }
+    });
   };
 
   if (AFRAME.components['loader-3dtiles']) {
diff --git a/src/editor/components/modals/GeoModal/GeoModal.component.jsx b/src/editor/components/modals/GeoModal/GeoModal.component.jsx
index fc07a53f7..03e7177f0 100644
--- a/src/editor/components/modals/GeoModal/GeoModal.component.jsx
+++ b/src/editor/components/modals/GeoModal/GeoModal.component.jsx
@@ -136,27 +136,20 @@ const GeoModal = ({ isOpen, onClose }) => {
       console.log(`elevation: ${data.ellipsoidalHeight}`);
 
       const geoLayer = document.getElementById('reference-layers');
-      const value = {
-        latitude: latitude,
-        longitude: longitude,
-        ellipsoidalHeight: data.ellipsoidalHeight,
-        orthometricHeight: data.orthometricHeight,
-        geoidHeight: data.geoidHeight
-      };
-
-      if (geoLayer.hasAttribute('street-geo')) {
-        AFRAME.INSPECTOR.execute('entityupdate', {
-          entity: geoLayer,
-          component: 'street-geo',
-          value
-        });
-      } else {
-        AFRAME.INSPECTOR.execute('componentadd', {
+      AFRAME.INSPECTOR.execute(
+        geoLayer.hasAttribute('street-geo') ? 'entityupdate' : 'componentadd',
+        {
           entity: geoLayer,
           component: 'street-geo',
-          value
-        });
-      }
+          value: {
+            latitude: latitude,
+            longitude: longitude,
+            ellipsoidalHeight: data.ellipsoidalHeight,
+            orthometricHeight: data.orthometricHeight,
+            geoidHeight: data.geoidHeight
+          }
+        }
+      );
     }
 
     setIsWorking(false);