fix compilation error and remove from parents subTasks on failure

Author: beanbag44

src/main/kotlin/com/lambda/module/modules/movement/TargetStrafe.kt

@@ -17,9 +17,9 @@
 
 package com.lambda.module.modules.movement
 
-import com.lambda.config.ConfigEditor.forEachSetting
-import com.lambda.config.ConfigEditor.hideAllExcept
 import com.lambda.config.blocks.WorldLineSettings
+import com.lambda.config.forEachSetting
+import com.lambda.config.hideAllExcept
 import com.lambda.config.withEdits
 import com.lambda.context.SafeContext
 import com.lambda.event.events.MovementEvent

src/main/kotlin/com/lambda/task/Task.kt

@@ -132,7 +132,6 @@ abstract class Task<Result> : Nameable, Muteable {
     protected fun success(result: Result) {
         unsubscribe()
         state = State.Completed
-        parent?.subTasks?.remove(this)
 
         parent?.onSubTaskSuccess(this)
         parent?.onSubTaskCompletion(this)
@@ -141,6 +140,8 @@ abstract class Task<Result> : Nameable, Muteable {
             successCallbacks.forEach { it.invoke(this, result) }
             completionCallbacks.forEach { it.invoke(this) }
         }
+
+        parent?.subTasks?.remove(this)
     }
 
     @Ta5kBuilder
@@ -177,6 +178,8 @@ abstract class Task<Result> : Nameable, Muteable {
             failureCallbacks.forEach { it.invoke(this, e) }
             completionCallbacks.forEach { it.invoke(this) }
         }
+
+        parent?.subTasks?.remove(this)
     }
 
     @Ta5kBuilder