Avoid crashes on environments without process.env. Fixes #39

reduxjs · Aug 9, 2015 · 238e7d9 · 238e7d9
1 parent b7ef178
commit 238e7d9
Showing 1 changed file with 26 additions and 17 deletions.
diff --git a/src/components/createConnect.js b/src/components/createConnect.js
@@ -144,23 +144,6 @@ export default function createConnect(React) {
           this.trySubscribe();
         }
 
-        componentWillUpdate() {
-          if (process.env.NODE_ENV !== 'production') {
-            if (this.version === version) {
-              return;
-            }
-
-            // We are hot reloading!
-            this.version = version;
-
-            // Update the state and bindings.
-            this.trySubscribe();
-            this.recomputeStateProps();
-            this.recomputeDispatchProps();
-            this.recomputeState();
-          }
-        }
-
         componentWillReceiveProps(nextProps) {
           if (!shallowEqual(nextProps, this.props)) {
             this.recomputeState(nextProps);
@@ -189,6 +172,32 @@ export default function createConnect(React) {
         }
       }
 
+      if ((
+        // Node-like CommonJS environments (Browserify, Webpack)
+        typeof process !== 'undefined' &&
+        typeof process.env !== 'undefined' &&
+        process.env.NODE_ENV !== 'production'
+       ) ||
+        // React Native
+        typeof __DEV__ !== 'undefined' &&
+        __DEV__ //eslint-disable-line no-undef
+      ) {
+        Connect.prototype.componentWillUpdate = function componentWillUpdate() {
+          if (this.version === version) {
+            return;
+          }
+
+          // We are hot reloading!
+          this.version = version;
+
+          // Update the state and bindings.
+          this.trySubscribe();
+          this.recomputeStateProps();
+          this.recomputeDispatchProps();
+          this.recomputeState();
+        };
+      }
+
       return Connect;
     };
   };