fix - Fixed false negative card validity

--- When there are multiple new lines, VisualCard might inadvertently look for the beginning of another vCard, which doesn't exist in this case. This fixes this bug where VC thinks that the otherwise valid vCard is "invalid." --- Type: fix Breaking: False Doc Required: False Backport Required: False Part: 1/1
Aptivi · Oct 3, 2024 · 965e065 · 965e065
1 parent b34e179
commit 965e065
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/VisualCard.Calendar/CalendarTools.cs b/VisualCard.Calendar/CalendarTools.cs
@@ -107,10 +107,7 @@ public static Parts.Calendar[] GetCalendars(StreamReader stream)
 
                 // Process the line for begin, version, and end specifiers
                 if (string.IsNullOrEmpty(CalendarLine))
-                {
-                    if (!stream.EndOfStream)
-                        continue;
-                }
+                    continue;
                 else if ((!prefix.EqualsNoCase(VcardConstants._beginSpecifier) &&
                           !prefix.EqualsNoCase(VcardConstants._versionSpecifier) &&
                           !prefix.EqualsNoCase(VcardConstants._endSpecifier)) ||

diff --git a/VisualCard/CardTools.cs b/VisualCard/CardTools.cs
@@ -111,10 +111,7 @@ public static Card[] GetCards(StreamReader stream)
 
                 // Process the line for begin, version, and end specifiers
                 if (string.IsNullOrEmpty(CardLine))
-                {
-                    if (!stream.EndOfStream)
-                        continue;
-                }
+                    continue;
                 else if ((!prefix.EqualsNoCase(VcardConstants._beginSpecifier) &&
                          !prefix.EqualsNoCase(VcardConstants._versionSpecifier) &&
                          !prefix.EqualsNoCase(VcardConstants._endSpecifier)) || isAgent)