Avoid exception at ParseExactDate (#198)

claudiamurialdo · cmurialdo · web-flow · commit 186283ed592c · 2020-09-19T09:59:28.000-03:00
Co-authored-by: claudia &lt;cmurialdo@gmail.com&gt;
diff --git a/dotnet/src/dotnetframework/GxClasses/Core/GXUtilsCommon.cs b/dotnet/src/dotnetframework/GxClasses/Core/GXUtilsCommon.cs
@@ -2779,10 +2779,15 @@ DateTime ParseExactDate(string strDate, int picFmt)
 			string[] fmtVec = DateFormatFromPicture(picFmt);
 			string[] fmts = modifyFormatStrings(fmtVec);
 			string oldSeparator = cultureInfo.DateTimeFormat.DateSeparator;
+
 			try
 			{
+				DateTime dValue;
 				if (oldSeparator != "/") cultureInfo.DateTimeFormat.DateSeparator = "/";
-				return DateTime.ParseExact(strDate.Trim(), fmts, cultureInfo.DateTimeFormat, DateTimeStyles.AllowWhiteSpaces | DateTimeStyles.NoCurrentDateDefault);
+				if (DateTime.TryParseExact(strDate.Trim(), fmts, cultureInfo.DateTimeFormat, DateTimeStyles.AllowWhiteSpaces | DateTimeStyles.NoCurrentDateDefault, out dValue))
+					return dValue;
+				else
+					return nullDate;
 			}
 			catch
 			{

Original file line number	Diff line number	Diff line change
`@@ -2779,10 +2779,15 @@ DateTime ParseExactDate(string strDate, int picFmt)`
`2779`	`2779`	`string[] fmtVec = DateFormatFromPicture(picFmt);`
`2780`	`2780`	`string[] fmts = modifyFormatStrings(fmtVec);`
`2781`	`2781`	`string oldSeparator = cultureInfo.DateTimeFormat.DateSeparator;`
	`2782`	`+`
`2782`	`2783`	`try`
`2783`	`2784`	`{`
	`2785`	`+ DateTime dValue;`
`2784`	`2786`	`if (oldSeparator != "/") cultureInfo.DateTimeFormat.DateSeparator = "/";`
`2785`		`- return DateTime.ParseExact(strDate.Trim(), fmts, cultureInfo.DateTimeFormat, DateTimeStyles.AllowWhiteSpaces \| DateTimeStyles.NoCurrentDateDefault);`
	`2787`	`+ if (DateTime.TryParseExact(strDate.Trim(), fmts, cultureInfo.DateTimeFormat, DateTimeStyles.AllowWhiteSpaces \| DateTimeStyles.NoCurrentDateDefault, out dValue))`
	`2788`	`+ return dValue;`
	`2789`	`+ else`
	`2790`	`+ return nullDate;`
`2786`	`2791`	`}`
`2787`	`2792`	`catch`
`2788`	`2793`	`{`