Sheetjs: js-xlsx not support jszip 3.x

[hstarorg]

In https://github.com/SheetJS/js-xlsx/blob/master/xlsx.js#L1920, there used the global JsZip lib at first. In fact, the xlsx.js all packed the jszip 2.x, so I think use global jszip is unnecessary.

In my project, I used the JsZip as global variable, so the xlsx has a compatibility question.(not support jszip 3.x)

[reviewher]

JSZip 3.x removed critical features that we use, so you have to use 2.x.

Due to other changes, the script is included in the node module and the dist xlsx.full.min.js ships with a version of JSZip that plays nice.

[hstarorg]

Though the dist xlsx.full.min.js ships with a version of JSZip, the xlsx have priority use the global JsZip.
See code : https://github.com/SheetJS/js-xlsx/blob/master/xlsx.js#L1932

[martinnov92]

Can you reopen this issue please? Even when using xlsx.full.min.js it is not working

[hstarorg]

@martinnov92 In my project, I removed the code that use global JsZip.

[martinnov92]

@hstarorg This is not really solution in our project, we have a large code base :/ This should be fixed in xlsx in my opinion

[hstarorg]

@martinnov92 But I think it's not quickly.