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

Use box/spout instead of phpoffice/phpexcel #61

Open
garak opened this issue Oct 15, 2017 · 9 comments · May be fixed by portphp/spout#1
Open

Use box/spout instead of phpoffice/phpexcel #61

garak opened this issue Oct 15, 2017 · 9 comments · May be fixed by portphp/spout#1

Comments

@garak
Copy link
Contributor

garak commented Oct 15, 2017

No description provided.

@ddeboer
Copy link
Member

ddeboer commented Dec 8, 2017

Why? What are the advantages?

@garak
Copy link
Contributor Author

garak commented Dec 8, 2017

Memory consumption on large data

@ddeboer
Copy link
Member

ddeboer commented Dec 10, 2017

Good point, @garak! Is this something you want to work on? We should create a separate portphp/spout repository for box/spout readers and writers.

@garak
Copy link
Contributor Author

garak commented Dec 10, 2017

OK

@ddeboer
Copy link
Member

ddeboer commented Dec 10, 2017

Created the repo at https://github.com/portphp/spout. @garak Please proceed by creating a PR against that repo. Feel free to add yourself as an author to the readme and composer.json. Thank you very much!

@aaa2000
Copy link
Contributor

aaa2000 commented Feb 8, 2018

@garak Did you start an implementation ?

@garak
Copy link
Contributor Author

garak commented Feb 9, 2018

not yet

@aaa2000 aaa2000 linked a pull request Mar 5, 2018 that will close this issue
2 tasks
@infomaniac50
Copy link
Contributor

I noticed the array buffering in portphp/excel. The next thing I was going to work was that. If PHPOffice/Spreadsheet has some kind of iterable interface, then we could use that instead of buffering with toArray()

@infomaniac50
Copy link
Contributor

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

Successfully merging a pull request may close this issue.

5 participants