-
Notifications
You must be signed in to change notification settings - Fork 146
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
Is there no longer support for XLS(X)? #28
Comments
Hi john, it's not included by design since support for xlsx format isn't fully ready.. |
Ok, thanks for the update. Is there an ETA on that? |
No ETA yet, I'm open if anyone want to contribute 😁 |
May do so cause I need it! :) Do you have requirements, ideas, etc. sketched out or listed anywhere? |
Any update on this? |
Not to my knowledge. What we ended up doing was integrating another library (doh!) to simply convert any files that were xls/xlsx into csv format and then proceed with the code we'd already written for parsing through files with SimpleExcel. We used PHPExcel for that bridge: http://phpexcel.codeplex.com/wikipage?title=Examples Still find it funny that the file format for which the library was named isn't supported! |
@johnblythe that's exactly what I did as well. @faisalman Can you help us out? We're excited to only use one library! |
@faisalman pls, remove XLSX from heading of project. As your project does not support that format, but in google your repo in top10 of results by query like "php parse write xlsx" |
Mas bro, ini gk bisa write XLS / XLSX ya? Tapi di descriptionnya ditulis 😄 |
There are still files related to parsing and writing to XLSX, but this filetype is not included in the validXTypes definitions in SimpleExcel.php, lines 99 & 100:
$this->validParserTypes = array('XML', 'CSV', 'TSV', 'HTML', 'JSON');
$this->validWriterTypes = array('XML', 'CSV', 'TSV', 'HTML', 'JSON');
Am I overlooking something or has support been dropped, put on hold, etc?
The text was updated successfully, but these errors were encountered: