3.2.0 Global not defined with in isPlainObject

[jaidetree]

I'm building our app files with browserify, babel 6.4, envify, react-redux, react 0.14 and the bundles are throwing this error that global is not defined. I take it envify or loose envify is not factoring out the globals?

For now I have reverted to 3.1.7 of redux and 4.1.2 of react-redux but of course it would be nice to see a real fix.

[gaearon]

Thank you for reporting. What bundler do you use? We should definitely fix this but a temporary workaround would be to enable global polyfill in your bundler (e.g. Webpack does this by default).

cc @jdalton: this means libraries can't depend on Lodash modules using global without making assumptions about bundlers in browsers.

[gaearon]

Another option would be to temporarily switch to UMD build (redux/dist/redux.min) which shouldn't have this problem.

[gaearon]

The way I see it, unless Lodash agrees to fix it on their end reasonably quickly by removing dependency on global in CommonJS modules, we have to do the following:

1. Release 3.2.1 and 4.2.1 that back out of using Lodash
2. Either wait for Lodash to fix this or...
3. Bump the major and tell CommonJS consumers we expect global to be polyfilled by their bundler when they use CommonJS build in the browser

I don't know how to enable global polyfill with Browserify but I assume it shouldn't be difficult.

[phated]

@gaearon browserify is supposed to pick up whether the global variable is reference and place it into the arguments list.

[gaearon]

Maybe detectGlobals Browserify option isn't true in your config? See lodash/lodash#915 (comment) which is about the same exact issue.

[phated]

^ was just going to post that - the other thing that might be wrong is having noParse set on Redux or all node_modules or something

[phated]

@gaearon in relation to #1346 (comment) - see #1339 (comment) where JD mentions he will be releasing isPlainObject on npm without the global reference in the next version.

[jdalton]

I can release the update this evening.

[gaearon]

Awesome, thank you for quick response!
I'll leave this open then, and when you're ready ping us, and I'll update the minimal version.

[jdalton]

Awesome, thank you for quick response!

No problem. Third-party integration, like Redux, is our top priority.

I'll leave this open then, and when you're ready ping us, and I'll update the minimal version.

You'll also be able to drop the global webpack plugin addition too.

[jaidetree]

Oooh awesome! Glad a fix is on the way. Anyway, that is a good catch I do have detect globals off and while enabling it would fix the immediate error it wasn't needed before so it seemed worth reporting.