[wp-trac] [WordPress Trac] #31050: Better PDF Upload Management
WordPress Trac
noreply at wordpress.org
Sat Feb 13 00:07:00 UTC 2016
#31050: Better PDF Upload Management
-------------------------------------------------+-------------------------
Reporter: celloexpressions | Owner:
Type: feature request | Status: new
Priority: normal | Milestone: 4.5
Component: Media | Version:
Severity: normal | Resolution:
Keywords: has-patch dev-feedback needs-unit- | Focuses: ui
tests needs-testing |
-------------------------------------------------+-------------------------
Comment (by mikeschroder):
> Replying to [comment:24 markoheijnen]:
> > Fine with only PDF. Was just checking if we could support more.
Fallback can be removed since the same check is inside
wp_image_editor_supports.
Ah, nevermind, you are correct -- instantiating with
`wp_get_image_editor()` uses `_wp_image_editor_choose()` internally, which
is what `wp_image_editor_supports()` runs before loading the image, so
we're fine to skip checking for editor support ahead of time, like you did
in the previous patch.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/31050#comment:27>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list