Require specifying the view to re-evaluate

Author: Aeltumn

interfaces/src/main/kotlin/com/noxcrew/interfaces/properties/LazyProperty.kt

@@ -52,7 +52,7 @@ public abstract class LazyProperty<T : Any>(
     public abstract suspend fun load(reload: Boolean = true): T
 
     /** Initializes this property if it hasn't already. */
-    public suspend fun initialize(view: InterfaceView? = null) {
+    public suspend fun initialize(view: InterfaceView?) {
         if (!initialized || value == null) {
             reevaluate(reload = !initialized, view = view)
         }
@@ -63,10 +63,10 @@ public abstract class LazyProperty<T : Any>(
      * If [reload] is given, all data should be fully reloaded.
      */
     public suspend fun reevaluate(
+        view: InterfaceView?,
         reload: Boolean = true,
         trigger: Boolean = true,
         debounce: Duration = 200.milliseconds,
-        view: InterfaceView? = null
     ): T {
         // Mark down if we want an update to happen
         if (trigger) {

interfaces/src/main/kotlin/com/noxcrew/interfaces/properties/StateProperty.kt

@@ -39,7 +39,7 @@ public abstract class StateProperty(
     private var initialized: Boolean = false
 
     /** Initializes this property if it hasn't already. */
-    public suspend fun initialize(view: InterfaceView? = null) {
+    public suspend fun initialize(view: InterfaceView?) {
         if (!initialized) {
             refresh(view = view)
         }
@@ -49,7 +49,7 @@ public abstract class StateProperty(
      * Refreshes this property, updating before triggering the state.
      * Ignored if last refresh was within [debounce].
      */
-    public suspend fun refresh(debounce: Duration = 200.milliseconds, view: InterfaceView? = null) {
+    public suspend fun refresh(debounce: Duration = 200.milliseconds, view: InterfaceView?) {
         // Await any existing job if one is running
         if (updateJob != null) {
             updateJob?.await()