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.

Sign up for GitHub

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

Jump to bottom

Replace PHPExcel library with PHPSpreadsheet #8267

Closed
escopecz opened this issue Dec 30, 2019 · 0 comments
Closed

Replace PHPExcel library with PHPSpreadsheet #8267

escopecz opened this issue Dec 30, 2019 · 0 comments
Assignees
@mtshaw3
Projects
Mautic 3
Milestone
3.0.0-beta

Comments

@escopecz
Copy link
Sponsor Member

When you run composer install you'll get this warning:

Package phpoffice/phpexcel is abandoned, you should avoid using it. Use phpoffice/phpspreadsheet instead.

Let's replace the outdated library with its successor:

https://github.com/PHPOffice/PhpSpreadsheet

I made a quick search. It will need some refactoring in FormBundle and ReportBundle when the data are being exported to Excel.
@escopecz escopecz created this issue from a note in Mautic 3 (To do) Dec 30, 2019
@escopecz escopecz added this to the 3.0.0 milestone Dec 30, 2019
@escopecz escopecz added the essential This must be done to close the milestone label Dec 30, 2019
@escopecz escopecz moved this from To do to In progress in Mautic 3 Dec 31, 2019
@mtshaw3 mtshaw3 mentioned this issue Jan 2, 2020
Merged
@mtshaw3 mtshaw3 moved this from In progress to Needs code review and/or test in Mautic 3 Jan 7, 2020
@escopecz escopecz removed the essential This must be done to close the milestone label Jan 8, 2020
@escopecz escopecz closed this as completed Jan 8, 2020
Mautic 3 automation moved this from Needs code review and/or test to Done Jan 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mtshaw3 mtshaw3

Labels
None yet
Projects
No open projects
Mautic 3
  
Done
Milestone
3.0.0-beta
Development

No branches or pull requests
2 participants
@escopecz @mtshaw3