Excel Serialization Module to Support PHPSpreadsheet ^2.0

I've faced today as well, interested on this to be merged asap. Thank you @crzdev

Yes this is much needed and it will be great if this module get a security release asap. Thanks

Worked for me

Hi,

Thank you for the update of dependency.
Would it be possible to merge it into 1.3 version?

K.R.

This MR was already committed... I think the status be changed to Fixed? But now this module just needs an updated release.

jhedstrom → credited theemstra → .

I've released 1.4 that contains this update.

Great.
Thank you guys.

As indicated by the automated test fail, the module still uses a removed method getCellByColumnAndRow()

This leads to a fatal error, so this call needs to be replace per https://github.com/PHPOffice/PhpSpreadsheet/issues/3930

Hmm never mind, that needs to be fixed in views_data_export. But the combo seems to be broken now

Thanks!! Please @theemstra, adjust issue credits if is ok four you

It was incorrect, it shouldn't go to ^2.0, but to ^2.2.1 which was done in 3cff0c74b5f81339b16814f81a4b4d1544ff9002

@jhedstrom Can you create a release for the patch? I've already tagged it.

^2.0 also allows to install ^2.2.1 too & avoid security advisories, besides issue detection, creation, specify proper description, testing & review was also work done and not credited, so not just talking for me, only you was credited... but is okay, is up to you, thanks anyway

While this new version fixes the security warning, shouldn't it be marked as a security release ?

Automatically closed - issue fixed for 2 weeks with no activity.