🐞 Fix: RNIModalSheetView.onModalSheetStateDidChange Event

dominicstop · Sep 29, 2024 · 65b307a · 65b307a
1 parent 432a15a
commit 65b307a
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/ios/RNIModalSheetView/RNIModalSheetViewDelegate.swift b/ios/RNIModalSheetView/RNIModalSheetViewDelegate.swift
@@ -326,13 +326,16 @@ extension RNIModalSheetViewDelegate: ModalSheetPresentationStateEventsNotifiable
     prevState: ModalSheetState?,
     currentState: ModalSheetState
   ) {
-    let payload: Dictionary<String, Any> = [
-      "prevState": currentState.asDictionary,
+    var payload: Dictionary<String, Any> = [
       "currentState": currentState.asDictionary
     ];
 
+    payload.unwrapAndMerge(withOther: [
+      "prevState": prevState?.asDictionary
+    ]);
+
     self.dispatchEvent(
-      for: .onModalSheetStateWillChange,
+      for: .onModalSheetStateDidChange,
       withPayload: payload
     );
   };

diff --git a/ios/Temp/ModalSheetPresentationStateMachine.swift b/ios/Temp/ModalSheetPresentationStateMachine.swift
@@ -24,6 +24,10 @@ public class ModalSheetPresentationStateMachine {
   // ---------------
 
   public func setStateExplicit(nextState: ModalSheetState){
+    guard self.currentState != nextState else {
+      return;
+    };
+
     let prevState = self.prevState;
     let currentState = self.currentState;