Make WordPress Core

Opened 3 months ago

Last modified 3 months ago

#61983 new defect (bug)

Part of `wp_generate_attachment_metadata` missing `avif` extension

Reported by: adamsilverstein's profile adamsilverstein Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 6.5
Component: Media Keywords: has-patch
Focuses: Cc:

Description

This ticket proposes a fix to wp_generate_attachment_metadata which is missing extension support for AVIF images. The code path in question is only used as a fallback, it would be good to add a test that validates the fix.

Change History (2)

This ticket was mentioned in PR #7291 on WordPress/wordpress-develop by @adamsilverstein.


3 months ago
#1

  • Keywords has-patch added

#2 @ironprogrammer
3 months ago

Thanks for the report and patch, @adamsilverstein 🙌🏻

Yeah, wpGenerateAttachmentMetadata.php doesn't cover that kind of scenario yet. What are the conditions required to validate the use of the fallback extension? When the attachment is video/audio, then $metadata['image']['data'] isn't set, which skips that path.

Last edited 3 months ago by ironprogrammer (previous) (diff)
Note: See TracTickets for help on using tickets.