fixed react (and several other callback patching-related problems) #…

…640
Domiii · Jan 11, 2022 · 9c60cae · 9c60cae
1 parent e4965a7
commit 9c60cae
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 9 deletions.
diff --git a/dbux-runtime/src/async/CallbackPatcher.js b/dbux-runtime/src/async/CallbackPatcher.js
@@ -115,15 +115,15 @@ export default class CallbackPatcher {
       return originalCallback;
     }
 
-    // let f = getPatchedFunctionOrNull(originalFunction);
-    // if (!f) {
-    //   const calleePatcher = this.defaultCalleePatcher;
-
-    //   f = monkeyPatchFunctionOverride(
-    //     originalFunction,
-    //     calleePatcher.bind(this, callId)
-    //   );
-    // }
+    let f = getPatchedCallback(originalCallback);
+    if (f) {
+      return f;
+      // const calleePatcher = this.defaultCalleePatcher;
+      // f = monkeyPatchFunctionOverride(
+      //   originalFunction,
+      //   calleePatcher.bind(this, callId)
+      // );
+    }
 
     // const self = this; // NOTE: `this` will be the callback's `this`
 

diff --git a/samples/__samplesInput__/patching/callbacks0.js b/samples/__samplesInput__/patching/callbacks0.js
@@ -38,6 +38,7 @@ function nativeCallCompare(cb) {
    *    and many other callback management mechanisms cannot work correctly.
    */
   nativeCallStore(cb);
+  debugger;
   nativeCallCompare(cb);
 
   // TODO: more testing here, with checks for multi-patching