Fixed sprite animation groups not advancing to their first frame immediately

Author: Ellpeck

CHANGELOG.md

@@ -3,6 +3,10 @@ MLEM uses [semantic versioning](https://semver.org/). Potentially breaking chang
 
 ## 8.1.0 (In Development)
 
+### MLEM
+Fixes
+- Fixed sprite animation groups not advancing to their first frame immediately
+
 ### MLEM.Ui
 Additions
 - Added UiSystem.OnElementAreaDirty and Element.OnAreaDirty events

MLEM/Animations/SpriteAnimationGroup.cs

@@ -16,7 +16,8 @@ public class SpriteAnimationGroup : GenericDataHolder {
         /// </summary>
         public SpriteAnimation CurrentAnimation {
             get {
-                this.SortAnimationsIfDirty(true);
+                if (this.SortAnimationsIfDirty() || this.currentAnimation == null)
+                    this.FindAnimationToPlay();
                 return this.currentAnimation?.Animation;
             }
         }
@@ -99,7 +100,7 @@ public SpriteAnimation ByName(string name) {
         }
 
         private void FindAnimationToPlay() {
-            this.SortAnimationsIfDirty(false);
+            this.SortAnimationsIfDirty();
 
             ConditionedAnimation animToPlay = null;
             if (this.currentAnimation != null && this.currentAnimation.ShouldPlay())
@@ -122,13 +123,12 @@ private void FindAnimationToPlay() {
             }
         }
 
-        private void SortAnimationsIfDirty(bool findAnimationToPlay) {
-            if (this.animationsDirty) {
-                this.animationsDirty = false;
-                this.animations.Sort((a1, a2) => a2.Priority.CompareTo(a1.Priority));
-                if (findAnimationToPlay)
-                    this.FindAnimationToPlay();
-            }
+        private bool SortAnimationsIfDirty() {
+            if (!this.animationsDirty)
+                return false;
+            this.animationsDirty = false;
+            this.animations.Sort((a1, a2) => a2.Priority.CompareTo(a1.Priority));
+            return true;
         }
 
         /// <summary>