Fixed by childReactiveState to call onReactiveStateAttached(child)

Author: wkornewald

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 5.12.0
+
+* Fixed `by childReactiveState` to call `onReactiveStateAttached(child)` (the arguments were inverted).
+
 ## 5.11.0
 
 * Added `MutableStateFlow<T>.collectAsMutableState()` to create a two-way binding.

reactivestate-core/src/commonMain/kotlin/com/ensody/reactivestate/OnReactiveStateAttached.kt

@@ -9,9 +9,3 @@ package com.ensody.reactivestate
 public interface OnReactiveStateAttached {
     public fun onReactiveStateAttached(reactiveState: ReactiveState<out ErrorEvents>)
 }
-
-public fun ReactiveState<out ErrorEvents>.attachTo(owner: Any) {
-    if (owner is OnReactiveStateAttached) {
-        owner.onReactiveStateAttached(this)
-    }
-}

reactivestate-core/src/commonMain/kotlin/com/ensody/reactivestate/ReactiveState.kt

@@ -104,7 +104,7 @@ public fun <E : ErrorEvents, P : ReactiveState<out E>, RS : ReactiveState<E>> P.
     launch(withLoading = null) {
         eventNotifier.emitAll(child.eventNotifier)
     }
-    attachTo(child)
+    (this as? OnReactiveStateAttached)?.onReactiveStateAttached(child)
     return WrapperProperty(child)
 }
 

reactivestate/src/androidMain/kotlin/com/ensody/reactivestate/android/ReactiveViewModelExt.kt

@@ -9,9 +9,9 @@ import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
 import com.ensody.reactivestate.ErrorEvents
 import com.ensody.reactivestate.NamespacedStateFlowStore
+import com.ensody.reactivestate.OnReactiveStateAttached
 import com.ensody.reactivestate.ReactiveState
 import com.ensody.reactivestate.StateFlowStore
-import com.ensody.reactivestate.attachTo
 import kotlinx.coroutines.CoroutineScope
 import kotlin.collections.set
 import kotlin.reflect.KClass
@@ -47,7 +47,7 @@ public fun <E : ErrorEvents> Lazy<ReactiveState<E>>.attachLazyReactiveState(
         value.eventNotifier.handleEvents(handler, owner)
     }
     owner.launchOnceStateAtLeast(Lifecycle.State.CREATED) {
-        value.attachTo(owner)
+        (owner as? OnReactiveStateAttached)?.onReactiveStateAttached(value)
     }
 }