Skip to content

Commit

Permalink
fix: Fix global resolution for Safari
Browse files Browse the repository at this point in the history
  • Loading branch information
medikoo committed Oct 29, 2019
1 parent ae908fb commit 00731d2
Show file tree
Hide file tree
Showing 2 changed files with 31 additions and 8 deletions.
34 changes: 27 additions & 7 deletions global.js
Original file line number Diff line number Diff line change
@@ -1,15 +1,35 @@
var naiveFallback = function () {
if (typeof self === "object" && self) return self;
if (typeof window === "object" && window) return window;
throw new Error("Unable to resolve global `this`");
};

module.exports = (function () {
if (this) return this;

// Unexpected strict mode (may happen if e.g. bundled into ESM module), be nice
// Unexpected strict mode (may happen if e.g. bundled into ESM module)

// Fallback to standard globalThis if available
if (typeof globalThis === "object" && globalThis) return globalThis;

// Thanks @mathiasbynens -> https://mathiasbynens.be/notes/globalthis
// In all ES5+ engines global object inherits from Object.prototype
// (if you approached one that doesn't please report)
Object.defineProperty(Object.prototype, "__global__", {
get: function () { return this; },
configurable: true
});
try { return __global__; }
finally { delete Object.prototype.__global__; }
try {
Object.defineProperty(Object.prototype, "__global__", {
get: function () { return this; },
configurable: true
});
} catch (error) {
// Unfortunate case of updates to Object.prototype being restricted
// via preventExtensions, seal or freeze
return naiveFallback();
}
try {
// Safari case (window.__global__ works, but __global__ does not)
if (!__global__) return naiveFallback();
return __global__;
} finally {
delete Object.prototype.__global__;
}
})();
5 changes: 4 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,10 @@
{
"files": "global.js",
"globals": {
"__global__": true
"__global__": true,
"globalThis": true,
"self": true,
"window": true
},
"rules": {
"strict": "off"
Expand Down

0 comments on commit 00731d2

Please sign in to comment.