Merge pull request #171 from sttz/main

Fixed exception when viewing a Yarn Project in the inspector that contains no declarations

CHANGELOG.md

@@ -24,6 +24,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 - `ActionManager` no longer logs every single command that it registers. (#165)
 - Line view should no longer have unusual interactions around enabling and disabling different effects (#161 and #153).
 - Improved the type inference system around the use of functions.
+- Fixed exception when viewing a Yarn Project in the inspector that contains no declarations, in Unity 2021.2 and earlier (#168)
 
 This has two pieces, the first is in YarnSpinner Core and adds in support for partial backwards type inference.
 This means in many situations where either the l-value or r-value of an expression is known that can be used to provide a type to the other side of the equation.

Editor/Importers/YarnProjectImporter.cs

@@ -855,6 +855,15 @@ private bool ShouldShowElement(int index)
 
         private float GetElementHeight(int index, bool useSearch)
         {
+#if !UNITY_2022_1_OR_NEWER
+            // Before Unity 2022.1, this callback gets called with 
+            // index 0 even if the list is empty
+            if (serializedProperty.arraySize == 0)
+            {
+                return 0;
+            }
+#endif
+
             if (useSearch == false || ShouldShowElement(index))
             {
                 var item = serializedProperty.GetArrayElementAtIndex(index);