From 3c6f051b97dcd0e253b4d90afc50991f9f68f579 Mon Sep 17 00:00:00 2001
From: Laurent Christen <laurent@laurent-christen.ch>
Date: Fri, 10 Nov 2023 08:55:06 +0100
Subject: [PATCH] fix marker id comparison (#22)

Co-authored-by: lolochristen <lolochristen@users.noreply.github.com>
---
 src/OpenLayers.Blazor.Demo.Components/Pages/Index.razor | 2 +-
 src/OpenLayers.Blazor/Map.razor.cs                      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/OpenLayers.Blazor.Demo.Components/Pages/Index.razor b/src/OpenLayers.Blazor.Demo.Components/Pages/Index.razor
index 543000d..3292592 100644
--- a/src/OpenLayers.Blazor.Demo.Components/Pages/Index.razor
+++ b/src/OpenLayers.Blazor.Demo.Components/Pages/Index.razor
@@ -50,7 +50,7 @@ Visible Extent: @_map?.VisibleExtent
         </div>
     </Popup>
     <Features>
-        <Marker Type="MarkerType.MarkerPin" Coordinate="new Coordinate(1197650, 2604200)"></Marker>
+        <Marker Type="MarkerType.MarkerPin" Coordinate="new Coordinate(1197650, 2604200)" Popup></Marker>
         <Marker Type="MarkerType.MarkerFlag" Coordinate="new Coordinate(1177650, 2624200)" Title="Hallo" BackgroundColor="#449933" Popup></Marker>
          <Line Points="new[] { new Coordinate(1197650, 2604200), new Coordinate(1177650, 2624200) }" BorderColor="cyan"></Line>
      </Features>
diff --git a/src/OpenLayers.Blazor/Map.razor.cs b/src/OpenLayers.Blazor/Map.razor.cs
index 213033e..8d68920 100644
--- a/src/OpenLayers.Blazor/Map.razor.cs
+++ b/src/OpenLayers.Blazor/Map.razor.cs
@@ -310,7 +310,7 @@ public async Task OnInternalMarkerClick(Internal.Marker marker)
 #if DEBUG
         Console.WriteLine($"OnInternalMarkerClick: {JsonSerializer.Serialize(marker)}");
 #endif
-        var m = MarkersList.FirstOrDefault(p => p.InternalFeature.Id == marker.Id);
+        var m = MarkersList.FirstOrDefault(p => string.Equals(p.InternalFeature.Id.ToString(), marker.Id.ToString(), StringComparison.OrdinalIgnoreCase));
 
         if (m != null)
         {