Fix Cache Key. Use Mapper Type Instead of Instance

CollaboratingPlatypus · Nov 4, 2023 · 2279cf9 · 2279cf9
1 parent 63c885c
commit 2279cf9
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/PetaPoco/Core/PocoData.cs b/PetaPoco/Core/PocoData.cs
@@ -16,7 +16,7 @@ public class PocoData
     {
         private static readonly object _converterLock = new object();
 
-        private static Cache<Tuple<IMapper, Type>, PocoData> _pocoDatas = new Cache<Tuple<IMapper, Type>, PocoData>();
+        private static Cache<Tuple<Type, Type>, PocoData> _pocoDatas = new Cache<Tuple<Type, Type>, PocoData>();
         private static List<Func<object, object>> _converters = new List<Func<object, object>>();
         private static MethodInfo fnGetValue = typeof(IDataRecord).GetMethod("GetValue", new Type[] { typeof(int) });
         private static MethodInfo fnIsDBNull = typeof(IDataRecord).GetMethod("IsDBNull");
@@ -125,7 +125,7 @@ public static PocoData ForType(Type type, IMapper defaultMapper)
             if (type == typeof(System.Dynamic.ExpandoObject))
                 throw new InvalidOperationException("Cannot use dynamic types with this method");
 
-            var key = new Tuple<IMapper, Type>(Mappers.GetMapper(type, defaultMapper), type);
+            var key = new Tuple<Type, Type>(Mappers.GetMapper(type, defaultMapper).GetType(), type);
             return _pocoDatas.GetOrAdd(key, () => new PocoData(type, defaultMapper));
         }