PB-1480: always set shouldCenter to true when the string is a valid coordinated.

Author: ismailsunni

packages/mapviewer/src/router/storeSync/SearchParamConfig.class.js

@@ -1,6 +1,7 @@
 import AbstractParamConfig, {
     STORE_DISPATCHER_ROUTER_PLUGIN,
 } from '@/router/storeSync/abstractParamConfig.class'
+import coordinateFromString from '@/utils/coordinates/coordinateExtractors'
 import { removeQueryParamFromHref } from '@/utils/searchParamUtils'
 
 export const URL_PARAM_NAME_SWISSSEARCH = 'swisssearch'
@@ -15,9 +16,15 @@ export const URL_PARAM_NAME_SWISSSEARCH = 'swisssearch'
 function dispatchSearchFromUrl(to, store, urlParamValue) {
     // avoiding dispatching the search query to the store when there is nothing to set. Not avoiding this makes the CI test very flaky
     if (urlParamValue) {
+        let shouldCenter = !(to.query.crosshair && to.query.center)
+        // When the query is a valid coordinate, we want to center the map
+        const extractedCoordinate = coordinateFromString(to.query[URL_PARAM_NAME_SWISSSEARCH])
+        if (extractedCoordinate) {
+            shouldCenter = true
+        }
         store.dispatch('setSearchQuery', {
             query: urlParamValue,
-            shouldCenter: !(to.query.crosshair && to.query.center),
+            shouldCenter: shouldCenter,
             dispatcher: STORE_DISPATCHER_ROUTER_PLUGIN,
             originUrlParam: true,
         })

packages/mapviewer/src/store/modules/search.store.js

@@ -98,7 +98,7 @@ const actions = {
             // there are situations where we don't want to center on the features or coordinates searched.
             // for example: when we are sharing a position with a search query. In those situation, the
             // 'zoom to extent' should be avoided. We center by default.
-            if (extractedCoordinate && shouldCenter) {
+            if (extractedCoordinate  && shouldCenter) {
                 let coordinates = [...extractedCoordinate.coordinate]
                 if (extractedCoordinate.coordinateSystem !== currentProjection) {
                     // special case for LV03 input, we can't use proj4 to transform them into