[Map] Display warning when trying to define a custom icon for a Marker that already has an Icon

Author: ker0x

src/Map/src/Bridge/Google/CHANGELOG.md

@@ -3,6 +3,7 @@
 ## 2.30
 
 -  Ensure compatibility with PHP 8.5
+-  Display a warning when trying to define `bridgeOptions.content` for a `Marker` that already has an `Icon`
 
 ## 2.29.0
 

src/Map/src/Bridge/Google/assets/dist/map_controller.js

@@ -248,6 +248,11 @@ var map_controller_default = class extends abstract_map_controller_default {
       this.createInfoWindow({ definition: infoWindow, element: marker });
     }
     if (icon) {
+      if (Object.prototype.hasOwnProperty.call(bridgeOptions, "content")) {
+        console.warn('[Symfony UX Map] Defining "bridgeOptions.content" for a marker with a custom icon is not supported and will be ignored.');
+      } else if (Object.prototype.hasOwnProperty.call(rawOptions, "content")) {
+        console.warn('[Symfony UX Map] Defining "rawOptions.content" for a marker with a custom icon is not supported and will be ignored.');
+      }
       this.doCreateIcon({ definition: icon, element: marker });
     }
     return marker;

src/Map/src/Bridge/Google/assets/src/map_controller.ts

@@ -188,6 +188,12 @@ export default class extends AbstractMapController<
         }
 
         if (icon) {
+            if (Object.prototype.hasOwnProperty.call(bridgeOptions, 'content')) {
+                console.warn('[Symfony UX Map] Defining "bridgeOptions.content" for a marker with a custom icon is not supported and will be ignored.');
+            } else if (Object.prototype.hasOwnProperty.call(rawOptions, 'content')) {
+                console.warn('[Symfony UX Map] Defining "rawOptions.content" for a marker with a custom icon is not supported and will be ignored.');
+            }
+
             this.doCreateIcon({ definition: icon, element: marker });
         }
 

src/Map/src/Bridge/Leaflet/CHANGELOG.md

@@ -3,6 +3,7 @@
 ## 2.30
 
 -  Ensure compatibility with PHP 8.5
+-  Display a warning when trying to define `bridgeOptions.icon` for a `Marker` that already has an `Icon`
 
 ## 2.29.0
 

src/Map/src/Bridge/Leaflet/assets/dist/map_controller.js

@@ -228,6 +228,11 @@ var map_controller_default = class extends abstract_map_controller_default {
       this.createInfoWindow({ definition: infoWindow, element: marker2 });
     }
     if (icon2) {
+      if (Object.prototype.hasOwnProperty.call(bridgeOptions, "icon")) {
+        console.warn('[Symfony UX Map] Defining "bridgeOptions.icon" for a marker with a custom icon is not supported and will be ignored.');
+      } else if (Object.prototype.hasOwnProperty.call(rawOptions, "icon")) {
+        console.warn('[Symfony UX Map] Defining "rawOptions.icon" for a marker with a custom icon is not supported and will be ignored.');
+      }
       this.doCreateIcon({ definition: icon2, element: marker2 });
     }
     return marker2;

src/Map/src/Bridge/Leaflet/assets/src/map_controller.ts

@@ -154,6 +154,12 @@ export default class extends AbstractMapController<
         }
 
         if (icon) {
+            if (Object.prototype.hasOwnProperty.call(bridgeOptions, 'icon')) {
+                console.warn('[Symfony UX Map] Defining "bridgeOptions.icon" for a marker with a custom icon is not supported and will be ignored.');
+            } else if (Object.prototype.hasOwnProperty.call(rawOptions, 'icon')) {
+                console.warn('[Symfony UX Map] Defining "rawOptions.icon" for a marker with a custom icon is not supported and will be ignored.');
+            }
+
             this.doCreateIcon({ definition: icon, element: marker });
         }