Merge pull request #16 from dotnetprojects/complSign

bugfix constants

Author: jogibear9988

TiaCodeGen.Tests/SampleTests.cs

@@ -358,5 +358,15 @@ public void ComplexVariableAccessTest2()
             var xml = sb.ToString();
             Assert.AreEqual("<Access Scope=\"LocalVariable\" UId=\"1\">\r\n<Symbol>\r\n<Component Name=\"A\">\r\n</Component>\r\n<Component Name=\"B\">\r\n</Component>\r\n<Component Name=\"C\">\r\n<Access Scope=\"LocalVariable\">\r\n<Symbol>\r\n<Component Name=\"D\" />\r\n<Component Name=\"E\" />\r\n<Component Name=\"F\" />\r\n</Symbol>\r\n</Access>\r\n<Access Scope=\"LocalVariable\">\r\n<Symbol>\r\n<Component Name=\"G\" />\r\n<Component Name=\"H\" />\r\n<Component Name=\"I\" />\r\n</Symbol>\r\n</Access>\r\n</Component>\r\n</Symbol>\r\n</Access>\r\n".Replace("\n", "").Replace("\r", ""), xml.Replace("\n", "").Replace("\r", ""));
         }
+
+        [Test]
+        public void TestConstant()
+        {
+            var sb = new StringBuilder();
+            var s = new Signal("AAA", SignalType.Constant);
+            s.AddXmlToStringBuilder(1, sb);
+            var xml = sb.ToString();
+            Assert.AreEqual("<Access Scope=\"GlobalConstant\" UId=\"1\">\r\n<Constant Name=\"AAA\">\r\n</Constant>\r\n</Access>\r\n".Replace("\n", "").Replace("\r", ""), xml.Replace("\n", "").Replace("\r", ""));
+        }
     }
 }

TiaCodegen/Commands/Signals/Signal.cs

@@ -102,7 +102,10 @@ public void AddXmlToStringBuilder(ulong id, StringBuilder sb)
             {
                 var value = Name.IndexOf(",");
                 sb.AppendLine("<Access Scope=\"GlobalConstant\" UId=\"" + id + "\">");
-                sb.AppendLine("<Constant Name=\"" + Name.Substring(0, value) + "\">");
+                if (value == -1)
+                    sb.AppendLine("<Constant Name=\"" + Name + "\">");
+                else
+                    sb.AppendLine("<Constant Name=\"" + Name.Substring(0, value) + "\">");
                 sb.AppendLine("</Constant>");
                 sb.AppendLine("</Access>");
             }