Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Empty records array with node v0.12.0 #33

Open
mechanicalduck opened this issue Mar 26, 2015 · 10 comments
Open

Empty records array with node v0.12.0 #33

mechanicalduck opened this issue Mar 26, 2015 · 10 comments

Comments

@mechanicalduck
Copy link

mechanicalduck commented Mar 26, 2015

After upgrading node from 0.10.36 to 0.12.0,
the records array given to callback from parseXlsx is just an empty array.

The input xlsx file haven't been changed and
downgrading back to node v0.10.32 resolves the issue.

@Magiccamera
Copy link

Magiccamera commented Sep 16, 2016

This works when running job via terminal "node index.js" spits out the content of xlsx file no problem. But when I run it in the app via a drag and drop the xlsx file array comes back empty and will not render anything in html page. Might be something in our script to read our file, but can't isolate any issue.

@mtnoronha
Copy link

I'm using Node v7.1.0 and I also get an empty array :(

Running from 'node index.js'

@clickclickonsal
Copy link
Collaborator

I verified that this isn't working with v7.1.0, so I will try and fix this over the weekend.
As a work around for now, this does work in node v6.9.x 😄

As for node v.0.12.0, I don't have any plans for supporting any version less than the 6.x.x.

@carmilso
Copy link

carmilso commented Apr 7, 2017

FYI, I have tested it with node v6.10.2 and it does not work either.

@moyuanhua
Copy link

@carmilso me too

@mrbatista
Copy link

Same problem node v6.10.3

@clickclickonsal
Copy link
Collaborator

clickclickonsal commented Jul 17, 2017

Is this still an issue?
Sometimes it helps to isolate the code.

var parseXlsx = require('excel');

parseXlsx('./Workbook1.xlsx', function(err, data) {
  if(err) {
    console.log(err)
  }

  console.log(data)
    // data is an array of arrays
});

I ran this with node 6.10.2 and it works just fine.

@trevordixon
Copy link
Owner

Thanks for the reports everybody.

@mrbatista @jackmo2014 @carmilso @clickclickonsal @mtnoronha
Out of curiosity, do you know about https://github.com/SheetJS/js-xlsx? If so, why use excel.js instead of that? I haven't worked on this in a long time, and I wonder if I should just point people to that other option, or if there's something compelling about excel.js that I should make more clear or enhance.

@mtnoronha
Copy link

Trevor, I didnt know about that.

I actually dicided to go for:
https://www.npmjs.com/package/xlsx

But, not the prettiest syntax/usage though.

@shivampandey2021
Copy link

I also am getting an empty array. My node.js version is v8.9.1. Checked on node.js version v6.10.3 and it gives an array of data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants