Merge pull request cerebral#82 from etienne-dldc/next

christianalfoni · web-flow · commit b4f7b163d6aa · 2018-09-15T12:02:00.000+02:00
fix(action-chain): add fall through in effect proxying
diff --git a/packages/node_modules/action-chain/src/utils.ts b/packages/node_modules/action-chain/src/utils.ts
@@ -35,11 +35,13 @@ function createProxyGetHandler(
         }
         return result
       }
-    } else if (isObject(target[prop])) {
+    }
+    if (isObject(target[prop])) {
       return new Proxy(target[prop], {
         get: createProxyGetHandler(path + '.' + prop.toString(), cb),
       })
     }
+    return target[prop]
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -35,11 +35,13 @@ function createProxyGetHandler(`
`35`	`35`	`}`
`36`	`36`	`return result`
`37`	`37`	`}`
`38`		`- } else if (isObject(target[prop])) {`
	`38`	`+ }`
	`39`	`+ if (isObject(target[prop])) {`
`39`	`40`	`return new Proxy(target[prop], {`
`40`	`41`	`get: createProxyGetHandler(path + '.' + prop.toString(), cb),`
`41`	`42`	`})`
`42`	`43`	`}`
	`44`	`+ return target[prop]`
`43`	`45`	`}`
`44`	`46`	`}`
`45`	`47`