Fix for non-browser environments

[oskarrough]

Hi, this fixes a bug where document is undefined in non-browser environment.

I saw aacee47 but needed this extra check as well.

[sindresorhus]

We already define document at the top of the file, so I don't really see what this is fixing.

[oskarrough]

In the case of Ember.js server-side rendering (ember fastboot), window is defined but window.document is not. How about this line instead?

var document = typeof window.document !== 'undefined' ? window.document : {};

[sindresorhus]

Sure

[kevva]

Wouldn't you need to check window too?

[oskarrough]

I think you're right. How about this?

var hasDocument = !window || typeof window.document !== 'undefined';
var document = hasDocument ? window.document : {};

[sindresorhus]

Needs to be:

var document = window && typeof window.document !== 'undefined' ? window.document : {};

[oskarrough]

Tak!

[el4v]

this fix broke server environments
/frontend/node_modules/screenfull/dist/screenfull.js:9
var document = window && typeof window.document !== 'undefined' ? window.document : {};
^

ReferenceError: window is not defined
at /frontend/node_modules/screenfull/dist/screenfull.js:9:17

[Tazer]

Same here :(

[oskarrough]

@EgorLyutov @Tazer can you verify whether this check works for you?

var document = typeof window !== 'undefined' && typeof window.document !== 'undefined' ? window.document : {};

Checking typeof window instead of just window should avoid the reference error, I hope.

[Tazer]

@oskarrough yes when changing that it works much better. So it works for me.