feat: optionally allow dispatching the current route by disabling the…

… preventDoubleDispatch middleware

packages/rudy/src/core/createReducer.js

@@ -26,7 +26,7 @@ export default (initialState: Object, routes: Routes) => (
     return { ...st, ...act, ...location }
   }
 
-  if (r && r.path && (l.url !== st.url || /load|reset/.test(l.kind))) {
+  if (r && r.path) {
     const { type, params, query, state, hash, basename } = action
     const { universal } = st
     const s = { type, params, query, state, hash, basename, universal, ...l }

packages/rudy/src/core/createRouter.js

@@ -28,6 +28,7 @@ import {
   pathlessRoute,
   anonymousThunk,
   transformAction,
+  preventDoubleDispatch,
   call,
   enter,
   changePageTitle,
@@ -41,6 +42,7 @@ export default (
     anonymousThunk,
     pathlessRoute('thunk'),
     transformAction, // pipeline starts here
+    preventDoubleDispatch,
     // Hydrate: skip callbacks called on server to produce initialState (beforeEnter, thunk, etc)
     // Server: don't allow client-centric callbacks (onEnter, onLeave, beforeLeave)
     call('beforeLeave', { prev: true }),

packages/rudy/src/middleware/index.js

@@ -1,4 +1,5 @@
 export { default as transformAction } from './transformAction'
+export { default as preventDoubleDispatch } from './preventDoubleDispatch'
 export { default as enter } from './enter'
 export { default as call } from './call'
 

packages/rudy/src/middleware/preventDoubleDispatch.js

@@ -0,0 +1,5 @@
+export default () => (req, next) => {
+  if (!req.route.path) return next()
+  if (req.isDoubleDispatch()) return req.handleDoubleDispatch() // don't dispatch the same action twice
+  return next()
+}

packages/rudy/src/middleware/transformAction/index.js

@@ -5,8 +5,6 @@ export default () => (req, next) => {
 
   req.action = formatAction(req)
 
-  if (req.isDoubleDispatch()) return req.handleDoubleDispatch() // don't dispatch the same action twice
-
   const { type, params, query, state, hash, basename, location } = req.action
   Object.assign(req, { type, params, query, state, hash, basename, location }) // assign to `req` for conevenience (less destructuring in callbacks)