protobi / js-xlsx

XLSX / XLSM / XLSB (Excel 2007+ Spreadsheet) / ODS parser and writer
http://oss.sheetjs.com/js-xlsx
Other
814 stars 416 forks source link

Error in use by requirejs #121

Open randyou opened 5 years ago

randyou commented 5 years ago

https://github.com/SheetJS/js-xlsx/tree/master/demos/requirejs

require(["xlsx"], function(XLSX) {
    /* use XLSX here */
});

XLSX is not XLSX which is JSZip !

then fix it in xlsx.full.min.js like below

!function(e) {
    if ("object" == typeof exports && "undefined" != typeof module)
        module.exports = e();
    else if ("function" == typeof define && define.amd) {
       //fix here
        window.JSZip = e();
        define([], function() {
            return window.XLSX;
        });
    } else {
        var f;
        "undefined" != typeof window ? f = window : "undefined" != typeof global ? f = global : "undefined" != typeof self && (f = self),
        f.JSZip = e()
    }
}
....