Fix: Resolve ambiguity in TryGetElementType() for Mono runtime (#3395)

npgsql · Dec 5, 2024 · 6cb9612 · 6cb9612
1 parent c6fb5e2
commit 6cb9612
Showing 1 changed file with 17 additions and 13 deletions.
diff --git a/src/Shared/SharedTypeExtensions.cs b/src/Shared/SharedTypeExtensions.cs
@@ -215,23 +215,27 @@ public static Type GetSequenceType(this Type type)
             return null;
         }
 
-        var types = GetGenericTypeImplementations(type, interfaceOrBaseType);
+	    var types = GetGenericTypeImplementations(type, interfaceOrBaseType);
 
-        Type? singleImplementation = null;
-        foreach (var implementation in types)
-        {
-            if (singleImplementation is null)
-            {
-                singleImplementation = implementation;
-            }
-            else
+	    if (!types.Any())
+    	{	
+            return null;
+    	}
+
+    	if (type.IsArray)
+    	{
+            var elementType = type.GetElementType();
+            if (elementType != null)
             {
-                singleImplementation = null;
-                break;
+                var arrayCompatible = types.FirstOrDefault(t => t.GenericTypeArguments.Contains(elementType));
+                if (arrayCompatible != null)
+                {
+                    return arrayCompatible.GenericTypeArguments.First();
+                }
             }
-        }
+    	}
 
-        return singleImplementation?.GenericTypeArguments.FirstOrDefault();
+    	return types.First().GenericTypeArguments.FirstOrDefault();
     }
 
 #nullable disable