From 842b59b307dbf77c549b5ae2e51d7f0318350e44 Mon Sep 17 00:00:00 2001
From: Suren <dsuren1@gmail.com>
Date: Fri, 30 Aug 2024 04:25:40 -0700
Subject: [PATCH] #10521 - Update center comparison in OL map component
 (#10525)

---
 web/client/components/map/openlayers/Map.jsx  | 16 +++++++--
 .../map/openlayers/__tests__/Map-test.jsx     | 36 +++++++++++++++++++
 2 files changed, 50 insertions(+), 2 deletions(-)

diff --git a/web/client/components/map/openlayers/Map.jsx b/web/client/components/map/openlayers/Map.jsx
index c1a2fcb233..b8208e028b 100644
--- a/web/client/components/map/openlayers/Map.jsx
+++ b/web/client/components/map/openlayers/Map.jsx
@@ -534,11 +534,23 @@ class OpenlayersMap extends React.Component {
         return new View(viewOptions);
     };
 
+    isNearlyEqual = (a, b) => {
+        /**
+         * this implementation will update the map only if the movement
+         * between 8 decimals (coordinate precision in mm) in the reference system
+         * to avoid rounded value changes due to float mathematic operations or transformed value
+        */
+        if (a === undefined || b === undefined) {
+            return false;
+        }
+        return a.toFixed(8) - b.toFixed(8) <= 0.00000001;
+    };
+
     _updateMapPositionFromNewProps = (newProps) => {
         var view = this.map.getView();
         const currentCenter = this.props.center;
-        const centerIsUpdated = newProps.center.y === currentCenter.y &&
-            newProps.center.x === currentCenter.x;
+        const centerIsUpdated = this.isNearlyEqual(newProps.center.y, currentCenter.y) &&
+            this.isNearlyEqual(newProps.center.x, currentCenter.x);
 
         if (!centerIsUpdated) {
             // let center = ol.proj.transform([newProps.center.x, newProps.center.y], 'EPSG:4326', newProps.projection);
diff --git a/web/client/components/map/openlayers/__tests__/Map-test.jsx b/web/client/components/map/openlayers/__tests__/Map-test.jsx
index b354395bea..f070e7cd2d 100644
--- a/web/client/components/map/openlayers/__tests__/Map-test.jsx
+++ b/web/client/components/map/openlayers/__tests__/Map-test.jsx
@@ -1489,4 +1489,40 @@ describe('OpenlayersMap', () => {
             expect(customHooRegister.getHook(MapUtils.ZOOM_TO_EXTENT_HOOK)).toBeTruthy();
         });
     });
+    it('update map position center based on nearly equal match when receiving new props', () => {
+        let map = ReactDOM.render(
+            <OpenlayersMap
+                center={{y: 43.9323233378, x: 10.3346776780}}
+                zoom={1}
+                projection="EPSG:4326"
+                measurement={{}}
+            />
+            , document.getElementById("map"));
+        map = ReactDOM.render(
+            <OpenlayersMap
+                center={{y: 43.9323233388, x: 10.3346776790}}
+                zoom={1}
+                measurement={{}}
+                projection="EPSG:4326"
+                mapOptions={undefined}
+            />
+            , document.getElementById("map")
+        );
+        expect(map).toBeTruthy();
+        // center is not modified
+        expect(map.map.getView().getCenter()).toEqual([10.3346776780, 43.9323233378]);
+
+        map = ReactDOM.render(
+            <OpenlayersMap
+                center={{y: 43.9323234388, x: 10.3346773790}}
+                zoom={1}
+                measurement={{}}
+                projection="EPSG:4326"
+                mapOptions={undefined}
+            />
+            , document.getElementById("map")
+        );
+        // center is modified
+        expect(map.map.getView().getCenter()).toEqual([10.3346773790, 43.9323234388]);
+    });
 });