#39821 closed enhancement (duplicate)
Please make image extensions array filterable in wp_attachment_is()
| Reported by: | Hinjiriyo | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Media | Version: | 4.7.2 |
| Severity: | normal | Keywords: | |
| Cc: | Focuses: |
Description
The function wp_attachment_is()[1] checks the type of an attachment. While the checks for audio files and video files call subfunctions with filterable arrays of extensions [2][3] the check for image files uses a unfilterable - thus immutable - array.
Please
1) make the array of image file extensions filterable like for audios and videos to let more image file formats allowed in WP
2) set that line in a subfunction just to be consistent with the functions for audios and videos, lets call it wp_get_image_extensions()
[1] wp_attachment_is() in /wp-includes/post.php
[2] wp_get_audio_extensions() in /wp-includes/media.php
[2] wp_get_video_extensions() in /wp-includes/media.php
Attachments (1)
Change History (5)
This ticket was mentioned in Slack in #core-media by mike. View the logs.
8 years ago
#3
@
8 years ago
- Keywords has-patch removed
- Milestone Awaiting Review
- Resolution → duplicate
- Status new → closed
Thanks so much for the report!
Because this is currently in a milestone with a patch in #41801, going to mark this ticket as a duplicate for now.
If you think that ticket does not cover what you're asking for, please feel free to comment here and the ticket can be reopened.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
39821.diff Adds a filter inline, named to match the audio and video equivalents. I'm not sure there's a need to pull this out into a function like the others, aside from consistency.