Cisco: fix bug with RefObjectName

Botinoc · Botinoc · commit f5eb8bcbe5af · 2022-07-26T08:55:17.000+03:00
diff --git a/CiscoMigration/CiscoCommands.cs b/CiscoMigration/CiscoCommands.cs
@@ -1846,6 +1846,7 @@ public override void Parse(CiscoCommand command, CiscoCommand prevCommand, Dicti
         }
     }
 
+
     public class Cisco_AccessList : CiscoCommand
     {
         public enum ActionType { NA, Deny, Permit };
@@ -1907,6 +1908,9 @@ public SourceDest(List<string> words) : this()
                         if (words.Count > 1)
                         {
                             RefObjectName = words[1];
+                            var NameValidityRegex = @"[^A-Za-z0-9_.-]";
+                            RefObjectName = Validators.ChangeNameAccordingToRules(RefObjectName);
+                            Regex.Replace(RefObjectName, NameValidityRegex, "_");
                             WordsCount = 2;
                         }
                         break;
@@ -1916,6 +1920,9 @@ public SourceDest(List<string> words) : this()
                         if (words.Count > 1)
                         {
                             RefObjectName = InterfacePrefix + words[1];
+                            var NameValidityRegex = @"[^A-Za-z0-9_.-]";
+                            RefObjectName = Validators.ChangeNameAccordingToRules(RefObjectName);
+                            Regex.Replace(RefObjectName, NameValidityRegex, "_");
                             WordsCount = 2;
                         }
                         break;

Original file line number	Diff line number	Diff line change
`@@ -1846,6 +1846,7 @@ public override void Parse(CiscoCommand command, CiscoCommand prevCommand, Dicti`
`1846`	`1846`	`}`
`1847`	`1847`	`}`
`1848`	`1848`
	`1849`	`+`
`1849`	`1850`	`public class Cisco_AccessList : CiscoCommand`
`1850`	`1851`	`{`
`1851`	`1852`	`public enum ActionType { NA, Deny, Permit };`
`@@ -1907,6 +1908,9 @@ public SourceDest(List<string> words) : this()`
`1907`	`1908`	`if (words.Count > 1)`
`1908`	`1909`	`{`
`1909`	`1910`	`RefObjectName = words[1];`
	`1911`	`+ var NameValidityRegex = @"[^A-Za-z0-9_.-]";`
	`1912`	`+ RefObjectName = Validators.ChangeNameAccordingToRules(RefObjectName);`
	`1913`	`+ Regex.Replace(RefObjectName, NameValidityRegex, "_");`
`1910`	`1914`	`WordsCount = 2;`
`1911`	`1915`	`}`
`1912`	`1916`	`break;`
`@@ -1916,6 +1920,9 @@ public SourceDest(List<string> words) : this()`
`1916`	`1920`	`if (words.Count > 1)`
`1917`	`1921`	`{`
`1918`	`1922`	`RefObjectName = InterfacePrefix + words[1];`
	`1923`	`+ var NameValidityRegex = @"[^A-Za-z0-9_.-]";`
	`1924`	`+ RefObjectName = Validators.ChangeNameAccordingToRules(RefObjectName);`
	`1925`	`+ Regex.Replace(RefObjectName, NameValidityRegex, "_");`
`1919`	`1926`	`WordsCount = 2;`
`1920`	`1927`	`}`
`1921`	`1928`	`break;`