Add error handling on window functions

repetere · Jan 17, 2018 · 34ddccd · 34ddccd
1 parent c41b0a2
commit 34ddccd
Show file tree

Hide file tree

Showing 4 changed files with 64 additions and 8 deletions.
diff --git a/dist/rjx.cjs.js b/dist/rjx.cjs.js
@@ -761,9 +761,23 @@ function getFunctionFromProps(options) {
 
     if (propFunc.indexOf('func:window') !== -1) {
       if (functionNameArray.length === 3) {
-        return window[functionNameArray[1]][functionName].bind(this);
+        try {
+          return window[functionNameArray[1]][functionName].bind(this);
+        } catch (e) {
+          if (debug) {
+            logError(e);
+          }
+          return window[functionNameArray[1]][functionName];
+        }
       } else {
-        return window[functionName].bind(this);
+        try {
+          return window[functionName].bind(this);
+        } catch (e) {
+          if (debug) {
+            logError(e);
+          }
+          return window[functionName];
+        }
       }
     } else if (functionNameArray.length === 4) {
       return this.props[functionNameArray[2]][functionName];

diff --git a/dist/rjx.esm.js b/dist/rjx.esm.js
@@ -755,9 +755,23 @@ function getFunctionFromProps(options) {
 
     if (propFunc.indexOf('func:window') !== -1) {
       if (functionNameArray.length === 3) {
-        return window[functionNameArray[1]][functionName].bind(this);
+        try {
+          return window[functionNameArray[1]][functionName].bind(this);
+        } catch (e) {
+          if (debug) {
+            logError(e);
+          }
+          return window[functionNameArray[1]][functionName];
+        }
       } else {
-        return window[functionName].bind(this);
+        try {
+          return window[functionName].bind(this);
+        } catch (e) {
+          if (debug) {
+            logError(e);
+          }
+          return window[functionName];
+        }
       }
     } else if (functionNameArray.length === 4) {
       return this.props[functionNameArray[2]][functionName];

diff --git a/dist/rjx.umd.js b/dist/rjx.umd.js
@@ -27643,9 +27643,23 @@ function getFunctionFromProps(options) {
 
     if (propFunc.indexOf('func:window') !== -1) {
       if (functionNameArray.length === 3) {
-        return window[functionNameArray[1]][functionName].bind(this);
+        try {
+          return window[functionNameArray[1]][functionName].bind(this);
+        } catch (e) {
+          if (debug) {
+            logError(e);
+          }
+          return window[functionNameArray[1]][functionName];
+        }
       } else {
-        return window[functionName].bind(this);
+        try {
+          return window[functionName].bind(this);
+        } catch (e) {
+          if (debug) {
+            logError(e);
+          }
+          return window[functionName];
+        }
       }
     } else if (functionNameArray.length === 4) {
       return this.props[functionNameArray[2]][functionName];

diff --git a/src/props.js b/src/props.js
@@ -165,9 +165,23 @@ export function getFunctionFromProps(options) {
 
     if (propFunc.indexOf('func:window') !== -1) {
       if (functionNameArray.length === 3) {
-        return window[ functionNameArray[ 1 ] ][ functionName ].bind(this);
+        try {
+          return window[ functionNameArray[ 1 ] ][ functionName ].bind(this);
+        } catch (e) {
+          if (debug) {
+            logError(e);
+          }
+          return window[ functionNameArray[ 1 ] ][ functionName ];
+        }
       } else {
-        return window[ functionName ].bind(this);
+        try {
+          return window[ functionName ].bind(this);
+        } catch (e) {
+          if (debug) {
+            logError(e);
+          }
+          return window[ functionName ];
+        }
       }
     } else if (functionNameArray.length === 4) {
       return this.props[ functionNameArray[ 2 ] ][ functionName ];