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 output #508

Closed
basma-emad opened this issue Nov 24, 2016 · 3 comments
Closed

Empty output #508

basma-emad opened this issue Nov 24, 2016 · 3 comments

Comments

@basma-emad
Copy link

attached excel file is not parsed right, empty output was returned.
Zeinab shaker_CoverageList_20161103_02-37-34.xlsx

@masterzhang
Copy link

hi basma-emad
i have some problem,can you do me a favor?

when i run project,it has error

WARNING in ./~/.0.8.0@xlsx/jszip.js
Critical dependencies:
12:436-443 This seems to be a pre-built javascript file. Though this is possible, it's not recommended. Try to require the original source to get better results.
 @ ./~/.0.8.0@xlsx/jszip.js 12:436-443

ERROR in ./~/.0.8.0@xlsx/xlsx.js
Module not found: Error: Cannot resolve module 'fs' in F:\GitHub\YiSuan\node_modules\.0.8.0@xlsx
 @ ./~/.0.8.0@xlsx/xlsx.js 1204:27-40 1340:8-24

ERROR in ./~/.0.8.0@xlsx/ods.js
Module not found: Error: Cannot resolve module 'fs' in F:\GitHub\YiSuan\node_modules\.0.8.0@xlsx
 @ ./~/.0.8.0@xlsx/ods.js 58:8-24

ERROR in ./~/.0.8.0@xlsx/dist/cpexcel.js
Module not found: Error: Cannot resolve 'file' or 'directory' ./cptable in F:\GitHub\YiSuan\node_modules\.0.8.0@xlsx\dist
 @ ./~/.0.8.0@xlsx/dist/cpexcel.js 807:16-41

after i install fs, it still has this mistake

@basma-emad
Copy link
Author

try adding this line
var fs = require("fs");

@SheetJSDev
Copy link
Contributor

@basma-emad the file was not being parsed properly because of some unexpected namespaced xml tags. The relevant checks have been expanded.

saarCiklum pushed a commit to Folcon/js-xlsx that referenced this issue Aug 17, 2020
- basic support for parsing BIFF2-4
- basic support for writing BIFF2
- cleaned up some bad substr uses for IE6 compatibility
- added flow type annotations for xlsx.flow.js
- added numerous null guards (fixes SheetJS#255 h/t @martinheidegger)
- README cleanup (fixes SheetJS#539 h/t @oliversalzburg)
- pin jszip to local version (closes SheetJS#408 h/t @limouri)

bower issues:

|  id  | author            | comment                                   |
|-----:|:------------------|:------------------------------------------|
| SheetJS#254 | @kkirsche         | fixes SheetJS#254 by removing version from json  |
| protobi#165 | @vincentcialdella | fixes protobi#165 by changing default script     |
| protobi#180 | @owencraig        | fixes protobi#180 by using xlsx.core.min.js      |

format issues:

|  id  | author            | comment                                   |
|-----:|:------------------|:------------------------------------------|
| SheetJS#271 | @morstaine        | fixes SheetJS#271 by reworking related parse fns |
| SheetJS#504 | @JanSchuermannPH  | fixes SheetJS#504 detect FullPaths h/t @Mithgol  |
| SheetJS#508 | @basma-emad       | fixes SheetJS#508 offending file used `x:` NS    |
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

3 participants