Fixed ArgumentOutOfRangeException for FunctionSelect

erri120 · Nov 28, 2019 · d8239ac · d8239ac
1 parent 986e70d
commit d8239ac
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/OMODFramework.Test/ScriptTest.cs b/OMODFramework.Test/ScriptTest.cs
@@ -33,13 +33,16 @@ public class ScriptTest : DownloadTest
         [TestMethod]
         public void TestOBMMScript()
         {
+
             var omod = new OMOD(FileName);
 
             Assert.IsNotNull(omod);
 
             var scriptFunctions = new ScriptFunctions();
 
-            omod.RunScript(scriptFunctions);
+            var srd = omod.RunScript(scriptFunctions);
+
+            Assert.IsNotNull(srd);
         }
     }
 }
diff --git a/OMODFramework/Scripting/OBMMScriptHandler.cs b/OMODFramework/Scripting/OBMMScriptHandler.cs
@@ -230,9 +230,6 @@ internal static ScriptReturnData Execute(string inputScript, string dataPath, st
                     case "Continue":
                     case "Exit":
                         break;
-                    default:
-                        Warn($"Unrecognized function: {line[0]}!");
-                        break;
                     }
                 }
                 else
@@ -1015,15 +1012,15 @@ private static string[] FunctionSelect(IList<string> line, bool isMultiSelect, b
 
             for (var i = 0; i < line.Count / argsPerOption; i++)
             {
-                items[i] = line[i * argsPerOption];
+                items.Add(line[i * argsPerOption]);
                 if (hasPreviews)
                 {
-                    previews[i] = line[i * argsPerOption + 1];
-                    if (hasDescriptions) descriptions[i] = line[i * argsPerOption + 2];
+                    previews.Add(line[i * argsPerOption + 1]);
+                    if (hasDescriptions) descriptions.Add(line[i * argsPerOption + 2]);
                 }
                 else
                 {
-                    if (hasDescriptions) descriptions[i] = line[i * argsPerOption + 1];
+                    if (hasDescriptions) descriptions.Add(line[i * argsPerOption + 1]);
                 }
             }