make useReverse() not depend on current route;

improve id detection in outbox delete
wq · Aug 31, 2023 · 265d8e2 · 265d8e2
1 parent a025328
commit 265d8e2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/packages/outbox/src/outbox.js b/packages/outbox/src/outbox.js
@@ -176,8 +176,6 @@ class Outbox {
             data,
             options,
         });
-        data = item.data;
-        options = item.options;
 
         var currentId;
         if (data && data.id) {
@@ -193,6 +191,9 @@ class Outbox {
             }
         }
 
+        options = item.options;
+        data = item.data;
+
         var type, payload, commitType, rollbackType;
         const model = this._getModel(options.modelConf);
         if (model) {

diff --git a/packages/react/src/hooks.js b/packages/react/src/hooks.js
@@ -118,13 +118,13 @@ export function useRouteTitle(routeName) {
 }
 
 export function useReverse() {
-    const location = useSelector(selectLocationState);
+    const routesMap = useRoutesMap();
     return useCallback(
         (name, payload, query) => {
             const action = {
                 type: name.toUpperCase(),
             };
-            if (!location.routesMap[action.type]) {
+            if (!routesMap[action.type]) {
                 throw new Error(`Unknown route: ${action.type}`);
             }
             if (payload) {
@@ -139,7 +139,7 @@ export function useReverse() {
             }
             return action;
         },
-        [location]
+        [routesMap]
     );
 }