Previous fix to support data-format types broke SQLServer tests. Corrected with a patch to apply the data-format only when its of type String

mmccurrey · mmccurrey · commit 93dabef81909 · 2013-07-29T20:32:25.000Z
diff --git a/src/IBatisNet.DataMapper/Commands/DataReaderDecorator.cs b/src/IBatisNet.DataMapper/Commands/DataReaderDecorator.cs
@@ -252,7 +252,7 @@ DateTime IDataRecord.GetDateTime(int i)
         {
             string dateTimeFormat = ConfigurationManager.AppSettings["dateTimeFormat"];
 
-            if (!string.IsNullOrWhiteSpace(dateTimeFormat))
+            if ((!string.IsNullOrWhiteSpace(dateTimeFormat)) && (_innerDataReader.GetFieldType(i)==typeof(string)))
             {
                 return DateTime.ParseExact(_innerDataReader.GetString(i), dateTimeFormat, CultureInfo.InvariantCulture);
             }

Original file line number	Diff line number	Diff line change
`@@ -252,7 +252,7 @@ DateTime IDataRecord.GetDateTime(int i)`
`252`	`252`	`{`
`253`	`253`	`string dateTimeFormat = ConfigurationManager.AppSettings["dateTimeFormat"];`
`254`	`254`
`255`		`- if (!string.IsNullOrWhiteSpace(dateTimeFormat))`
	`255`	`+ if ((!string.IsNullOrWhiteSpace(dateTimeFormat)) && (_innerDataReader.GetFieldType(i)==typeof(string)))`
`256`	`256`	`{`
`257`	`257`	`return DateTime.ParseExact(_innerDataReader.GetString(i), dateTimeFormat, CultureInfo.InvariantCulture);`
`258`	`258`	`}`