Fix import from Node environment (to not break in server side rendering)

[gre]

the library is breaking in Node environment with a window is not defined – not at runtime, but at import time. that PR fixes this.

my only usecase is to have the library working with Server Side Rendering (in context of react). (in my case, the library don't actually gets used in Node, but it's still imported and embedded in the server bundle that will break before this PR)

[blueimp]

Thanks for your contribution, @gre !

I've just two change requests:

In js/load-image.js, could you change the urlAPI declaration to the following

  var urlAPI = ($.createObjectURL && $) ||
                ($.URL && URL.revokeObjectURL && URL) ||
                ($.webkitURL && webkitURL)

and in the same file, change window.FileReader to $.FileReader for consistency.

[blueimp]

And please make sure the code passes npm test :)

[gre]

fixed and test pass

[ram-you]

The new "2.13.0" breaks my project.
With "2.12.2" this works :

import loadImage from 'blueimp-load-image/js'
window.loadImage = loadImage

With the new version it doesn't work anymore.

[gre]

oops i'm not 100% sure but, I think reason is that (this) does not yield window in context of Browserify/Babel , but not sure?

@blueimp maybe we should drop the scopping thing and just typeof everywhere? in load-image.js ?

if it's not the reason, then it must be another issue?

[gre]

@ram-you have you tried to use import loadImage from 'blueimp-load-image' without the /js ? it seems to work for me mmh

[ram-you]

@gre

import loadImage from 'blueimp-load-image'

Sorry, but your solution doesn't work for me.

[blueimp]

@ram-you I've just released version 2.13.1 which should fix the issue.

[ram-you]

Thank you @blueimp . It's works like a charm