Remove StrictCompletionOrder from SlideCheckpoint

LumpBloom7 · Jul 22, 2022 · 1849be3 · 1849be3
1 parent d1f2418
commit 1849be3
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 19 deletions.
diff --git a/osu.Game.Rulesets.Sentakki/Objects/Drawables/DrawableSlideCheckpoint.cs b/osu.Game.Rulesets.Sentakki/Objects/Drawables/DrawableSlideCheckpoint.cs
@@ -28,13 +28,7 @@ public class DrawableSlideCheckpoint : DrawableSentakkiHitObject
         // All hits can only be done after the parent StartTime
         public bool IsHittable => Time.Current > ParentHitObject.HitObject.StartTime && isPreviousNodeHit();
 
-        private bool isPreviousNodeHit()
-        {
-            if (HitObject.StrictCompletionOrder)
-                return ThisIndex < 1 || parentSlide.SlideCheckpoints[ThisIndex - 1].IsHit;
-
-            return ThisIndex < 2 || parentSlide.SlideCheckpoints[ThisIndex - 2].IsHit;
-        }
+        private bool isPreviousNodeHit() => ThisIndex < 2 || parentSlide.SlideCheckpoints[ThisIndex - 2].IsHit;
 
         private Container<DrawableSlideCheckpointNode> nodes;
 

diff --git a/osu.Game.Rulesets.Sentakki/Objects/SlideBody.cs b/osu.Game.Rulesets.Sentakki/Objects/SlideBody.cs
@@ -44,11 +44,11 @@ protected override void CreateNestedHitObjects(CancellationToken cancellationTok
 
         protected virtual void CreateSlideCheckpoints()
         {
-            var distance = SlideInfo.SlidePath.TotalDistance;
-            var nodeCount = (int)Math.Floor(distance / 100);
+            double distance = SlideInfo.SlidePath.TotalDistance;
+            int nodeCount = (int)Math.Floor(distance / 100);
             for (int i = 0; i < nodeCount; i++)
             {
-                var progress = (double)(i + 1) / nodeCount;
+                double progress = (double)(i + 1) / nodeCount;
                 SlideCheckpoint checkpoint = new SlideCheckpoint()
                 {
                     Progress = (float)progress,

diff --git a/osu.Game.Rulesets.Sentakki/Objects/SlideCheckpoint.cs b/osu.Game.Rulesets.Sentakki/Objects/SlideCheckpoint.cs
@@ -8,11 +8,6 @@ namespace osu.Game.Rulesets.Sentakki.Objects
 {
     public class SlideCheckpoint : SentakkiHitObject
     {
-        public SlideCheckpoint(bool strictMode = false)
-        {
-            StrictCompletionOrder = strictMode;
-        }
-
         // Used to update slides visuals
         public float Progress { get; set; }
 
@@ -21,10 +16,6 @@ public SlideCheckpoint(bool strictMode = false)
 
         public int NodesToPass { get; set; } = 1;
 
-        // If this is true, then all previous checkpoints must be completed to hit this one
-        // This is in contrast to non-strict, where you can hit it if the n-2 checkpoint is hit
-        public bool StrictCompletionOrder { get; }
-
         protected override void CreateNestedHitObjects(CancellationToken cancellationToken)
         {
             foreach (var nodePosition in NodePositions)