[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