Merge pull request #179 from matwilko/idictionary-fix

Allow passing an IDictionary<string, object> as parameter object

Dapper NET40/SqlMapper.cs

@@ -2048,9 +2048,12 @@ private static CacheInfo GetCacheInfo(Identity identity, object exampleParameter
                     {
                         info.ParamReader = (cmd, obj) => { ((IDynamicParameters)obj).AddParameters(cmd, identity); };
                     }
-#if !CSHARP30
-                    // special-case dictionary && `dynamic`
-                    else if (exampleParameters is IEnumerable<KeyValuePair<string, object>> && exampleParameters is System.Dynamic.IDynamicMetaObjectProvider)
+#if CSHARP30
+                    else if (exampleParameters is IEnumerable<KeyValuePair<string, object>>)
+#else
+                    // special-case dictionary and `dynamic`
+                    else if (exampleParameters is IEnumerable<KeyValuePair<string, object>> || exampleParameters is System.Dynamic.IDynamicMetaObjectProvider)
+#endif
                     {
                         info.ParamReader = (cmd, obj) =>
                         {
@@ -2058,7 +2061,6 @@ private static CacheInfo GetCacheInfo(Identity identity, object exampleParameter
                             mapped.AddParameters(cmd, identity);
                         };
                     }
-#endif
                     else
                     {
                         var literals = GetLiteralTokens(identity.sql);

Tests/Tests.cs

@@ -3842,6 +3842,15 @@ public void SO25297173_DynamicIn()
             result.Contains(6).IsTrue();
         }
 
+        public void AllowIDictionaryParameters()
+        {
+            var parameters = new Dictionary<string, object>
+            {
+                { "param1", 0 }
+            };
+
+            connection.Query("SELECT @param1", parameters);
+        }
 #if POSTGRESQL
 
         class Cat