Merge pull request #31 from KalopsiaTwilight/fix/constructrow-exception

Fix System.MissingMethodException on constructing a row with a string[]

Author: Marlamin

DBCD.Tests/WritingTest.cs

@@ -66,6 +66,21 @@ public void TestWritingNewRowDb2WithArrayField()
             Assert.AreEqual(116146, outputStorage.Count);
         }
 
+        [TestMethod]
+        public void TestWritingNewRowDb2WithArrayOfStringsField()
+        {
+            DBCD dbcd = new(wagoDBCProvider, githubDBDProvider);
+            IDBCDStorage storage = dbcd.Load("BattlePetEffectProperties", "9.2.7.45745");
+
+            storage.Add(10, storage.ConstructRow(10));
+            storage.Save(Path.Join(OutputPath, "BattlePetEffectProperties.db2"));
+
+            DBCD localDbcd = new(new FilesystemDBCProvider(OutputPath), githubDBDProvider);
+            IDBCDStorage outputStorage = localDbcd.Load("BattlePetEffectProperties", "9.2.7.45745");
+
+            Assert.AreEqual(134, outputStorage.Count);
+        }
+
         [TestMethod]
         public void TestSavingSameStorageTwice()
         {

DBCD/DBCDStorage.cs

@@ -204,10 +204,19 @@ public DBCDRow ConstructRow(int index)
             foreach (var arrayField in arrayFields)
             {
                 var count = arrayField.GetCustomAttribute<CardinalityAttribute>().Count;
-                Array rowRecords = Array.CreateInstance(arrayField.FieldType.GetElementType(), count);
+                var elementType = arrayField.FieldType.GetElementType();
+                var isStringField = elementType == typeof(string);
+
+                Array rowRecords = Array.CreateInstance(elementType, count);
                 for (var i = 0; i < count; i++)
                 {
-                    rowRecords.SetValue(Activator.CreateInstance(arrayField.FieldType.GetElementType()), i);
+                    if (isStringField)
+                    {
+                        rowRecords.SetValue(string.Empty, i);
+                    } else
+                    {
+                        rowRecords.SetValue(Activator.CreateInstance(elementType), i);
+                    }
                 }
                 arrayField.SetValue(raw, rowRecords);
             }