search

experius / Magento-2-Module-Experius-WysiwygDownloads

Available At Packagist
https://packagist.org/search/?q=experius
Open Software License 3.0
98 stars 24 forks source link

GD2 interaction broken after upgrading extension #36

Closed indykoning closed 2 days ago

indykoning commented 3 months ago

Hi, after upgrading this module above 1.2.0 calling functions like getOriginalHeight() on the image always returns null.

This seems to be due to this commit

https://github.com/experius/Magento-2-Module-Experius-WysiwygDownloads/commit/f8d3824ccb316034f0b593f185c2ecab93fffd03

in combination with this check https://github.com/experius/Magento-2-Module-Experius-WysiwygDownloads/blob/master/Image/Adapter/Gd2.php#L49

where it will no longer open the file if it matches the whitelisted file pattern

zhovnirigor commented 2 months ago

Fixed by https://github.com/experius/Magento-2-Module-Experius-WysiwygDownloads/pull/38

iriks-it commented 2 days ago

Fixed in 1.2.4, Please upgrade.