The patches fixes the error for me as well.
However,
false
should be in all caps,FALSE
, to conform to Drupal's coding standards.Here's a Drupal 7 patch that also includes the patch for ✨ Allow other mime types Fixed .
The section they patch are the same, so there are patch conflicts without manual adjustments. This should make that easier.
- Status changed to Needs review
over 1 year ago 8:35pm 20 July 2023 Here's a patch for Drupal 8+ that fixes both this issue and ✨ Allow other mime types Fixed . (Again because of patch conflicts.)
I also used short array syntax.
- Status changed to RTBC
over 1 year ago 3:01am 21 July 2023 - last update
over 1 year ago PHPLint Failed - Status changed to Needs work
over 1 year ago 9:30pm 7 August 2023 The patches all replaced
@exif_read_data
withexif_read_data
. I now get warnings when using a PNG file ( ✨ Allow other mime types Fixed ).@exif_read_data
should be used.- Status changed to Needs review
over 1 year ago 9:35pm 7 August 2023 - Status changed to RTBC
over 1 year ago 1:25pm 8 August 2023 - 🇨🇦Canada joelpittet Vancouver
That's fair though strange we explicitly check for png mime type, does it sometimes work?
It works, but it just shows a warning.
Errors of level E_WARNING and/or E_NOTICE may be raised for unsupported tags or other potential error conditions, but the function still tries to read all comprehensible information.
-
heddn →
committed dabc2dd7 on 8.x-1.x authored by
solideogloria →
Issue #3220370 by solideogloria, joelpittet, heddn: exif_read_data():...
-
heddn →
committed dabc2dd7 on 8.x-1.x authored by
solideogloria →
- Status changed to Fixed
over 1 year ago 5:14pm 21 August 2023 Automatically closed - issue fixed for 2 weeks with no activity.