Fixes sillsdev/serval#365

sillsdev · Apr 16, 2024 · 5d3426f · 5d3426f
1 parent b4e91d0
commit 5d3426f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/SIL.Machine/Scripture/ScriptureRangeParser.cs b/src/SIL.Machine/Scripture/ScriptureRangeParser.cs
@@ -39,7 +39,7 @@ private Dictionary<string, List<int>> ParseSection(string section)
         Dictionary<string, List<int>> chaptersPerBook = new Dictionary<string, List<int>>();
 
         //*Specific chapters from one book*
-        if (char.IsDigit(section.Last()))
+        if (char.IsDigit(section.Last()) && section.Length > 3)
         {
             string bookName = section.Substring(0, 3);
             if (!_bookLengths.ContainsKey(bookName))

diff --git a/tests/SIL.Machine.Tests/Scripture/ScriptureRangeParserTests.cs b/tests/SIL.Machine.Tests/Scripture/ScriptureRangeParserTests.cs
@@ -27,6 +27,7 @@ public void TestParse(string rangeString, Dictionary<string, List<int>> expected
     public static IEnumerable<TestCaseData> GetCases()
     {
         yield return new TestCaseData("MAL", new Dictionary<string, List<int>> { { "MAL", new List<int>() } }, false);
+        yield return new TestCaseData("PS2", new Dictionary<string, List<int>> { { "PS2", new List<int>() } }, false);
         yield return new TestCaseData(
             "GEN,EXO",
             new Dictionary<string, List<int>> { { "GEN", new List<int>() }, { "EXO", new List<int>() } },